Skip to content

fix: set tokens on :host level to allow customization #354#355

Merged
rmenner merged 1 commit into
mainfrom
sunMota/customizing-token-bug
Oct 1, 2025
Merged

fix: set tokens on :host level to allow customization #354#355
rmenner merged 1 commit into
mainfrom
sunMota/customizing-token-bug

Conversation

@sun-mota

@sun-mota sun-mota commented Sep 26, 2025

Copy link
Copy Markdown
Member

Alaska Airlines Pull Request

Issue

Customizing with tokens weren’t working because tokens were re-defined on auro-button under shadowDom, which overrode consumer values.

Solution

Define all token values on :host level and have internal rules only consume them. This way, consumers can override and the cascade works as expected.

Checklist:

  • My update follows the CONTRIBUTING guidelines of this project
  • I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.

Thank you for your submission!

-- Auro Design System Team

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @sun-mota, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@github-actions

github-actions Bot commented Sep 26, 2025

Copy link
Copy Markdown

Surge demo deployment failed! 😭

@sun-mota
sun-mota force-pushed the sunMota/customizing-token-bug branch from 5e2c8d5 to ba06bc2 Compare September 26, 2025 17:56
@sun-mota sun-mota linked an issue Sep 26, 2025 that may be closed by this pull request
7 tasks
@sun-mota sun-mota self-assigned this Sep 26, 2025
@chrisfalaska

chrisfalaska commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

I am noticing color differences against the live DocSite

Screenshot 2025-09-30 at 3 21 32 PM Screenshot 2025-09-30 at 3 21 25 PM

@sun-mota

sun-mota commented Oct 1, 2025

Copy link
Copy Markdown
Member Author

@sun-mota
sun-mota force-pushed the sunMota/customizing-token-bug branch from ba06bc2 to ff308de Compare October 1, 2025 17:16
@sun-mota
sun-mota force-pushed the sunMota/customizing-token-bug branch from ff308de to e0ac8a7 Compare October 1, 2025 17:19

@chrisfalaska chrisfalaska left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and built locally. Looks great! 🚀

@rmenner
rmenner merged commit c045f10 into main Oct 1, 2025
6 of 7 checks passed
@rmenner
rmenner deleted the sunMota/customizing-token-bug branch October 1, 2025 17:50
@rmenner

rmenner commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 11.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rmenner rmenner added the released Completed work has been released label Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Completed work has been released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--ds-auro-button-* tokens do not work to customize colors

3 participants