Skip to content

chore: Introduce visual accent prop to box#4661

Closed
at-susie wants to merge 6 commits into
mainfrom
onetheme/explore-box-style
Closed

chore: Introduce visual accent prop to box#4661
at-susie wants to merge 6 commits into
mainfrom
onetheme/explore-box-style

Conversation

@at-susie

Copy link
Copy Markdown
Member

Description

Explore a new variant in box for the visual accent for one-theme

Related links, issue #, if available: n/a

How has this been tested?

Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

- Remove redundant `color: inherit` from `.awsui-accent-base` base class
- Increase specificity of accent color selectors from `.accent-*` to `.box.accent-*` (0,1,0 to 0,2,0) to ensure accent text colors override root default-text-style
- Add `color: inherit` to nested Box elements within accented containers to properly propagate parent accent color
- Remove explicit `variant="subtle"` from Icon in accent context to use inherited styling
- Remove unnecessary `color="inherit"` from Icon wrapper in table rows as specificity fix handles propagation
- Update SCSS documentation to clarify specificity strategy and color inheritance behavior
- Ensures accent content text colors take precedence and cascade correctly to descendants
@at-susie at-susie changed the title chore: Explore new box component variant for visual accent chore: Introduce visual accent prop to box Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant