You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Remove the green --brand-accent focus ring from TagInput's default variant so it matches the canonical chip chrome — ChipInput shows no focus ring, and TagInput's block variant already omits it
Fixes the green outline on the "Allowed emails" field in the Share file modal (and any other default-variant TagInput)
Update the variant TSDoc since "no focus ring" is no longer a block-only differentiator
Low Risk
Pure focus styling and comment changes in one component; no logic, API, or accessibility behavior changes beyond removing a visual ring.
Overview
Removes the green --brand-accentfocus ring from TagInput's default variant so focused fields (e.g. Share file modal “Allowed emails”) match ChipInput and the existing block variant, which already omit that ring.
The default variant still uses focus-within:outline-none; only the ring utilities are dropped. TSDoc for tagInputVariants is updated so “no focus ring” is documented for default as well, not only block.
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
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.
Summary
--brand-accentfocus ring fromTagInput'sdefaultvariant so it matches the canonical chip chrome —ChipInputshows no focus ring, andTagInput'sblockvariant already omits itTagInput)Type of Change
Testing
Tested manually
Checklist