Conversation
|
Since this process seems entirely deterministic, could this be an actual program instead of a skill? The LLM might even be able to get a head start on generating such a script, either with regexes or even an HTML parser. |
- Close button: keep the empty .btn-close (CSS-mask icon over currentcolor); drop the wrong 'add a child SVG' instruction that double-rendered the icon, and note .btn-close-white is gone (set text color instead) - Border radius: remove the fabricated $border-radius-2xl rename; document the real $radius base + $radii map (0-9) + --radius-* tokens - Font sizes: note lg+ are fluid clamp() values (not exact rems) and list the full xs-6xl scale - Add a 'Rebuilt behavior & new components' section: carousel→scroll-snap with opt-in autoplay, scrollspy→IntersectionObserver, the new components (combobox, chips, datepicker, range, strength, otp-input, nav-overflow, toggler), util/backdrop.js removal, CSS @layer, and --bs-*-rgb removal - Extend the Phase 6 grep checklist for all of the above
- Remove the 'ScrollSpy → IntersectionObserver' note: v5 (5.2+) already used IntersectionObserver, so it's not a v5→v6 change (the docs changelog omits it) - Expand carousel coverage to match the changelog: ride→autoplay, wrap→ends, removed control/caption/dark/stacked classes, .carousel-overlay requirement, control-icon renames - Add the remaining new components (submenu, stepper, avatar, form-adorn, prose) - Note the .rounded-* remap + --radius-pill, and link-*→underline-* utilities - Extend the Phase 6 grep list accordingly
The Sass deprecation list said to use `$border-radius-2xl`, but the Utilities section (and the source) removed the whole `$border-radius-*` family in favor of the `$radii` map / `--radius-*` tokens. Point the note there instead.
#42595 removed util/focustrap.js and util/scrollbar.js (dead code after the native <dialog> Dialog/Drawer rewrite). Record them alongside the existing util/backdrop removal in both the migration skill and the migration guide changelog.
- Split the v5/v6 namespace-vs-import example into two code fences so the duplicate `const tooltip` no longer trips eslint's no-redeclare, and add the blank line after the import - Add currentcolor, Datepicker, focustrap, oklab, oklch to the cspell dictionary; reword 'inerting' and 'Unlayered' to avoid coined words
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
2 participants
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.
No description provided.