Skip to content

Align disabled and read-only styling between input and select - #356

Open
zoharma wants to merge 1 commit into
mainfrom
zma/input-select-disabled-readonly
Open

zoharma wants to merge 1 commit into
mainfrom
zma/input-select-disabled-readonly

Conversation

@zoharma

@zoharma zoharma commented Oct 12, 2026

Copy link
Copy Markdown
Contributor
  • Soften disabled and read-only surface tokens (light and dark)
  • Pass disabled to FormControl in the Select story so the label is disabled
  • Document the different read-only props on TextField and Select

@zoharma zoharma added this to the 0.8.0 milestone Oct 12, 2026
@zoharma
zoharma requested a review from a team October 12, 2026 13:53
@zoharma
zoharma force-pushed the zma/input-select-disabled-readonly branch 2 times, most recently from 4d2b8f6 to b7043da Compare October 12, 2026 13:56
@VictoriaBeilsten-Edmands

Copy link
Copy Markdown
Collaborator

@zoharma What do you think about the contrast ratio between the Disabled label and the two-tone background. It's not very readable for me.
Screenshot 2026-10-14 at 16 00 25
Screenshot 2026-10-14 at 16 00 14

@zoharma

zoharma commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor Author

What do you think about the contrast ratio between the Disabled label and the two-tone background. It's not very readable for me.

@VictoriaBeilsten-Edmands I think this would be resolved in #359

States (post #356)

Read only Disabled
image image
image image

- Soften disabled and read-only surface tokens (light and dark)
- Pass `disabled` to FormControl in the Select story so the label is disabled
- Document the different read-only props on TextField and Select
@zoharma
zoharma force-pushed the zma/input-select-disabled-readonly branch from b7043da to 6e72724 Compare October 15, 2026 16:04

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants