Skip to content

refactor(semantic-shadow-tokens): update inconsistent token styles#12020

Closed
matgalla wants to merge 14 commits intodevfrom
matgalla/#10050-update-inconsistent-shadow-tokens
Closed

refactor(semantic-shadow-tokens): update inconsistent token styles#12020
matgalla wants to merge 14 commits intodevfrom
matgalla/#10050-update-inconsistent-shadow-tokens

Conversation

@matgalla
Copy link
Copy Markdown
Contributor

@matgalla matgalla commented Apr 22, 2025

Related Issue: #10050

Summary

Adds new semantic color tokens and updates existing semantic shadow tokens

Acceptance criteria

  • add --calcite-color-shadow-1
    • light mode: rgba({core.color.neutral.blk-240}, {core.opacity.4})
    • dark mode: rgba({core.color.neutral.blk-240}, {core.opacity.12})
  • add --calcite-color-shadow-2
    • light mode: rgba({core.color.neutral.blk-240}, {core.opacity.8})
    • dark mode: rgba({core.color.neutral.blk-240}, {core.opacity.12})
  • add --calcite-color-shadow-3
    • light mode: rgba({core.color.neutral.blk-240}, {core.opacity.10})
    • dark mode: rgba({core.color.neutral.blk-240}, {core.opacity.16})
  • update --calcite-color-shadow-sm
    • shadow 1:
      • "x": "{core.size.default.none}",
      • "y": "{core.size.default.6}",
      • "blur": "{core.size.default.20}",
      • "spread": "{core.size.default.none}",
      • "color": "{semantic.color.shadow.3}",
      • "type": "dropShadow"
    • shadow 2:
      • "x": "{core.size.default.none}",
      • "y": "{core.size.default.2}",
      • "blur": "{core.size.default.8}",
      • "spread": "{core.size.default.none}",
      • "color": "{semantic.color.shadow.1}",
      • "type": "dropShadow"
  • update --calcite-color-shadow-md
    • shadow 1:
      • "x": "{core.size.default.none}",
      • "y": "{core.size.default.6}",
      • "blur": "{core.size.default.20}",
      • "spread": "-{core.size.default.4}",
      • "color": "{semantic.color.shadow.3}",
      • "type": "dropShadow"
    • shadow 2:
      • "x": "{core.size.default.none}",
      • "y": "{core.size.default.4}",
      • "blur": "{core.size.default.12}",
      • "spread": "-{core.size.default.2}",
      • "color": "{semantic.color.shadow.2}",
      • "type": "dropShadow"

@github-actions github-actions bot added the refactor Issues tied to code that needs to be significantly reworked. label Apr 22, 2025
@matgalla matgalla added skip visual snapshots Pull requests that do not need visual regression testing. design-tokens Issues requiring design tokens. calcite-design-tokens Issues specific to the @esri/calcite-design-tokens package. visual changes Issues with visual changes that are added for consistency, but are not backwards compatible. labels Apr 22, 2025
@matgalla matgalla added this to the 2025-04-29 - Apr Milestone milestone Apr 22, 2025
@matgalla matgalla requested review from ashetland and jcfranco April 22, 2025 20:03
Comment thread packages/calcite-design-tokens/src/tokens/semantic/shadow.json Outdated
Copy link
Copy Markdown
Contributor

@ashetland ashetland left a comment

Choose a reason for hiding this comment

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

LGTM!

@ashetland
Copy link
Copy Markdown
Contributor

Do we need to update the pr description above?

@matgalla
Copy link
Copy Markdown
Contributor Author

Do we need to update the pr description above?

updated!

@matgalla
Copy link
Copy Markdown
Contributor Author

Ignore the previous 3 commits.. #wrongbranch 😅

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 8, 2025

This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions.

@github-actions github-actions bot added the Stale Issues or pull requests that have not had recent activity. label May 8, 2025
@github-actions github-actions bot removed the Stale Issues or pull requests that have not had recent activity. label May 16, 2025
@github-actions
Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions.

@github-actions github-actions bot added the Stale Issues or pull requests that have not had recent activity. label May 24, 2025
@matgalla
Copy link
Copy Markdown
Contributor Author

Fresh PR: #14152

@matgalla matgalla closed this Mar 24, 2026
@matgalla matgalla deleted the matgalla/#10050-update-inconsistent-shadow-tokens branch April 9, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

calcite-design-tokens Issues specific to the @esri/calcite-design-tokens package. design-tokens Issues requiring design tokens. refactor Issues tied to code that needs to be significantly reworked. skip visual snapshots Pull requests that do not need visual regression testing. Stale Issues or pull requests that have not had recent activity. visual changes Issues with visual changes that are added for consistency, but are not backwards compatible.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants