Skip to content

feat: support clockTenths pref from the website#2758

Merged
veloce merged 1 commit intolichess-org:mainfrom
tom-anders:clockTenths
Mar 18, 2026
Merged

feat: support clockTenths pref from the website#2758
veloce merged 1 commit intolichess-org:mainfrom
tom-anders:clockTenths

Conversation

@tom-anders
Copy link
Copy Markdown
Collaborator

The current behavior is to always show tenths below 10s, this PR uses the existing preference from account settings to also support "never" and "always".

Needs lichess-org/lila#19799

The current behavior is to always show tenths below 10s, this PR uses
the existing preference from account settings to also support "never" and "always".

Needs lichess-org/lila#19799
Copy link
Copy Markdown
Contributor

@veloce veloce left a comment

Choose a reason for hiding this comment

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

Nothing to say about the code.

However the new pref "always" may cause UI issues when the tenths are displayed while the clock still displays hours. We may have overflow in that case, so it should be tested.

@tom-anders
Copy link
Copy Markdown
Collaborator Author

Ah yes, I forgot about correspondence. Will test and post some Screenshots

@veloce
Copy link
Copy Markdown
Contributor

veloce commented Mar 17, 2026

Not only correspondence, but classical too.

@tom-anders
Copy link
Copy Markdown
Collaborator Author

Looks fine with hours (see screenshot below). And in correspondence, we're actually using a separate clock widget, so the preference does not actually have any effect there.

Screenshot_1773774802

@veloce veloce merged commit 9be9f08 into lichess-org:main Mar 18, 2026
1 check passed
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