-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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 3-5 fold positions in analysis #15999
Conversation
If you're wanting the indicator to be a glyph on the moved piece, custom svgs can provide uniformity (like blunder/mistake/inaccuracies). But as a user, I probably prefer the move list annotations alone. "Why is there is a 3 on my queen, stop ruining lichess!" |
Only checking for the last epd, which is the only one that could lead to three/five-fold repetition. Previously code could be triggered on EPDs that were repeated 3 times, but where neither players claimed three-fold.
The function was unused until now
Only seems to be used to CSS, values found in `nodeClasses`, `lila/ui/analyse/src/treeView/common.ts` ``` inaccuracy: glyphIds.includes(6), mistake: glyphIds.includes(2), blunder: glyphIds.includes(4), good: glyphIds.includes(1), brilliant: glyphIds.includes(3), interesting: glyphIds.includes(5), ```
Following feedback here: https://hq.lichess.ovh/#narrow/stream/1-general/topic/positions.20of.203-5.20fold.20repetitions/near/3445933 went for unicode number for now. The styling can always be polished later like for puzzle feedback, for the SVG icon I was thinking about the number inside ♻️ or |
ui/analyse/src/ctrl.ts
Outdated
const mainline = treeOps.mainlineNodeList(this.tree.root); | ||
if (this.data.game.status.name == 'draw') { | ||
add3or5FoldGlyphs(mainline); | ||
} |
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.
Not sure if it's the correct place to put
A way for people to quickly see which positions implied the 3-5 fold repetition (for #13459 and the many forum posts about it)
This is a WIP for now, looking for feedback on two things:
First attempt using unicode fractions, unfortunately
³⁄₃
does not render well on Safari/FirefoxAnother attempt
TODO: should not be shown in studies, only in games list
Another idea, list the positions at the end of the movelist (mockup, would be clickable)