FreeOTPPlus icon indicating copy to clipboard operation
FreeOTPPlus copied to clipboard

Change the background of icons in the dark mode

Open fullcavediver opened this issue 3 years ago • 11 comments

Please change the background of the icon to black for the dark mode (the same color as the background of the key / name frame). Many Thanks!

fullcavediver avatar May 10 '21 07:05 fullcavediver

It is something that has been bothering me a little bit too. If the maintainers are ok, I can fix that. Here is what I'm thinking about. Current version: image What I can do: image

Would that be ok?

carlphilipp avatar May 11 '21 00:05 carlphilipp

The gray theme was added to improve the contrast of some dark icons. If you make the background too dark, some darker icons may not be visible. The correct fix would be to make background adaptable to the icon. https://github.com/helloworld1/FreeOTPPlus/issues/63

helloworld1 avatar May 11 '21 01:05 helloworld1

It is something that has been bothering me a little bit too. If the maintainers are ok, I can fix that. Here is what I'm thinking about. Current version: image What I can do: image

Would that be ok?

Hi Carl-Philipp, yes, your proposal is perfect! That's exactly how I imagined it to be.

fullcavediver avatar May 11 '21 05:05 fullcavediver

The gray theme was added to improve the contrast of some dark icons. If you make the background too dark, some darker icons may not be visible. The correct fix would be to make background adaptable to the icon. Gray is too noticeable. #63

I understand, but the icon can be changed. Black background looks more elegant

fullcavediver avatar May 11 '21 05:05 fullcavediver

I have another suggestion - themes. I know it's a lot more work, but design is the second most important attribute besides functions. You can build in 3 themes - light, dark and custom with the options: text color, background color, primary color,... simply the most important colors to choose from

fullcavediver avatar May 11 '21 06:05 fullcavediver

@helloworld1 I'm not a UI person, but it sounds like what you are describing would be with having different background depending on the actual icon. And I'm not sure that would be very attractive visually. My proposition was more around having the same background but a dark version of the icon if needed. That's how I have seen it done in other apps

carlphilipp avatar May 12 '21 00:05 carlphilipp

I also think that the undepending black background for all icons is much nicer than an adaptive one. Everyone always has the possibility to choice an optimal icon

fullcavediver avatar May 12 '21 04:05 fullcavediver

unnamed

Do you think this design is elegant?

fullcavediver avatar May 12 '21 05:05 fullcavediver

Yes it does look great if all icons are not using black elements. Unfortunately some icons are designed only for light theme. Maybe another option is fixing all those icons to use lighter gray so it's compatible with both light and dark theme.

helloworld1 avatar May 12 '21 06:05 helloworld1

Also, three dot button (edit/delete menu) is difficult to see in standard and dark modes. Three dots are displayed in black on dark gray. The button is hard to find.

mrfuggy avatar Sep 07 '21 00:09 mrfuggy

Goddammit, this is over 2 years old now? I guess this won't be fixed. Too bad the dark theme can't be turned off anymore if the device itself is set to dark mode...

praecipitator avatar Jan 15 '24 09:01 praecipitator