OK, back from a travel. I did some more work to this branch:
- Added quick buttons for switching sides and play/pause "on top of the clock display". I made the play pause icon "smart" so it only shows play or pause. I did not test all combinations but it works across 2 separate web clients.
- Fixed a bug when choosing a new game in the cogwheel menu using web client. The new game did not work as the engine did not reply to the first move. The first move was lost due to a state error.
- I also brought back the mute button on a remote web client. It was lost in this branch. The mute buttons is now in the same location as before, but the upload/download PGN buttons I decided to move and place them "inside the board" - just below it with the sync, flip etc.". As you mentioned DJ Dekker there was room for two more buttons within the chess board and I thought that uploading or downloading a PGN game buttons belong there. To make the board look-and-feel consistent those upload/download buttons are now also shown on local web client so that the board looks the same across clients.
- On desktop portrait mode I got double vertical scroll bars when going into portrait mode. Fixed that!
- latest commit was that if I upload a PGN game, autoplay starts, but the new "smart" play-pause icon was showing play. Fixed it to show pause. Now uploading and watching a PGN replay is more UX friendly.
Now as a result of this new design, we actually have a way to start the users clock before making the first move, just like in a "real face-2-face" game. I can start my user clock if I play white, and then make my first move. I have not utilised this for any logic yet, but in discussions with Dirk earlier I have been missing a way to say that the game has started - before making the first move. One case where I needed it was that I would not like to start any analyzers and consume CPU unless the game really has started.... I would not like to consume CPU if you just start the web client.
I just basic tested all these changes. I need to test more and it would be good if someone else also tests more. I am sure I missed something when doing the fixes listed above. They are now committed into this branch so if you do a git pull you get them.
One thing that I noticed was that the clock font is quite big on the phone remote web client... For instance the text "depth 17 | score -0.17" will not fit on one portrait mode row in the mobile phone (but it does fit on desktop portrait).
-- Johan