Docile-Alligator / Infinity-For-Reddit

A Reddit client for Android
https://play.google.com/store/apps/details?id=ml.docilealligator.infinityforreddit
GNU Affero General Public License v3.0
4.26k stars 410 forks source link

"Color Primary Light Theme" is no longer applied to the floating action bar, despite the subtitle in the settings #1519

Closed SuperStormer closed 5 months ago

SuperStormer commented 1 year ago

Checklist

App version

6.0.2

Where did you get the app from

Other

Android version

13

Device model

No response

First occurred

yesterday

Steps to reproduce

  1. Open Infinity and note the color of the FAB
  2. Go to the "Customize Theme" page for your current theme
  3. Observe that "Color Primary Light Theme" is no longer the color used for the FAB, but that it is instead the "Color Accent" that is used

Example post, link, markdown...

No response

Expected behaviour

Use the "Color Primary Light Theme" for the FAB, or have the subtitle match the current behavior

Current behaviour

Use the "Color Accent" for the FAB

Screenshots ![Screenshot_20230711_021333_Infinity](https://github.com/Docile-Alligator/Infinity-For-Reddit/assets/41648788/bf63449d-9e2b-43cd-bb15-cc2b467b7ee9) ![Screenshot_20230711_021324_Infinity](https://github.com/Docile-Alligator/Infinity-For-Reddit/assets/41648788/e626f8db-264b-4ce9-9509-ad127344a33d)

Logs

No response

SuperStormer commented 1 year ago

Related issue: https://github.com/Docile-Alligator/Infinity-For-Reddit/issues/1344 I believe the current issue is the cause of the linked issue, which still occured on the current release when I imported my settings from an old version (don't remember which) to 6.0.2. The theme in question.


It looks like the FAB's coloration was changed in https://github.com/Docile-Alligator/Infinity-For-Reddit/commit/8a6eb8e025f388c9bc23e5ef30a0991eb33b7365 and https://github.com/Docile-Alligator/Infinity-For-Reddit/commit/c5d4ac64a0c93bec80e45cdbbed15ee348283d9e


The two obvious ways to fix this (at least to me) would be to removing FAB from the "Color Primary Light Theme" subtitle, or to change the FAB color back to "Color Primary Light Theme".

Here is my kinda biased opinion in support of the latter option: For a dark theme, "Color Accent" has to be a light color—otherwise, the headings on the settings pages would be unreadable, and likely other text using the accent color as the text color. This means that dark themes like Dracula and White Amoled which formerly had a dark colored FAB have a light colored one now, which IMO seems counterproductive for a dark theme (granted, their old colors lacked contrast against the post backgrounds, but that's a different issue). Of course, this is a bit of a https://xkcd.com/1172/ moment, so I would understand if you don't change it back.

Docile-Alligator commented 5 months ago

Subtitle changed.