From 19c735261f4f97bd6f5ecc1444a51b29b908da6e Mon Sep 17 00:00:00 2001
From: steven <stevenlgtm@gmail.com>
Date: Thu, 29 Sep 2022 22:51:12 +0800
Subject: [PATCH] chore: update button styles

---
 web/src/components/Alert.tsx                | 6 +++---
 web/src/components/ChangePasswordDialog.tsx | 4 ++--
 web/src/components/WorkspaceSetting.tsx     | 6 ++----
 web/src/pages/WorkspaceDetail.tsx           | 1 +
 4 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/web/src/components/Alert.tsx b/web/src/components/Alert.tsx
index 81b6f58..fa7e180 100644
--- a/web/src/components/Alert.tsx
+++ b/web/src/components/Alert.tsx
@@ -49,12 +49,12 @@ const Alert: React.FC<Props> = (props: Props) => {
       <DialogContent className="w-80">
         <p className="content-text mb-4">{content}</p>
         <div className="w-full flex flex-row justify-end items-center">
-          <button className="rounded px-3 py-2 mr-2 hover:opacity-80" onClick={handleCloseBtnClick}>
+          <button className="rounded px-3 leading-9 mr-4 hover:opacity-80" onClick={handleCloseBtnClick}>
             {closeBtnText}
           </button>
           <button
-            className={`rounded px-3 py-2 shadow bg-green-600 text-white hover:opacity-80 ${
-              style === "warning" ? "border-red-600 text-red-600 bg-red-100" : ""
+            className={`rounded px-3 leading-9 bg-green-600 text-white hover:opacity-80 ${
+              style === "warning" ? "border border-red-600 text-red-600 bg-red-100" : ""
             }`}
             onClick={handleConfirmBtnClick}
           >
diff --git a/web/src/components/ChangePasswordDialog.tsx b/web/src/components/ChangePasswordDialog.tsx
index e72e6f3..f0adce8 100644
--- a/web/src/components/ChangePasswordDialog.tsx
+++ b/web/src/components/ChangePasswordDialog.tsx
@@ -101,14 +101,14 @@ const ChangePasswordDialog: React.FC<Props> = (props: Props) => {
         <div className="w-full flex flex-row justify-end items-center">
           <button
             disabled={requestState.isLoading}
-            className={`rounded px-3 py-2 mr-2 hover:opacity-80 ${requestState.isLoading ? "opacity-80" : ""}`}
+            className={`rounded px-3 leading-9 mr-4 hover:opacity-80 ${requestState.isLoading ? "opacity-80" : ""}`}
             onClick={handleCloseBtnClick}
           >
             Cancel
           </button>
           <button
             disabled={requestState.isLoading}
-            className={`rounded px-3 py-2 shadow bg-green-600 text-white hover:bg-green-700 ${requestState.isLoading ? "opacity-80" : ""}`}
+            className={`rounded px-3 leading-9 bg-green-600 text-white hover:bg-green-700 ${requestState.isLoading ? "opacity-80" : ""}`}
             onClick={handleSaveBtnClick}
           >
             Save
diff --git a/web/src/components/WorkspaceSetting.tsx b/web/src/components/WorkspaceSetting.tsx
index 454d98e..f78657f 100644
--- a/web/src/components/WorkspaceSetting.tsx
+++ b/web/src/components/WorkspaceSetting.tsx
@@ -107,11 +107,9 @@ const WorkspaceSetting: React.FC<Props> = (props: Props) => {
   return (
     <>
       <div className="w-full flex flex-col justify-start items-start">
-        <p className="text-3xl mt-2 mb-4">{state.workspace.name}</p>
-        <p>{state.workspace.description}</p>
-
+        <p className="text-3xl mt-4 mb-4">{state.workspace.name}</p>
+        <p className="mb-4">{state.workspace.description}</p>
         <div className="border-t pt-4 mt-2 flex flex-row justify-start items-center">
-          <span className="text-gray-400 mr-2">Actions:</span>
           <div className="flex flex-row justify-start items-center space-x-2">
             {state.workspaceUser.role === "ADMIN" ? (
               <>
diff --git a/web/src/pages/WorkspaceDetail.tsx b/web/src/pages/WorkspaceDetail.tsx
index ac71b2b..b3d2c6c 100644
--- a/web/src/pages/WorkspaceDetail.tsx
+++ b/web/src/pages/WorkspaceDetail.tsx
@@ -142,6 +142,7 @@ const WorkspaceDetail: React.FC = () => {
               {location.hash === "#shortcuts" && <ShortcutListView workspaceId={state.workspace.id} shortcutList={shortcutList} />}
               {location.hash === "#members" && (
                 <MemberListView
+                  // enforce to re-fetch member list.
                   key={Date.now()}
                   workspaceId={state.workspace.id}
                   workspaceUser={state.workspaceUser}