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

The Time Shifts Around as the Digits Change #286

Open
jmain33 opened this issue Jan 30, 2024 · 3 comments
Open

The Time Shifts Around as the Digits Change #286

jmain33 opened this issue Jan 30, 2024 · 3 comments
Labels
enhancement New feature or request Planned This will be done in a future update

Comments

@jmain33
Copy link

jmain33 commented Jan 30, 2024

It is not a major issue since the many fonts options can be used to avoid it; but, the time display bounces around left-to-right as the digits change (most obviously when seconds are displayed). There are several fonts that avoid this intrinsically, but it would be nicer if the horizontal shifting were avoided regardless of what font is used. [Different placements and alignments don't affect the fact that the time will shift a bit.]

Bonjourr - version: 1.18.4.1
Chrome - version: 121.0.6167.86

@victrme
Copy link
Owner

victrme commented Jan 31, 2024

Hi @jmain33 !

It has been bothering me for some time as well. The issue is quite tricky to fix though.

Since the clock is centered and each character have a different width, when a bigger character appears, it pushes the clock to the left. The challenge would be to left-align the clock and determine the perfect width for it to appear centered. It also need to work for every font size and family.

It is definitely doable, and might even be fixed for the next update !

@jmain33
Copy link
Author

jmain33 commented Jan 31, 2024

Thanks for considering this refinement, Victrme. I don't know how much of a headache it would be to improve the coding. I enjoy the Bonjourr extension and find it to be reliable; so don't rack your brain over this, it isn't a big problem.

@victrme
Copy link
Owner

victrme commented Feb 5, 2024

It won't be in this update because it might introduce some breaking changes and I want it to be as smooth as possible. I think it is especially important because it's a small annoyance. This is what separate the good from the best 🤠🤠🤠

@victrme victrme added enhancement New feature or request Planned This will be done in a future update labels Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Planned This will be done in a future update
Projects
None yet
Development

No branches or pull requests

2 participants