Skip to content

Conversation

timmo001
Copy link
Member

@timmo001 timmo001 commented Aug 26, 2025

Breaking change

Proposed change

Builds on #26557

Adds a few face style options

  • Marker (default)
  • Numbers
  • Numbers (upright)
  • Roman numerals
  • Roman numerals (upright)

New options side by side (Medium size, hour ticks):
image

Mixed with other options:
image

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

  - clock_style: analog
    clock_size: large
    show_seconds: true
    no_background: false
    border: false
    ticks: minute
    face_style: roman
    type: clock

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@timmo001 timmo001 changed the title Add analog card clock styles Add analog card clock style options Aug 27, 2025
@timmo001 timmo001 force-pushed the feature/card-clock-analog-style branch from 6da771d to 14c2078 Compare August 27, 2025 09:29
defaulted(
union([
literal("markers"),
literal("numbers"),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm 50/50 on having non-upright numbers here. Looks like most analog clocks with numbers keep upright, while roman numerals tend to rotate with the clock.

The upright and standard rotated option visually looks ok for the roman numeral case though

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And 6 looks like 9 when it's non-upright ...

Copy link
Member Author

@timmo001 timmo001 Aug 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah that is the main one, It doesn't look too bad with hours/minutes ticks side by side with roman numeral cards, but as quarters looks off

@bramkragten
Copy link
Member

We have to watch out to not introduce too many options...

@bramkragten bramkragten added the Needs UX Pull requests requiring a review from the Home Assistant design team label Aug 27, 2025
timmo001 and others added 9 commits August 28, 2025 15:16
Co-authored-by: Norbert Rittel <norbert@rittel.de>
Show lines

Adjust

Adjust

Replace 'ticks_style' with 'face_style' and use numbers over numeric

Use numbers over numeric
@timmo001 timmo001 force-pushed the feature/card-clock-analog-style branch from 1826329 to fe229b8 Compare August 28, 2025 14:16
@timmo001 timmo001 marked this pull request as ready for review August 28, 2025 15:05
@timmo001
Copy link
Member Author

timmo001 commented Aug 28, 2025

Looking for feedback on this one. Especially the options, whether numbers should be always upright or allow for both. Based on my deep dive into clocks, most roman numeral clocks rotate with the face, but base 10 numbers are generally always upright

I think this will probably be the last option for analog clock, I think everything else could be done via theming potentially

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Needs UX Pull requests requiring a review from the Home Assistant design team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants