Skip to content

fix(move_list): apply piece notation font and padding scroll in stacked mode#3132

Merged
veloce merged 2 commits into
lichess-org:mainfrom
lsaudon:fix/stacked_move_piece_notation_font
May 8, 2026
Merged

fix(move_list): apply piece notation font and padding scroll in stacked mode#3132
veloce merged 2 commits into
lichess-org:mainfrom
lsaudon:fix/stacked_move_piece_notation_font

Conversation

@lsaudon

@lsaudon lsaudon commented May 7, 2026

Copy link
Copy Markdown
Contributor

I spent a bit of time trying to refactor move_list.
I noticed that in the stacked version, the Chess font wasn’t being used.
And same bug on padding and scroll view in stacked mode

Before After
Capture d'écran 2026-05-07 210849 Capture d'écran 2026-05-07 210600
Capture d'écran 2026-05-07 212427 image

I haven't included this in the PR, but inlineColor and inlineDecoration are never used by the calling widgets.

@veloce veloce left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good catch, thanks!

@veloce veloce merged commit b31344b into lichess-org:main May 8, 2026
1 check passed
@lsaudon lsaudon deleted the fix/stacked_move_piece_notation_font branch May 8, 2026 20:27
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