IceCubesApp icon indicating copy to clipboard operation
IceCubesApp copied to clipboard

Dark theme background colors are flipped

Open firatciftci opened this issue 1 year ago • 3 comments

When Nemesis Light Mode is selected, the background color of posts and pressable menu items in the settings screen are rendered #ffffff, with their surroundings rendered off-white. This is expected behavior. However, in Nemesis Dark Mode, the primary and secondary background colors don't follow expected behavior for Apple's theming guidelines. Specifically, the pressable menu items in the settings page should be #1c1c1e, and the color of the background view should be #000000, with the color of posts matching the background view's color this time around.

A good example for how this should look is the Settings app:

Settings (Light) Settings (Dark)
IMG_1558 IMG_1559

Compare this with Ice Cubes's theming method:

Ice Cubes (Light) Ice Cubes (Dark)
IMG_1570 IMG_1571

Continuing on this, I would also love to see a "default iOS look" theme; given that the app is written using SwiftUI and greatly matches Apple's own design aesthetic, I think following Apple's Foundations colors would make a lot of sense. I assume this could easily be added as two new theme options. More specifically, for the light theme, background color is #ffffff, secondary background color is f2f2f7, links are #007aff, and for the dark theme, background color is #000000, secondary background color is #1c1c1e, and links are #0A84ff. That said, it might be cool to see two dark modes, one regular and another AMOLED, as well.

firatciftci avatar Jan 03 '23 20:01 firatciftci

Here is how the app looks with the proposed "default iOS look" themes:

Light Mode (Regular) Dark Mode (Regular) Dark Mode (AMOLED)
IMG_1573 IMG_1580 IMG_1578

And the settings page (of course, as noted in the original post above, dark mode colors need to be flipped):

Light Mode (Regular) Dark Mode (Regular) Dark Mode (AMOLED)
IMG_1572 IMG_1579 IMG_1576

For the regular dark mode, I am using #1c1c1a for the background color and #2c2c2e for the secondary background color.

firatciftci avatar Jan 03 '23 20:01 firatciftci

Also, while creating this issue, I have stumbled upon another bug: while a light theme (i.e. a theme with dark text) is selected, if you change the background color to a dark one, the text stays dark, rendering everything unreadable. To fix it, you have to first pick a dark theme (i.e. a theme with light text) and then modify the colors.

I had Nemesis Light Mode selected and changed the background colors, resulting in the following: IMG_1575

firatciftci avatar Jan 03 '23 20:01 firatciftci

Quickly: the flip is actually a conscious design decision. I was wondering if anyone would comment on it haha.

Dimillian avatar Jan 03 '23 21:01 Dimillian