Skip to content

Comments

feat: change themes configuration#19700

Merged
BrayanDSO merged 3 commits intoankidroid:mainfrom
BrayanDSO:ref/theme
Jan 3, 2026
Merged

feat: change themes configuration#19700
BrayanDSO merged 3 commits intoankidroid:mainfrom
BrayanDSO:ref/theme

Conversation

@BrayanDSO
Copy link
Member

Purpose / Description

now you select the color scheme (follow system, day, night) under the Theme setting, and the themes themselves are selected under Day theme and Night theme

this makes things slightly more simple/predictable and allow the code to be simpler

also makes possible to have different and custom color palettes, while specifying if you want a light or dark theme.

Approach

  1. Create some PrefEnums to represent the app, day, and night themes
  2. Adapt the appearance preferences to the new setup
  3. Upgrade the preference values

How Has This Been Tested?

Emulator 31

Before upgrading the preferences

Screenshot_20251206_124131

After ugprading the preferences

Screenshot_20251206_124309
Emulator 25 Screenshot_20251206_130724 Screenshot_20251206_131032

Checklist

Please, go through these checks before submitting the PR.

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2025

Important

Maintainers: This PR contains Strings changes

  1. Sync Translations before merging this PR and wait for the action to complete
  2. Review and merge the auto-generated PR in order to sync all user-submitted translations
  3. Sync Translations again and merge the PR so the huge automated string changes caused by merging this PR are by themselves and easy to review

@BrayanDSO BrayanDSO added the squash-merge A squash & force push is required. The PR author may do this to speed up the merge process. label Dec 6, 2025
@BrayanDSO BrayanDSO marked this pull request as draft December 6, 2025 16:53
@BrayanDSO BrayanDSO marked this pull request as ready for review December 6, 2025 17:42
Copy link
Contributor

@criticalAY criticalAY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

<item>@string/full_screen_system</item>
<item>@string/full_screen_complete</item>
</string-array>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be reverted

Copy link
Member

@david-allison david-allison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks!

Note to Maintainer: One commit can be squashed, doesn't need to be the whole PR

@david-allison david-allison added Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels Dec 10, 2025
@BrayanDSO BrayanDSO force-pushed the ref/theme branch 2 times, most recently from 22d3a52 to 46059e8 Compare December 20, 2025 11:28
@lukstbit lukstbit added Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) and removed Needs Second Approval Has one approval, one more approval to merge labels Jan 3, 2026
@lukstbit
Copy link
Member

lukstbit commented Jan 3, 2026

Pending on me squashing and

now you select the color scheme (follow system, day, night) under the `Theme` setting, and the themes themselves are selected under `Day theme` and `Night theme`

this makes things slightly more simple/predictable and allow the code to be simpler
@BrayanDSO BrayanDSO removed the squash-merge A squash & force push is required. The PR author may do this to speed up the merge process. label Jan 3, 2026
@BrayanDSO
Copy link
Member Author

Squashed the necessary commits

@BrayanDSO BrayanDSO added this pull request to the merge queue Jan 3, 2026
@BrayanDSO BrayanDSO removed this pull request from the merge queue due to a manual request Jan 3, 2026
@BrayanDSO BrayanDSO added this pull request to the merge queue Jan 3, 2026
Merged via the queue into ankidroid:main with commit ed49039 Jan 3, 2026
15 checks passed
@github-actions github-actions bot added this to the 2.24 release milestone Jan 3, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2026

Maintainers: Please Sync Translations to produce a commit with only the automated changes from this PR.

Read more about updating strings on the wiki,

@github-actions github-actions bot removed the Pending Merge Things with approval that are waiting future merge (e.g. targets a future release, CI wait, etc) label Jan 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants