Skip to content

Commit

Permalink
Remove showing 'X' as warnings
Browse files Browse the repository at this point in the history
Hasn't worked for a while and the feature is too burdensome for its
narrow use case, it's also confusing and buggy creating non-intuive
interactions with cells accepting characters instead of only numbers
Closes #8
  • Loading branch information
JohanG-75 committed Jun 15, 2024
1 parent a28af01 commit 748ff96
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/sudoku-cell.vala
Original file line number Diff line number Diff line change
Expand Up @@ -505,13 +505,8 @@ private class SudokuCell : Widget
if (solution != 0)
error = this.value != solution;
}

value_label.set_label (this.value.to_string ());
}

else if (game.board.count_possibilities (row, col) == 0)
value_label.set_label ("X");

if (error)
add_css_class ("error");
else
Expand Down Expand Up @@ -542,7 +537,6 @@ private class SudokuCell : Widget
public void clear_warnings ()
{
var marks = game.board.get_earmarks (row, col);
value_label.set_label (this.value.to_string ());
remove_css_class ("error");
for (int num = 1; num <= marks.length; num++)
earmark_labels[num-1].remove_css_class ("error");
Expand Down

0 comments on commit 748ff96

Please sign in to comment.