Skip to content

Chinesecheckers#87

Merged
AEstrellaS merged 3 commits intomasterfrom
chinesecheckers
Mar 26, 2026
Merged

Chinesecheckers#87
AEstrellaS merged 3 commits intomasterfrom
chinesecheckers

Conversation

@aentum
Copy link
Copy Markdown
Member

@aentum aentum commented Feb 21, 2026

Current issues.

  1. It would be cool visually if the pieces can hop around in the path it moves, but to do that, ui will need path data besides source and destination. So it may get slightly complicated. I am unsure if there's anything in UI which supports that.
gameplay
  1. There is a collision problem for move arrows on some positions that makes it impossible to choose the desired move. Here is one game position that this happens. Even though red has a winning move at this position, it is overlapping with a red arrow.
Screenshot 2026-02-18 at 3 18 28 PM

Others:
The gamesman classic end of the game is named 'ccheckers.'
LMK if you prefer to rename it to 'chinesecheckers'

@aentum aentum requested a review from AEstrellaS February 21, 2026 23:49
Copy link
Copy Markdown
Member

@AEstrellaS AEstrellaS left a comment

Choose a reason for hiding this comment

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

Looking great! Just a small UX change

Comment thread games/image_autogui_data.py Outdated
"regular": {
'space': [272, 165],
'background': "chinesecheckers/board.svg",
'arrowWidth': 1,
Copy link
Copy Markdown
Member

@AEstrellaS AEstrellaS Feb 24, 2026

Choose a reason for hiding this comment

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

Arrows are a bit too thin, try 1.5 or 2

Copy link
Copy Markdown
Member

@AEstrellaS AEstrellaS left a comment

Choose a reason for hiding this comment

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

LGTM!

@AEstrellaS AEstrellaS merged commit 064a062 into master Mar 26, 2026
@AEstrellaS AEstrellaS deleted the chinesecheckers branch March 26, 2026 21:19
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