Skip to content

fix(docs-infra): define missing win95 variables in aria autocomplete examples#69939

Open
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:fix-win95-vars-aria-autocomplete
Open

fix(docs-infra): define missing win95 variables in aria autocomplete examples#69939
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:fix-win95-vars-aria-autocomplete

Conversation

@erkamyaman

Copy link
Copy Markdown
Contributor

Superseeds #69428

The retro-themed aria autocomplete examples style the clear button with var(--win95-gray), var(--win95-dark-gray), var(--win95-light), and var(--win95-shadow), but none of the three example stylesheets defines those custom properties. The variables resolve to nothing, so the button falls back to inherited theme colors — color: var(--win95-shadow) becomes the light-theme text color in dark mode, leaving the button unreadable.

Define the four variables in :host of each example, matching the values already used by the sibling aria tree examples, so the clear button keeps its Windows 95 look in both light and dark themes.

Before:

image image

After:

image image

…examples

The retro-themed aria autocomplete examples style the clear button with
var(--win95-gray), var(--win95-dark-gray), var(--win95-light), and
var(--win95-shadow), but none of the three example stylesheets defines
those custom properties. The variables resolve to nothing, so the button
falls back to inherited theme colors — color: var(--win95-shadow) becomes
the light-theme text color in dark mode, leaving the button unreadable.

Define the four variables in :host of each example, matching the values
already used by the sibling aria tree examples, so the clear button keeps
its Windows 95 look in both light and dark themes.
@pullapprove
pullapprove Bot requested a review from bencodezen July 25, 2026 07:59
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Jul 25, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jul 25, 2026
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 25, 2026
@JeanMeche
JeanMeche removed the request for review from bencodezen July 25, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs-infra Angular.dev application and infrastructure target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants