The app already uses a wake lock. Look at the immersive mode file. You can use that.

The app already uses a wake lock. Look at the immersive mode file. You can use that.

mobile/lib/src/utils/immersive_mode.dart

Line 5 in 76420d6

import 'package:wakelock_plus/wakelock_plus.dart';

Thanks for the hint, I added immersive mode to the clock screen 👍🏼

Looks cool! I was considering implementing this as well, but you beat me to it :) I found a few minor bugs:

In reality, 0+1 is 3 seconds +1. Currently, white has to accept their fate and flag.
Currently, selecting a time control also changes the default time control on the home screen. Not sure if we want that, probably not.

Disabling the reset, settings, and home buttons when the clock is not paused, similar to Lichobile, to prevent accidental clicks seems like a good idea.

Looks nice indeed! I agree with all the above comments from @HaonRekcef

• selecting a time control shouldn't change the selected one on Home Screen
• disable buttons while clock is active

Also since it's a new app, we should change the clock color. We could use another lichess color (defined in LichessColor.dart).

I also think the move counter is weirdly placed here. Could we not just put it just below the clock instead?

We can even prepend the string: "Moves played:" before, which is translated.

Thanks for all the feedback 🙌🏼

• Special time increment case with 0+1 gets handled as 3s +1
• Converted time control component to a "dumb component" and handle selection in parent to not change setting in home screen
• Buttons are now disabled when game is running
• Changed the clock color to lichess green

@veloce I am afraid I do not get your point with the move counter - a "move" is completed once both White and Black have played one turn so the same value has to be visible to both players. What do you mean with "below the clock"? Should every player see the move count below his own clock? That would duplicate the information and take up screen space 🤔

I don't really like the move counter being placed here.

But why is that counter useful in the first place? I thought it was only for some special clock setup which is not supported at the moment, isn't it?

To be honest, I migrated the move counter feature and design 1:1 from the old app, because I thought it could be useful to keep track for the 50 move draw rule in longer games.

If that's not desired, we can simply remove the feature.

I think we can keep the counter but with a better design.

Let's add a move counter, but per player. Each time the clock is tapped the counter increments (independently for each player).

Display the move number with a small font at the bottom right corner of each clock button.

Comments are all resolved now

Looks very good! Thanks @KarnerTh , I will test it on iOS asap.

I think we're missing one last important thing though: the click sound on tap. Let's not bother with the sound themes and use the same sound for all themes. We can reuse the sound of lichobile.

Added the clock sound from lichobile on every tap.

Tested only on Android, because I am still not able to run it on iOS - so help is appreciated :)