Skip to content

chore: Introduce accentColor prop#4679

Draft
at-susie wants to merge 9 commits into
mainfrom
onetheme/box-accent
Draft

chore: Introduce accentColor prop#4679
at-susie wants to merge 9 commits into
mainfrom
onetheme/box-accent

Conversation

@at-susie

@at-susie at-susie commented Jul 1, 2026

Copy link
Copy Markdown
Member

Description

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.

at-susie added 6 commits June 23, 2026 16:19
- 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
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