Blueforcer / awtrix3

Custom firmware for the Ulanzi Smart Pixel clock or self made awtrix. Getting started is easy as 1-2-3
https://blueforcer.github.io/awtrix3/
Other
1.26k stars 109 forks source link

[BUG] Issue with default Time app and low brightness level #475

Closed leinardi closed 8 months ago

leinardi commented 8 months ago

Bug report

Describe the bug

I just noticed an issue on the default clock when the brightness of the device is set to a very low value (e.g. on a dark room): the selected day of the week is indistinguishable from the others (see the picture below). The issue happens only with very low levels of brightness (e.g. 1).

Would it be possible to use different colors for the days of the week and the current day of the week? this should solve the problem.

Additional information

To Reproduce

Steps to reproduce the behavior:

Expected behavior

No matter what is the brightness level used, the current day of the week should be clearly distinct from the others, also for accessibility reasons. Maybe the current day could be the same red color as the top of the calendar?

Screenshots

image

This picture was taken with the brightness to the minimum. With an higher value of brightness the issue doesn't happen.

For accessibility reasons would still be good to use high contrast colors between the current day of the week and the others.

Blueforcer commented 8 months ago

That's normal because the LEDs doesn't get enough power at very low brightness. So blue for example disappearing first because it needs more power than the other two. Also the cheap LEDs are not calibrated and vary from batch to batch

leinardi commented 8 months ago

Yeah but the issue could still be solved using different colors for the current day of the week and the other days, instead of different brightness. If you really don't like this solution, could this be at least one option that can be set via the dev.json file?

Blueforcer commented 8 months ago

Its already possible to change the colors... https://blueforcer.github.io/awtrix-light/#/api?id=json-properties-1

leinardi commented 8 months ago

Oh neat, I didn't realize, tomorrow I'll try to change it and report back 👍

leinardi commented 8 months ago

It works like a charm: I used the WDCA property to set a different color!