Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docs] Number Input docs fixes #38521

Merged
merged 3 commits into from
Aug 29, 2023
Merged

Conversation

mj12albert
Copy link
Member

@mj12albert mj12albert commented Aug 17, 2023

Fixes several issues raised in #36119 (comment)

  1. Arrow aligment in the intro & basic demos:
Screenshot 2023-08-18 at 12 27 28 AM
  1. Fixed indentation in a snippet in the Basics section

  2. Step rounding - I've added a callout about decimals not being supported yet and linking to an issue to upvote

Preview https://deploy-preview-38521--material-ui.netlify.app/base-ui/react-number-input/

@mj12albert mj12albert added docs Improvements or additions to the documentation package: base-ui Specific to @mui/base labels Aug 17, 2023
@mui-bot
Copy link

mui-bot commented Aug 17, 2023

Netlify deploy preview

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against fe1ffa7

@mj12albert mj12albert force-pushed the docs/number-input-docs branch from e6a0d27 to 3ea9c60 Compare August 17, 2023 16:20
@mj12albert mj12albert marked this pull request as ready for review August 17, 2023 18:12
@oliviertassinari oliviertassinari changed the title [docs] Number input docs fixes [docs] Number Input docs fixes Aug 18, 2023
@oliviertassinari oliviertassinari added the component: number field This is the name of the generic UI component, not the React module! label Aug 18, 2023
@mnajdova mnajdova requested a review from michaldudak August 23, 2023 07:38
@@ -25,7 +25,7 @@ const pages: readonly MuiPage[] = [
{ pathname: '/base-ui/react-button', title: 'Button' },
{ pathname: '/base-ui/react-checkbox', title: 'Checkbox', planned: true },
{ pathname: '/base-ui/react-input', title: 'Input' },
{ pathname: '/base-ui/react-number-input', title: 'Number Input' },
{ pathname: '/base-ui/react-number-input', title: 'Number Input', newFeature: true },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove this and go with the solution from #38573 when it's merged.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

@mj12albert mj12albert force-pushed the docs/number-input-docs branch from 3ea9c60 to fe1ffa7 Compare August 29, 2023 03:24
@mj12albert mj12albert requested a review from michaldudak August 29, 2023 03:52
@mj12albert mj12albert merged commit 2ae2e79 into mui:master Aug 29, 2023
@mj12albert mj12albert deleted the docs/number-input-docs branch August 29, 2023 08:33
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Sep 8, 2023
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: number field This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation package: base-ui Specific to @mui/base
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants