Repository navigation
fix(#110): MEDIUM: the stylesheet and the markup have drifted — seven undefined custom prop - #427
Merged
Merged
Conversation
Added the toast warning variant (CSS + icon), replaced undefined var(--primary) with --line, gave the unstyled classes (attendee-status, empty-state, kanban-card-body, sidebar-task-*, recur-icon, google-badge-icon, btn-msgraph) real rules while moving their inline styles into CSS and dropping no-op flex-1/all-day, and deleted the dead compatibility block, dead print/utility selectors and the duplicate .sidebar rule (print rules now target the real grid/agenda elements; print_stylesheet.test.js updated to match). Covered by four Playwright tests reading computed styles and a node test checking, across all of public/, that every custom property is defined, every class selector is used, and no top-level rule appears twice. Refs #110 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #110
Added the toast warning variant (CSS + icon), replaced undefined var(--primary) with --line, gave the unstyled classes (attendee-status, empty-state, kanban-card-body, sidebar-task-*, recur-icon, google-badge-icon, btn-msgraph) real rules while moving their inline styles into CSS and dropping no-op flex-1/all-day, and deleted the dead compatibility block, dead print/utility selectors and the duplicate .sidebar rule (print rules now target the real grid/agenda elements; print_stylesheet.test.js updated to match). Covered by four Playwright tests reading computed styles and a node test checking, across all of public/, that every custom property is defined, every class selector is used, and no top-level rule appears twice.
Driver gates (measured, not claimed)
red/green output
🤖 Generated with Claude Code