This repository has been archived by the owner on Apr 13, 2022. It is now read-only.
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.
Updates to color names and descriptions
Changes
What: Drop action from the icon interaction states
Why: Action is implicit in this case and not needed in the name
•
iconActionHovered->iconHovered
•
iconActionPressed->iconPressed
What: Drop action from the interaction states
Why: Action is repetitive in the naming. Designers found this naming convention confusing and made them wonder if there were actions that aren't interactive 🤔
•
actionInteractive->interactive
•
actionInteractiveDisabled->interactiveDisabled
•
actionInteractiveHovered->interactionHovered
•
actionInteractivePressed->interactionPressed
What: Add surface to the naming
Why: Follows the naming structure of leading with the element
•
primarySelected->surfacePrimarySelected
•
primarySelected->surfacePrimarySelectedHovered
•
primarySelected->surfacePrimarySelectedPressed
What: Change link to interactive
Why: Follows the naming structure of interactive and allows us to map to the same components with an error state (critical links, selected checkboxes in an error state)
•
linkCritical->interactiveCritical
•
linkCriticalDisabled->interactiveCriticalDisabled
•
linkCriticalHovered->interactiveCriticalHovered
•
linkCriticalPressed->interactiveCriticalPressed
What: remove
secondaryfromicon
,iconHovered
, andiconPressed
,text
,textSubdued
,textDisabled
descriptionsWhy: Secondary is describing the color and not the purpose here. These are the default or primary colors for text and icons.
• updated description of
interactive
• remove
actionInteractiveSubdued
-> not being used in Polaris-react or Figma• updated description of
surfaceSelected
examples to referenceactionList
instead ofoptionList
• Updated the description for
interactiveCritical