Skip to content

[By coding agent] Allow pasting board position from string#2603

Closed
sensuikan1973 wants to merge 1 commit into
mainfrom
feature/paste-board-position
Closed

[By coding agent] Allow pasting board position from string#2603
sensuikan1973 wants to merge 1 commit into
mainfrom
feature/paste-board-position

Conversation

@sensuikan1973
Copy link
Copy Markdown
Owner

@sensuikan1973 sensuikan1973 commented May 21, 2025

This PR is trial to resolve #2338 .
It's because this change are created by google jules.

Implements the ability to paste a full board position directly into the application using a 65-character string format.
The string consists of 64 characters representing the board ('X' for black, 'O' for white, '-' for empty) followed by one character indicating the current player ('X' or 'O').

Changes include:
- Added `PastePositionShortcut` (Alt+V) to handle clipboard input.
- Modified `SetboardRequest` to accept a 64-char board string and current player color.
- Updated `BoardNotifier` to use the new `SetboardRequest` for pasting and refactored the "arrange discs" mode to be compatible.
- Added localization for the "Paste Position" shortcut label.
- Added new unit tests for the shortcut and notifier logic.

Fixes #2338
@sensuikan1973
Copy link
Copy Markdown
Owner Author

low quolity....

@sensuikan1973 sensuikan1973 deleted the feature/paste-board-position branch May 21, 2025 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow pasting positions

1 participant