Skip to content

Fix empty read-only logic and align Dropdown validation precedence#6829

Open
joshwooding wants to merge 2 commits into
mainfrom
fix/readonly-zero-and-dropdown-validation
Open

Fix empty read-only logic and align Dropdown validation precedence#6829
joshwooding wants to merge 2 commits into
mainfrom
fix/readonly-zero-and-dropdown-validation

Conversation

@joshwooding

Copy link
Copy Markdown
Contributor
  • Input: treat numeric 0 as a real value when read-only instead of showing the empty read-only marker
  • PillInput: same read-only empty-value handling for consistency (internal)
  • Dropdown: FormField validation status now takes precedence over the local validationStatus prop, matching the other form controls

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
saltdesignsystem Ready Ready Preview, Comment Jul 17, 2026 2:56pm

Request Review

@changeset-bot

changeset-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: dfa10f6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@salt-ds/core Patch
@salt-ds/lab Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@joshwooding
joshwooding force-pushed the fix/readonly-zero-and-dropdown-validation branch from 26d1004 to df01d57 Compare July 13, 2026 23:20
@joshwooding
joshwooding force-pushed the fix/readonly-zero-and-dropdown-validation branch from df01d57 to 2fc4eec Compare July 15, 2026 09:45
@joshwooding
joshwooding requested a review from tycup July 15, 2026 10:03
@joshwooding joshwooding changed the title Fix read-only Input zero value and align Dropdown validation precedence Fix empty read-only logic and align Dropdown validation precedence Jul 15, 2026
@joshwooding
joshwooding requested review from mark-tate and wemmyo July 15, 2026 12:11
tycup
tycup previously approved these changes Jul 16, 2026
wemmyo
wemmyo previously approved these changes Jul 16, 2026
Comment thread .changeset/dropdown-validation-precedence.md
@joshwooding joshwooding added the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 16, 2026
- Input: treat numeric 0 as a real value when read-only instead of showing the empty read-only marker
- PillInput: same read-only empty-value handling for consistency (internal)
- Dropdown: FormField validation status now takes precedence over the local validationStatus prop, matching the other form controls
- Add e2e tests for Input zero value, Dropdown validation precedence, and ComboBox read-only marker
@joshwooding
joshwooding force-pushed the fix/readonly-zero-and-dropdown-validation branch from 2fc4eec to 864277c Compare July 16, 2026 14:16
@github-actions github-actions Bot removed the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 16, 2026
@joshwooding joshwooding added the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 16, 2026
@github-actions github-actions Bot removed the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 16, 2026
@joshwooding
joshwooding dismissed stale reviews from wemmyo and tycup via f79a044 July 17, 2026 14:47
@joshwooding
joshwooding requested a review from tycup July 17, 2026 14:47
@joshwooding joshwooding added the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 17, 2026
@joshwooding
joshwooding requested a review from wemmyo July 17, 2026 14:48
@joshwooding
joshwooding force-pushed the fix/readonly-zero-and-dropdown-validation branch from f79a044 to dfa10f6 Compare July 17, 2026 14:51
@github-actions github-actions Bot removed the chromatic Run chromatic on the current PR. Will be removed by the CI once submitted. label Jul 17, 2026
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.

3 participants