Skip to content

[docs] Fix Field validity API descriptions#4592

Merged
atomiks merged 3 commits intomui:masterfrom
oliviertassinari:improve-otp-input-api-description
Apr 14, 2026
Merged

[docs] Fix Field validity API descriptions#4592
atomiks merged 3 commits intomui:masterfrom
oliviertassinari:improve-otp-input-api-description

Conversation

@oliviertassinari
Copy link
Copy Markdown
Member

@oliviertassinari oliviertassinari commented Apr 13, 2026

I was having a quick look at #4365. This didn't read quite right; this seems like a typo. I can only think of two options here:

  • "is in a valid state"
  • "is in the valid state"

Judging from https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Selectors/:invalid, the former seems correct.

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. component: otp field labels Apr 13, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 13, 2026

commit: 3461972

@code-infra-dashboard
Copy link
Copy Markdown

code-infra-dashboard bot commented Apr 13, 2026

Bundle size

Bundle Parsed size Gzip size
@base-ui/react 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 1,265.35 ms ▼-174.45 ms(-12.1%) | Renders: 53 (+0) | Paint: 1,974.43 ms ▼-205.15 ms(-9.4%)

Test Duration Renders
Select open (500 options) 47.37 ms 🔺+1.19 ms(+2.6%) 14 (+0)
Menu open (500 items) 81.01 ms 🔺+1.76 ms(+2.2%) 12 (+0)
Dialog mount (300 instances) 78.20 ms ▼-26.93 ms(-25.6%) 2 (+0)
Tabs mount (200 instances) 206.01 ms ▼-41.73 ms(-16.8%) 4 (+0)
Scroll Area mount (300 instances) 89.30 ms ▼-17.72 ms(-16.6%) 3 (+0)

...and 7 more. View full report

Details of benchmark changes


Check out the code infra dashboard for more information about this PR.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 13, 2026

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 3461972
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/69ddf2d747cf6f0008539eb1
😎 Deploy Preview https://deploy-preview-4592--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@oliviertassinari oliviertassinari force-pushed the improve-otp-input-api-description branch from 986cabe to a99695b Compare April 13, 2026 12:11
@atomiks
Copy link
Copy Markdown
Contributor

atomiks commented Apr 13, 2026

All Field/Form components have this same wording at the moment

@oliviertassinari oliviertassinari force-pushed the improve-otp-input-api-description branch from eca885d to 4f657b2 Compare April 13, 2026 12:38
@atomiks atomiks changed the title [otp field][docs] Fix API description [docs] Fix Field validity API descriptions Apr 13, 2026
@oliviertassinari oliviertassinari added the scope: all components Widespread work has an impact on almost all components. label Apr 13, 2026
@atomiks atomiks merged commit 41da0c1 into mui:master Apr 14, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. scope: all components Widespread work has an impact on almost all components. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants