From 562013e3bdb5094bda946ae3ad965b7cf4f6f69d Mon Sep 17 00:00:00 2001
From: JimmFly <yangjinfei001@gmail.com>
Date: Mon, 3 Jul 2023 16:46:16 +0800
Subject: [PATCH] style: adjust icon button hover color (#2976)

---
 apps/web/src/components/pure/help-island/style.ts | 2 --
 packages/component/src/ui/button/styles.ts        | 4 ++--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/apps/web/src/components/pure/help-island/style.ts b/apps/web/src/components/pure/help-island/style.ts
index 2205013d4f..f7dab69a56 100644
--- a/apps/web/src/components/pure/help-island/style.ts
+++ b/apps/web/src/components/pure/help-island/style.ts
@@ -50,7 +50,6 @@ export const StyledIconWrapper = styled('div')(() => {
     transition: 'background-color 0.2s',
     position: 'relative',
     ':hover': {
-      color: 'var(--affine-primary-color)',
       backgroundColor: 'var(--affine-hover-color)',
     },
   };
@@ -74,7 +73,6 @@ export const StyledTriggerWrapper = styled('div')<{
     ...displayFlex('center', 'center'),
     ...positionAbsolute({ left: '4px', bottom: '4px' }),
     ':hover': {
-      color: 'var(--affine-primary-color)',
       backgroundColor: spread ? 'var(--affine-hover-color)' : null,
     },
   };
diff --git a/packages/component/src/ui/button/styles.ts b/packages/component/src/ui/button/styles.ts
index 1df90306fb..41b7717b51 100644
--- a/packages/component/src/ui/button/styles.ts
+++ b/packages/component/src/ui/button/styles.ts
@@ -51,7 +51,7 @@ export const StyledIconButton = styled('button', {
       borderRadius,
 
       ':hover': {
-        color: hoverColor ?? 'var(--affine-primary-color)',
+        color: hoverColor ?? 'var(--affine-icon-color)',
         background: hoverBackground || 'var(--affine-hover-color)',
         ...(hoverStyle ?? {}),
       },
@@ -169,7 +169,7 @@ export const StyledButton = styled('button', {
       border: noBorder ? 'none' : '1px solid',
       WebkitAppRegion: 'no-drag',
       ...displayInlineFlex('center', 'center'),
-      gap: '10px',
+      gap: '8px',
       position: 'relative',
       // TODO: disabled color is not decided
       ...(disabled