Skip to content

fix: respect 24h time display preference#19265

Merged
lukstbit merged 1 commit intoankidroid:mainfrom
ericli3690:ericli3690-review-reminders-24-hours
Sep 25, 2025
Merged

fix: respect 24h time display preference#19265
lukstbit merged 1 commit intoankidroid:mainfrom
ericli3690:ericli3690-review-reminders-24-hours

Conversation

@ericli3690
Copy link
Member

@ericli3690 ericli3690 commented Sep 23, 2025

Purpose / Description

Fixed an issue where the time displayed for ReviewReminders did not respect the OS setting for whether to use 12h or 24h time.

Fixes

GSoC 2025: Review Reminders

Approach

Accomplished by simply utilizing a Context.

How Has This Been Tested?

Builds and runs on a physical Samsung S23, API 34. Switching languages, changing the 12h/24h setting works, etc.

Checklist

  • 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

GSoC 2025: Review Reminder

Fixed an issue where the time displayed for ReviewReminders did not respect the OS setting for whether to use 12h or 24h time. Accomplished by simply utilizing a Context.
@ericli3690 ericli3690 added Needs Review GSoC Pull requests authored by a Google Summer of Code participant [Candidate/Selected], for GSoC mentors labels Sep 23, 2025
@david-allison david-allison added Needs Second Approval Has one approval, one more approval to merge and removed Needs Review labels Sep 25, 2025
@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 Sep 25, 2025
@lukstbit lukstbit added this pull request to the merge queue Sep 25, 2025
Merged via the queue into ankidroid:main with commit 00f902f Sep 25, 2025
13 checks passed
@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 Sep 25, 2025
@github-actions github-actions bot added this to the 2.23 release milestone Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Pull requests authored by a Google Summer of Code participant [Candidate/Selected], for GSoC mentors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments