Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show "pass" mark on the board even when show-move-number is true #873

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kaorahi
Copy link
Contributor

@kaorahi kaorahi commented Feb 12, 2021

I expect the large "pass" mark on the board even when I select "All" or "Last" for "Show Move Number" in Settings. How do you think?

@hope366
Copy link

hope366 commented Feb 12, 2021

I think this is a good idea, but by applying this PR, the pass mark is no longer displayed when passing with the move number set to hidden.
I don't know if it was a deliberate change, but I think it's better to show the pass mark even when the move number is hidden.

@hope366
Copy link

hope366 commented Feb 13, 2021

Thank you for your prompt response.
I have confirmed that the 92e298d is working fine.

kaorahi pushed a commit to kaorahi/lizzie that referenced this pull request Feb 17, 2021
@kaorahi kaorahi mentioned this pull request Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants