ogarcia / opensudoku

Open Source Sudoku game for Android.
GNU General Public License v3.0
327 stars 144 forks source link

Selection / highlighting strangeness when bidirectionality is turned off #128

Closed nikclayton closed 2 years ago

nikclayton commented 3 years ago

The selection behaviour in single number input mode results in some strangeness in the UI. I'm not sure if this is deliberately intended or not, so flagging here.

To reproduce:

Screenshot_1630686788

Expectation: At the start of the game, the number in the selected cell should match the selected keyboard button.

Screenshot_1630686853

Expectation: Clicking the 8 should have highlighted all the 8s in the grid, and the "1" keyboard button should still be selected.

Screenshot_1630688889

Expectation: Pressing the 2 button should have selected all the 2s in the grid, and deselected the 8 and the 1s.

Screenshot_1630686916

Expectation: Clicking the 5 should have selected all the 5s, deselecting everything else.

Screenshot_1630687132

Expectation: Going to another activity and coming back should not reset the selected cell.

All of this works as expected if the bidirectional option is turned on -- but that's strange, because that is only supposed to synchronise the number in the selected cell with the keypad.

Other thoughts

If you can confirm the expected behaviour I can put together a PR to try and fix.

ogarcia commented 2 years ago

I'm migrating Open Sudoku development from GitHub to GitLab. To make the migration as clean as possible I'm going to close this issue. If you are still interested (or if it is still relevant), please open it again in GitLab from July 26.

Thank you very much for your understanding.