Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clock: 12 hour "midnight format" #341

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

956MB
Copy link
Member

@956MB 956MB commented Jan 1, 2025

What's new

I agree with @GonzalesGabe in #317 that this could probably be an option for users to select their preference on how the 12hr clock is displayed.

Adds the options 12:XX and 00:XX in Settings > System > Midnight Format to display the preferred format past midnight on all clocks (Desktop, Main Menu MNTM style, Lock screen and Nightstand clock app). "12:30 AM" -> "12:30" OR "00:30".

"Midnight Format" could also be changed to something like "12AM Format" to eliminate the scrolling.

12:XX

image (11)

00:XX

image (14)


For the reviewer

  • I've uploaded the firmware with this patch to a device and verified its functionality
  • I've confirmed the bug to be fixed / feature to be stable

- References Next-Flip#317 and adds the options `12:XX` and `00:XX` in `Settings > System > Midnight Format` to display the preferred clock format past midnight on all clocks (Desktop, Main Menu MNTM style, Lock screen and Nightstand clock app). "12:30 AM" -> "00:30" OR "12:30".
Copy link
Member

@Willy-JL Willy-JL left a comment

Choose a reason for hiding this comment

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

RTC is "sacred" xD

its limited and will persist between firmwares, bad idea to add things here if not also added to OFW

i'd say propose to OFW and see what they think, if they shut it down then we can just use momentum settings

@Willy-JL Willy-JL marked this pull request as draft January 21, 2025 03:22
956MB added 3 commits January 21, 2025 18:54
- Also felt like the midnight format setting was too out of place in `MNTM > Interface` with the 5 other submenu entries, so I put it in Misc for now.
@956MB 956MB marked this pull request as ready for review January 22, 2025 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

2 participants