mobile icon indicating copy to clipboard operation
mobile copied to clipboard

"View the solution" issue

Open wassupluke opened this issue 1 year ago • 5 comments

What is the problem?

Using the "View the solution" button shows the first step of the solution and then proceeds to visually spam the ">" button for a while without actually progressing through the subsequent moves.

Steps to reproduce the bug

  1. Open a chess tactics trainer session
  2. Click "View the solution"

App version

0.12.0

Device Info

Pixel 8 Pro / Android 15

Additional information

https://github.com/user-attachments/assets/9885b582-4718-49ac-8e1b-48ab4d28ca41

wassupluke avatar Oct 04 '24 22:10 wassupluke

This feature was introduced in pull request #774. I agree that at the moment it seems like the app is attempting to show the complete solution but gets stuck. However the blinking is meant to indicate to the user that they can manually go through the solution. I'm not sure what the optimal implementation would be. Here are some possible options:

  1. Keep the current behavior but also play through the complete solution automatically(this would differ from the website's behavior).
  2. Do not play through the moves automatically not even the first move, instead let the arrow blink to signal that the user can now manually play through the moves.
  3. Indicate in a different way than blinking that the moves can be manually played through, though how that would look.
  4. Leave it as it is.

HaonRekcef avatar Dec 03 '24 12:12 HaonRekcef

Maybe good first issue @tom-anders ?

Boubou78000 avatar Dec 13 '24 21:12 Boubou78000

Maybe good first issue @tom-anders ?

Hmm not sure, it's not obvious what the "correct" fix would be here.

To me it looks like the blinking of the button should simply happen at a slower rate, but there are also other solutions, as @HaonRekcef stated above

tom-anders avatar Dec 14 '24 08:12 tom-anders

Is this still an issue? I'm just seeing two slow blinks now on the next move button. The menu blinking seems unnecessary though.

v0.14.1 Pixel 6 Pro / Android 15

https://github.com/user-attachments/assets/9990efce-74c7-4d0d-8b43-6d2829137b8f

CloudyDino avatar Jan 30 '25 16:01 CloudyDino

Still the same bug with app v0.13.12 on Pixel 8 Pro on Android Baklava. Issue is identical to my initial screen recording.

wassupluke avatar Jan 31 '25 03:01 wassupluke