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

Implemented PL #30

Closed
wants to merge 3 commits into from
Closed

Implemented PL #30

wants to merge 3 commits into from

Conversation

tkrajina
Copy link
Contributor

Implemented PL (next player color).

Fixes #29

@tkrajina
Copy link
Contributor Author

Sorry, now I realised I made a pull request with two additional files (which shouldn't be part of this PR), closing this now and opening a new PR later.

@tkrajina tkrajina closed this Nov 18, 2022
@yewang
Copy link
Owner

yewang commented Nov 18, 2022

Thank you for the submissions. Sorry, I had not had time to review the main parts, but I look forward to future contributions. Are you okay with releasing under MIT License?

Do your changes handle saving the PL label as well?

@tkrajina
Copy link
Contributor Author

Yes, MIT license is OK.

I didn't check saving (viewing was most important for me), but I'll take a look, thanks for the reminder!

And one more thing I'd probably like to add is a button to switch player when editing the SGF.

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.

Besogo ignores PL
2 participants