Support shorthand notation and area names in grid cell positioning #6084
Assignees
Comments
liady
pushed a commit
that referenced
this issue
Dec 13, 2024
**Problem:** Shorthand notation for grid cells positioning should be supported across the board (moving / resizing, not just parsing) and likewise area names from the template should be reflected in the positioning props. **Fix:** 1. Print the area names instead of the number values for grid positioning (`gridRowStart`, `gridRowEnd`, ...) if there's a match 2. Use (and understand) the shorthand notation (`gridRow`, `gridColumn`) when the underlying values are either the same or skewed by 1 (for start → end) 3. Unify the CSS property set logic so it can be reused among strategies, making it behave according to the above ^ **Notes** - We do support parsing shorthand notations with `/`-separated values, but they are never printed for now. If there are multiple values, it will currently use the longhand notation. **Demo:** https://github.com/user-attachments/assets/c70ee34b-c921-44c0-a269-2d13af70da2d Fixes #6084
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: