Skip to content

Fix bottom border radius corner order - #438

Open
aschojz wants to merge 1 commit into
GrapesJS:masterfrom
aschojz:fix/border-radius-corner-order
Open

aschojz wants to merge 1 commit into
GrapesJS:masterfrom
aschojz:fix/border-radius-corner-order

Conversation

@aschojz

@aschojz aschojz commented Sep 22, 2026

Copy link
Copy Markdown

Summary

  • order the bottom border-radius properties according to the CSS shorthand
  • add a regression test for the Style Manager configuration

Problem

The non-detached border-radius composite serializes its values in CSS shorthand order:

  1. top-left
  2. top-right
  3. bottom-right
  4. bottom-left

The last two Style Manager properties were configured as bottom-left and bottom-right. As a result, changing the bottom-left field affected the bottom-right corner and vice versa.

Verification

  • npm run lint
  • npm test -- --runInBand (20 tests passed)
  • npm run build

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.

1 participant