[GH-ISSUE #2297] [Bug] The icon picker for the workspace name dark mode #958

Closed
opened 2026-03-23 20:43:39 +00:00 by mirror · 2 comments
Owner

Originally created by @annieappflowy on GitHub (Apr 19, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2297

Originally assigned to: @akarsh-jain-790 on GitHub.

Bug Description

image

There isn't a hover state when hovering on an icon
Doesn't look as polished as other components in the dark mode

How to Reproduce

Go to Settings
Go to User
Switch to use another icon

Expected Behavior

Hovering on an icon has a hover state
The menu looks better

Operating System

MacOS

AppFlowy Version(s)

Main branch

Screenshots

No response

Additional Context

No response

Originally created by @annieappflowy on GitHub (Apr 19, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2297 Originally assigned to: @akarsh-jain-790 on GitHub. ### Bug Description <img width="411" alt="image" src="https://user-images.githubusercontent.com/12026239/233064594-c58c0a4f-b3e3-4e9d-96fc-eefe9ead6aba.png"> There isn't a hover state when hovering on an icon Doesn't look as polished as other components in the dark mode ### How to Reproduce Go to Settings Go to User Switch to use another icon ### Expected Behavior Hovering on an icon has a hover state The menu looks better ### Operating System MacOS ### AppFlowy Version(s) Main branch ### Screenshots _No response_ ### Additional Context _No response_
mirror 2026-03-23 20:43:39 +00:00
Author
Owner

@akarsh-jain-790 commented on GitHub (Apr 25, 2023):

@annieappflowy I would like to work on this.

<!-- gh-comment-id:1521302328 --> @akarsh-jain-790 commented on GitHub (Apr 25, 2023): @annieappflowy I would like to work on this.
Author
Owner

@hyj1204 commented on GitHub (Apr 26, 2023):

@akarsh-jain-790
I think the background color should be cardColor from the Theme and hover color you can refer secondaryContainer or tertiaryContainer to see which one is better.

<!-- gh-comment-id:1523636079 --> @hyj1204 commented on GitHub (Apr 26, 2023): @akarsh-jain-790 I think the background color should be `cardColor` from the Theme and hover color you can refer `secondaryContainer` or `tertiaryContainer` to see which one is better.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#958
No description provided.