InfiniTimeOrg / InfiniTime

Firmware for Pinetime smartwatch written in C++ and based on FreeRTOS
GNU General Public License v3.0
2.63k stars 901 forks source link

Alarm: improve preview UX #2049

Open liamcharger opened 2 months ago

liamcharger commented 2 months ago

This PR improves the user experience when viewing the time until the alarm goes off.

Previously, it was a navy blue rectangle with no indication of how to dismiss the dialog. It’s not hard to figure out how to dismiss it, but as the InfiniTime vision states, it should be predictable and easy to use.

I think the updated UI can be improved, so any feedback is welcome.

Before: InfiniSim_2024-04-02_063810 After: InfiniSim_2024-04-02_063712

github-actions[bot] commented 2 months ago
Build size and comparison to main: Section Size Difference
text 377208B 192B
data 940B 0B
bss 63548B 0B
JF002 commented 2 months ago

Hi @liamcharger ! If it' s not too much to ask, could you provide a before/after picture (from a camera or using InfiniSim) of your changes?

liamcharger commented 2 months ago

Sure, I've just updated my original comment!

minacode commented 2 months ago

Nice idea to make it fit better to the rest :) I think it would look good if you align the hours, minutes, ... vertically like they are in the previous version. That makes it easier to understand.