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

Puzzles for certain openings do not show side selection control #15592

Closed
C-Higgins opened this issue Jun 24, 2024 · 4 comments
Closed

Puzzles for certain openings do not show side selection control #15592

C-Higgins opened this issue Jun 24, 2024 · 4 comments

Comments

@C-Higgins
Copy link

Example URL: https://lichess.org/training/Englund_Gambit_Complex

: !data.replay && !ctrl.streak && angle.opening && renderColorForm(ctrl),

The side selection control is not rendered because angle.opening is false. This is apparently not an "opening", but a "theme". I'm not sure what the distinction is, but there's no reason these can't be filtered by color. In fact, changing the URL to include the color does work:

Since this feature is fully supported in the URL, the UI control should not be hidden based on angle.opening.

@ornicar
Copy link
Collaborator

ornicar commented Jul 2, 2024

It makes sense to select a color when, and only when, we're training a given opening.

@ornicar ornicar closed this as not planned Won't fix, can't repro, duplicate, stale Jul 2, 2024
@C-Higgins
Copy link
Author

You don't think Englund Gambit Complex is something people might want to train for their openings?

@ornicar ornicar reopened this Jul 4, 2024
@ornicar
Copy link
Collaborator

ornicar commented Jul 4, 2024

I see, that's just the opening naming being weird and inconsistent.

"Englund Gambit Complex" doesn't exist, it's not a chess position. There exist named openings that include it in their name, such as Englund Gambit Complex: Felbecker Gambit or even more confusingly Englund Gambit Complex: Englund Gambit.

But "Englund Gambit Complex" is not a thing and should therefore not have any puzzles associated to it.

fynsta added a commit to fynsta/eco that referenced this issue Jul 4, 2024
Everywhere else just calls it "Englund Gambit", and it only creates problems.
@ornicar ornicar closed this as completed in 91bb2b4 Jul 4, 2024
niklasf added a commit to lichess-org/chess-openings that referenced this issue Jul 4, 2024
@C-Higgins
Copy link
Author

C-Higgins commented Jul 4, 2024

Thanks for the swift turnaround! 🙏

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

No branches or pull requests

2 participants