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][material-ui][TextField] type="number" error #44963

Open
lalitbansal40 opened this issue Jan 7, 2025 · 10 comments
Open

[docs][material-ui][TextField] type="number" error #44963

lalitbansal40 opened this issue Jan 7, 2025 · 10 comments
Assignees
Labels
component: text field This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation package: material-ui Specific to @mui/material

Comments

@lalitbansal40
Copy link

lalitbansal40 commented Jan 7, 2025

Steps to reproduce

Steps:

  1. Open this link to live example: (required)
bugmui.mp4

Current behavior

Bug in Number Type Textfield if i hold increase button and rease but number is increasing.

Expected behavior

Bug in Number Type Textfield if i hold increase button and rease but number is increasing. but it should not do.

Context

No response

Your environment

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: Number Type Textfied error

@lalitbansal40 lalitbansal40 added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Jan 7, 2025
@samuelsycamore
Copy link
Contributor

Hi @lalitbansal40 👋 It's difficult to understand what exactly is going on in the video you've shared. Can you please create a minimal, reproducible example of the problem you're facing (on CodeSandbox, in a GH repo or similar) so the maintainers can take a look?

@samuelsycamore samuelsycamore added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 7, 2025
@lalitbansal40
Copy link
Author

Expected Behavior:
When i press and hold the increment button, the value should increase continuously. However, as soon as you release the button, the increment action should stop immediately.

Observed Bug:
Instead, the value keeps increasing even after i release the button.

1 similar comment
@lalitbansal40
Copy link
Author

Expected Behavior:
When i press and hold the increment button, the value should increase continuously. However, as soon as you release the button, the increment action should stop immediately.

Observed Bug:
Instead, the value keeps increasing even after i release the button.

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Jan 7, 2025
@lalitbansal40 lalitbansal40 reopened this Jan 7, 2025
@github-actions github-actions bot removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Jan 7, 2025
Copy link

github-actions bot commented Jan 7, 2025

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Note

@lalitbansal40 How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

@lalitbansal40
Copy link
Author

Expected Behavior:
When i press and hold the increment button, the value should increase continuously. However, as soon as you release the button, the increment action should stop immediately.

Observed Bug:
Instead, the value keeps increasing even after i release the button.

@ZeeshanTamboli
Copy link
Member

I am not able to reproduce this in the form props section of the TextField documentation you shared in the description. See the video below:

Screen-Recording.1.mp4

I tried with both my laptop mouse touchpad and with keyboard up and down arrow keys. Releasing them stops the increment/decrement.

@ZeeshanTamboli ZeeshanTamboli added component: text field This is the name of the generic UI component, not the React module! status: waiting for author Issue with insufficient information labels Jan 8, 2025
@lalitbansal40
Copy link
Author

i will send new video with bug.

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Jan 8, 2025
@ZeeshanTamboli ZeeshanTamboli added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 8, 2025
@mj12albert
Copy link
Member

mj12albert commented Jan 8, 2025

I'm surprised we have this example!

We strongly advise against using type="number" with this big callout here which seems contradictory to this, what do you think @samuelsycamore

@samuelsycamore
Copy link
Contributor

We strongly advise against using type="number" with this big callout here which seems contradictory to this, what do you think @samuelsycamore

Ah interesting, I didn't realize this demo implemented something we advise against. Sounds like we should refactor the demo and also consider moving this warning higher up on the page maybe? It's kind of buried towards the bottom. The doc's poor info architecture isn't doing us any favors.

@DiegoAndai DiegoAndai added docs Improvements or additions to the documentation and removed status: waiting for author Issue with insufficient information labels Jan 9, 2025
@DiegoAndai
Copy link
Member

Adding the docs label and assigning Sam to remove this example.

@DiegoAndai DiegoAndai changed the title Number Type Textfied error [docs][material-ui][TextField] type="number" error Jan 9, 2025
@DiegoAndai DiegoAndai added the package: material-ui Specific to @mui/material label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: text field This is the name of the generic UI component, not the React module! docs Improvements or additions to the documentation package: material-ui Specific to @mui/material
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants