fix: recognize CSS whitespace in named grid areas - #556
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe grid-area rule now treats tabs, newlines, spaces, and repeated whitespace as cell separators. Tests cover valid mixed whitespace and invalid uneven rows. ChangesGrid Area Whitespace Handling
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized change makes named grid areas recognize tabs, line feeds, and spaces as CSS whitespace without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2 files. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
| /** | ||
| * Regular expression to match sequences of CSS whitespace | ||
| */ | ||
| const cssWhitespace = /[\t\n ]+/u; |
There was a problem hiding this comment.
| const cssWhitespace = /[\t\n ]+/u; | |
| const cssWhitespace = /[\t ]+/u; |
CSS in general allows newlines as whitespace but not inside strings (Example)
Prerequisites checklist
AI acknowledgment
What is the purpose of this pull request?
grid-template-areas.unevenGridAreaerror.What changes did you make? (Give an overview)
/[\t\n ]+/uregular expression to match sequences of CSS whitespace.grid-template-areasrows to split on CSS whitespace instead of literal spaces.Related Issues
fixes #537
Is there anything you'd like reviewers to focus on?
Summary by CodeRabbit
Bug Fixes
Tests