Skip to content

Conversation

@tors42
Copy link
Contributor

@tors42 tors42 commented Aug 4, 2023

In case the orientation is set to Automatic,
and if the value of the PGN tags [White "player1"] or [Black "player2"] matches current user id,
set the orientation to the matching color.

The specific use case which got me this idea,
was after I played a game with black pieces on another site which I happened to have the same username on,
and then imported the PGN in a study.

But maybe it is such a rare occurrence that a pasted PGN contains player tags which matches the Lichess user id,
that this change does not carry its weight - and that the user can instead manually choose the correct orientation in the Choice-box.

The dialog (no changes in dialog, just including screenshot for reference)
create-chapter-pgn

tors42 and others added 4 commits August 4, 2023 14:01
In case the orientation is set to Automatic,
and if the value of the PGN tags [White "player1"] or [Black "player2"]
matches the user id of the user creating the chapter,
set the orientation to the matching color.
* master: (35 commits)
  hide ublog links in kid mode - closes lichess-org#13391
  scala tweaks
  fix untranslated title
  fix css code duplication
  use infinite glyph
  New Crowdin updates (lichess-org#13302)
  add color field to /api/tv/channels endpoint
  fix: no indication that infinite analysis is on
  improve sound on puzzles
  prettier on lexicon sources
  move phrase edits from generated artifact to upstream source
  remove leftover console.log
  allow optional config values
  Fix build
  Domains in Vietnam
  Fix move-vi.json
  refactor: annotations on board
  Scalachess 15.6.5
  refresh user puzzle perf after batch solve - closes lichess-org#13371
  remove unused code
  ...
@ornicar ornicar merged commit bf2c28f into lichess-org:master Aug 11, 2023
@tors42 tors42 deleted the orientation-by-player-tag branch August 11, 2023 16:16
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