Skip to content

Clock Tool - Low time warning settings#2771

Closed
r3econ wants to merge 2 commits intolichess-org:mainfrom
r3econ:clock-low-time-warning-settings
Closed

Clock Tool - Low time warning settings#2771
r3econ wants to merge 2 commits intolichess-org:mainfrom
r3econ:clock-low-time-warning-settings

Conversation

@r3econ
Copy link
Copy Markdown
Contributor

@r3econ r3econ commented Mar 15, 2026

Demo

SCR-20260315-tsgi-tile
test.mov

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 15, 2026

Hey @r3econ love your work you've been contributing but I feel for this one a setting isn't really needed. To be honest I'm not even that picky on when the warning kicks in, 20 or 30 seconds to match a regular timed game in the app works for me but I'm down for whatever, but I don't think it really needs a 7th button. The sound that you hear when time is running low in an in-app timed game would be nice. But I don't see a big deal where people
Will complain or care much when the warning kicks in

@r3econ
Copy link
Copy Markdown
Contributor Author

r3econ commented Mar 15, 2026

Yes, I've been thinking that a setting is an overkill.

We could simplify it by only having the animation (see the video) and use the already existing time interval of 10% of the time set for the threshold when the animation would start. And remove the setting whatsoever.

The sound you're talking about is being played at the emergency threshold. This is already there

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 16, 2026

Yes, I've been thinking that a setting is an overkill.

We could simplify it by only having the animation (see the video) and use the already existing time interval of 10% of the time set for the threshold when the animation would start. And remove the setting whatsoever.

The sound you're talking about is being played at the emergency threshold. This is already there

Sounds great to me!

@r3econ
Copy link
Copy Markdown
Contributor Author

r3econ commented Mar 16, 2026

Ok, I'll close this PR and implement the change in a new one 👍🏾

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.

Would it make sense for the chess clock to have a low time warning like real time games do

2 participants