Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to supoprt new Auro design tokens #94 #95

Merged
merged 7 commits into from
Oct 30, 2023

Conversation

blackfalcon
Copy link
Member

Alaska Airlines Pull Request

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes: #94

Summary:

This commit updates all the token references to support the new design token naming spec. All CSS also supports fallback values to ensure a smooth transition from legacy to new.

TODO: There are 2 to-dos in this commit. Once there is an assumption that most teams have transitioned to the new token spec, the code that these to-dos are referencing can be removed as well as any npm package or reference to the legacy tokens.

Changes to be committed:
modified: package-lock.json
modified: package.json
modified: src/alaskaStyle.scss
modified: src/iconStyle.scss
modified: src/style.scss

Type of change:

Please delete options that are not relevant.

  • New capability
  • Revision of an existing capability
  • Infrastructure change (automation, etc.)
  • Other (please elaborate)

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

@blackfalcon blackfalcon self-assigned this Oct 11, 2023
@blackfalcon blackfalcon linked an issue Oct 11, 2023 that may be closed by this pull request
@blackfalcon blackfalcon marked this pull request as ready for review October 11, 2023 04:26
@blackfalcon blackfalcon requested a review from a team as a code owner October 11, 2023 04:26
blackfalcon added a commit to AlaskaAirlines/Icons that referenced this pull request Oct 11, 2023
This commit reverts the revert for using the new Auro design token name
spec. The revert happened due to a side-effect with auro-icon and the
customSize feature. The auro-icon element was updated to allow for more
flexibility in token use.

See AlaskaAirlines/auro-icon#95

This update has been verified that regardless of version of tokens a
consumer has installed, or none at all, the customSize feature will work
as expected.

Changes to be committed:
modified:   src/data/icons.json
package.json Outdated Show resolved Hide resolved
src/iconStyle.scss Outdated Show resolved Hide resolved
blackfalcon added a commit to AlaskaAirlines/Icons that referenced this pull request Oct 12, 2023
This commit reverts the revert for using the new Auro design token name
spec. The revert happened due to a side-effect with auro-icon and the
customSize feature. The auro-icon element was updated to allow for more
flexibility in token use.

See AlaskaAirlines/auro-icon#95

This update has been verified that regardless of version of tokens a
consumer has installed, or none at all, the customSize feature will work
as expected.

Changes to be committed:
modified:   src/data/icons.json
blackfalcon added a commit to AlaskaAirlines/Icons that referenced this pull request Oct 12, 2023
This commit reverts the revert for using the new Auro design token name
spec. The revert happened due to a side-effect with auro-icon and the
customSize feature. The auro-icon element was updated to allow for more
flexibility in token use.

See AlaskaAirlines/auro-icon#95

This update includes a solution that applies a variable with a variable
fallback with a hard value fallback. The --auro token was specifically
listed first as this is an order of expectation from in-the-wild auro
elements using the legacy --auro token.

It has been verified that regardless of version of tokens a
consumer has installed, or none at all, the customSize feature will work
as expected.

Changes to be committed:
modified:   src/_icons.scss
modified:   src/data/icons.json
This commit updates all the token references to support the new design
token naming spec. All CSS also supports fallback values to ensure
smooth transition from legacy to new.

TODO: There are 2 to-dos in this commit. Once there is an assumption
that most teams have transitioned to the new token spec, the code that
these to-dos are referencing can be removed as well as any npm package
or reference to the legacy tokens.

Changes to be committed:
modified:   demo/index.html
modified:   package-lock.json
modified:   package.json
modified:   src/alaskaStyle.scss
modified:   src/iconStyle.scss
modified:   src/style.scss
This commit also removed the unnecessary dist:js action that is no
longer needed due to the updated .npmignore build pipeline spec.

Changes to be committed:
modified:   package-lock.json
modified:   package.json
@jason-capsule42 jason-capsule42 self-requested a review October 12, 2023 13:34
blackfalcon added a commit to AlaskaAirlines/Icons that referenced this pull request Oct 12, 2023
This commit reverts the revert for using the new Auro design token name
spec. The revert happened due to a side-effect with auro-icon and the
customSize feature. The auro-icon element was updated to allow for more
flexibility in token use.

See AlaskaAirlines/auro-icon#95

This update includes a solution that applies a variable with a variable
fallback with a hard value fallback. The --auro token was specifically
listed first as this is an order of expectation from in-the-wild auro
elements using the legacy --auro token.

It has been verified that regardless of version of tokens a
consumer has installed, or none at all, the customSize feature will work
as expected.

Changes to be committed:
modified:   src/_icons.scss
modified:   src/data/icons.json
blackfalcon added a commit to AlaskaAirlines/Icons that referenced this pull request Oct 12, 2023
This commit reverts the revert for using the new Auro design token name
spec. The revert happened due to a side-effect with auro-icon and the
customSize feature. The auro-icon element was updated to allow for more
flexibility in token use.

See AlaskaAirlines/auro-icon#95

This update includes a solution that applies a variable with a variable
fallback with a hard value fallback. The --auro token was specifically
listed first as this is an order of expectation from in-the-wild auro
elements using the legacy --auro token.

It has been verified that regardless of version of tokens a
consumer has installed, or none at all, the customSize feature will work
as expected.

Changes to be committed:
modified:   src/_icons.scss
modified:   src/data/icons.json
The code changes here represent the last step of removing any repo
dependency on the legacy --auro design tokens.

Changes to be committed:
modified:   demo/index.html
modified:   package-lock.json
modified:   package.json
modified:   src/style.scss
jordanjones243
jordanjones243 previously approved these changes Oct 30, 2023
This commit updates the default URI from unpkg to jsdelivr as unpkg
continues to be unstable in most cases.

Changes to be committed:
modified:   src/auro-icon.js
Changes to be committed:
modified:   docs/api.md
Changes to be committed:
modified:   package-lock.json
modified:   package.json
package.json Outdated Show resolved Hide resolved
Changes to be committed:
modified:   package.json
@blackfalcon blackfalcon merged commit 8d3151f into master Oct 30, 2023
2 checks passed
@blackfalcon blackfalcon deleted the dsande/newTokens/94 branch October 30, 2023 21:19
@blackfalcon
Copy link
Member Author

🎉 This PR is included in version 4.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@blackfalcon blackfalcon added the released Completed work has been released label Oct 30, 2023
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.

auro-icon: support new WCSS and auro design tokens
3 participants