Skip to content

PGN Update#307

Closed
TimLbhnschl wants to merge 20 commits intomasterfrom
SaveGame
Closed

PGN Update#307
TimLbhnschl wants to merge 20 commits intomasterfrom
SaveGame

Conversation

@TimLbhnschl
Copy link
Collaborator

@TimLbhnschl TimLbhnschl commented Feb 16, 2026 •
edited
Loading

Purpose

PGN Browser Update and added comments are shown

Pre-merge TODOs and Checks

  • (exclude checks that are not relevant for your feature)
  • PGN file loads correctly
  • Navigating through the move history works
  • Starting the engine shows moves (test for both sides)
  • Selecting a different variant loads a new board with the variant and you can make moves on the board
  • In Crazyhouse the pocket pieces are shown and you can drop pieces

LuisW04
LuisW04 approved these changes Feb 18, 2026
View reviewed changes
Copy link
Collaborator

@LuisW04 LuisW04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested all implemented features and I found no flaws. The code looks clean as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LuisW04 LuisW04 LuisW04 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TimLbhnschl @LuisW04