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

Fixed timezone UI problems - Changed timezone set behavoir - changed dimming applying #92

Merged
merged 3 commits into from
Oct 31, 2024

Conversation

Martinius79
Copy link
Collaborator

Please do a squash merge!

Fixed timezone UI problems ->
Dimming was not applied correctly if timezone changed from the menu and reached night time/day time.
menu was not redrawn (so not visible anymore), if the tens hours digit changed while timezone set.

Changed timezone set behavoir ->
The timezone offset value is now "wrapped around" if -12 or +12 h is reached.

Changed timezone menu drawing ->
For negative values a minus (-) is drawn in front of the actual offset.
For positive values a plus (+) is drawn in front of the actual offset.
For 0:00 no sign at all.

minor changes in the serial/debug outputs and the code formatting

renamed and changed the variables and functions for dimming in main.cpp

added a public setDimming() method to Backlights class and put member variable "dimming" to private

placed all "auto night time dimming" stuff behind IFDEFs

@Martinius79 Martinius79 changed the title Fixed timezone UI problems - Changed timezone set behavoir - changed dimming appling Fixed timezone UI problems - Changed timezone set behavoir - changed dimming applying Oct 26, 2024
@Martinius79 Martinius79 marked this pull request as ready for review October 26, 2024 10:45
@aly-fly aly-fly merged commit 1487152 into aly-fly:main Oct 31, 2024
2 checks passed
@Martinius79 Martinius79 deleted the FixedTimezoneUIProblems branch November 3, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants