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.

Sign up for GitHub

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

Jump to bottom

Restore --calcite-color-focus in the design-tokens output #11713

Open
3 of 6 tasks
jcfranco opened this issue Mar 7, 2025 · 0 comments
Open
3 of 6 tasks

Restore --calcite-color-focus in the design-tokens output #11713

jcfranco opened this issue Mar 7, 2025 · 0 comments
Labels
0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. Calcite (dev) Issues logged by Calcite developers. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone needs triage Planning workflow - pending design/dev review.

Comments

@jcfranco
Copy link
Member

jcfranco commented Mar 7, 2025

Check existing issues

Actual Behavior

--calcite-color-focus is no longer represented in a token after #11711.

Expected Behavior

--calcite-color-focus present in both token source JSON and build output. It should output --calcite-color-focus: var(--calcite-color-brand) (custom prop unresolved), to associate it with the brand color while still allowing overrides.

Reproduction Sample

https://unpkg.com/browse/@esri/calcite-design-tokens@3.0.0/dist/css/light.css

Reproduction Steps

Notice how --calcite-color-focus is not defined.

Notes:

Reproduction Version

3.1.0-next.22 (or version that ships #11711)

Relevant Info

Follow up from #11711.

Regression?

No response

Priority impact

impact - p1 - need for current milestone

Impact

No response

Calcite package

  • @esri/calcite-components
  • @esri/calcite-components-react
  • @esri/calcite-design-tokens
  • @esri/calcite-ui-icons
  • @esri/eslint-plugin-calcite-components

Esri team

Calcite (dev)
@jcfranco jcfranco added 0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review. labels Mar 7, 2025
@github-actions github-actions bot added Calcite (dev) Issues logged by Calcite developers. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone labels Mar 7, 2025
@jcfranco jcfranco mentioned this issue Mar 7, 2025
Merged
jcfranco added a commit that referenced this issue Mar 7, 2025
@jcfranco
fix: allow global focus color token to inherit fallback value (#11711)
f8c63e3 
**Related Issue:** #10510  

## Summary  

Removes the global focus color introduced in
[#10512](#10512) to
match the behavior of the deprecated `--calcite-ui-focus-color`.
Previously, `--calcite-color-focus` was defined at `:root`, preventing
it from inheriting updates to `--calcite-color-brand` (its fallback).

This is **not** considered a breaking change because:  

* it only affects outputs that are not publicly documented
* the default color will still be applied as expected
* `--calcite-color-focus` remains available for overriding the focus
color

**Note**: #11713
will restore `--calcite-color-focus` after #11391 lands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. Calcite (dev) Issues logged by Calcite developers. impact - p1 - need for current milestone User set priority impact status of p1 - need for current milestone needs triage Planning workflow - pending design/dev review.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jcfranco