ogarcia / opensudoku

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

WIP: Centre note support #127

Closed nikclayton closed 2 years ago

nikclayton commented 3 years ago

This is most of the way to supporting centre notes. "Most of the way" because only the single number input mode has been modified, to allow for feedback.

Key UI changes:

nikclayton commented 3 years ago

Here's a screenshot of the UI with this PR. The first three rows all contain cells with centre notes, the remaining notes are corner notes.

Screenshot_1630516758

ogarcia commented 3 years ago

Hi @nikclayton

The idea of rearranging the input buttons in a 3x3 grid is great. But that you have two methods to add notes, not so much. I have a lot of feedback of users and don't like these changes.

I think that is better have a grid of 4x3 buttons. 3x3 numeric pad and one column with delete, change entry mode and toggle notes/numbers. Something like the mockups of #115

But in your screenshot, I like the icons. They can be used in toggle button to switch between entry notes or numbers.

ogarcia commented 3 years ago

When I say "have two methods to add notes" I refer to keyboard, but it can be a config option. Use corner notes or center notes.

nikclayton commented 3 years ago

Thanks for the feedback.

I think we disagree about the "or" in:

Use corner notes or center notes.

I don't think that matches the feature request, or how this is intended to be used.

Corner ("edge" might be a better term) marks indicate "This number might be in this cell, but so might other numbers".

Centre marks indicate "This cell can only be one of these candidates".

And it's realistic/natural to use both while solving a puzzle.

For an example of this in action, see e.g. https://www.youtube.com/watch?v=9m9t8ie9-EE (1m50 to 4m40 in that video explains the technique, and when to use corner marks and when to use centre marks, when solving the puzzle both are used). https://cracking-the-cryptic.web.app/sudoku/PrfmJPgdft is the web app with the same functionality.

I agree with you that the UI changes in #115 also make sense, but if I tried to make those changes here too the PR would be even larger.

Speaking of UI changes, do you have a specific set of devices (emulators?) that you use when checking that new layouts work OK?

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 PR. If you are still interested in it, please open it again in GitLab following the contributing guide.

Thank you very much for your understanding.