Skip to content

fix(webkit): draw the progress bar's track on --bg-placeholder - #893

Merged
isaque-bock-azion merged 4 commits into
mainfrom
fix/progress-bar-track-token
Aug 21, 2026
Merged

fix(webkit): draw the progress bar's track on --bg-placeholder#893
isaque-bock-azion merged 4 commits into
mainfrom
fix/progress-bar-track-token

Conversation

@gabriel-lisboa-azion

Copy link
Copy Markdown
Collaborator

Summary

  • The track used --bg-surface-raised, an opaque surface, so its 3:1 contrast against the fill only held on the background that token was chosen for. On a raised or overlay background the track disappeared into it.
  • --bg-placeholder is an alpha overlay, so the track keeps its contrast on any background the bar sits over. Spec's Theme gaps + a11y notes updated to match (checksum + last_updated refreshed).

How to test

  1. cd packages/webkit && npx vitest run src/components/feedback/progress-bar → 18 passed.
  2. pnpm storybook:dev → Components/Feedback/ProgressBar. The track reads as a recessed groove in both themes; place a bar inside a CardBox (raised) and inside a Dialog (overlay) and the track is still visible against both.

Notes

  • Token swap only; no prop, event, or slot change. catalog.json needs no rebuild (--bg-placeholder is already in its token set).

The track was an opaque raised surface, so it only held its 3:1 contrast against
the fill on the one background that surface was picked for. --bg-placeholder is an
alpha overlay, so the track keeps that contrast on any background the bar is
placed over — canvas, surface, raised or overlay.
@isaque-bock-azion
isaque-bock-azion requested a review from a team August 19, 2026 17:43
@robson-junior-azion
robson-junior-azion removed the request for review from a team August 20, 2026 13:09
@isaque-bock-azion
isaque-bock-azion merged commit 0abc4d0 into main Aug 21, 2026
23 checks passed
@isaque-bock-azion
isaque-bock-azion deleted the fix/progress-bar-track-token branch August 21, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants