-
Notifications
You must be signed in to change notification settings - Fork 888
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
add support for annotations #235
Conversation
jtwires
commented
May 3, 2020
- add methods for creating/retrieving/removing annotations
- parse bracket- and semicolon-style annotations when loading PGNs
- output bracket-style annotations when producing PGNs
Thanks Jake. I'll review this over the next few days. |
Thanks @jtwires. It's clear you put a lot of work into this PR. I've got a few questions and suggestions before I merge.
Many people have been requesting this capability. Thanks again for your contribution. |
README.md
Outdated
Retrieve the annotation for the current position, if it exists. | ||
|
||
```js | ||
const chess new Chess() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo const chess = new Chess()
@@ -1541,14 +1615,46 @@ var Chess = function(fen) { | |||
} | |||
} | |||
|
|||
var to_hex = function(string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment here to explain what you're doing with hex and URI encoding? It's not immediately clear to me. Thanks
cool, |
* add methods for creating/retrieving/removing comments * parse bracket- and semicolon-style comments when loading PGNs * output bracket-style comments when producing PGNs
0.11.0 has been pushed to npm. I'll gen up a changelog sometime over the weekend. |