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

Customize 24h vs 12h time formatting globally #29

Open
MikeShi42 opened this issue Sep 22, 2023 · 4 comments
Open

Customize 24h vs 12h time formatting globally #29

MikeShi42 opened this issue Sep 22, 2023 · 4 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@MikeShi42
Copy link
Contributor

Some users have a preference for 12h time instead of 24h time, today we do a mix of both depending on the UI they're looking at (sessions is primarily 12h, logs are primarily 24h) - we should allow users to globally toggle 12/24h time and update the time format across the app accordingly.

I've already started a bit of the implementation, but there is still a lot of way to go:

Todo:

  • Update all time formats around the app in UI components to accept timeFormat as a property
  • Update page components to inject timeFormat as a property to UI components (or shortcut this and just read from the provider directly)
  • Add UI for user preferences in the settings page somewhere.
  • Persist user preferences in localstorage
@MikeShi42 MikeShi42 added the good first issue Good for newcomers label Sep 22, 2023
@treypisano
Copy link
Contributor

Hi @MikeShi42 , can give resolving this issue a try?

@MikeShi42
Copy link
Contributor Author

Sounds good! Let me know if you need any help or pointers :) Feel free to hop in on our Discord if there's some implementations ideas you want to chat about.

I'd also mention we have a hook called useLocalStorage that can help persisting the settings.

@treypisano
Copy link
Contributor

Hi @MikeShi42 , just want to let you know I'm still working on this, just needed some time to get familiar with the codebase.

@ptdatta
Copy link

ptdatta commented Feb 3, 2024

@MikeShi42 Is it completed? I would like to give it a try?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants