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

TextField step setting needs inputProps instead of 'I'nputProps #25546

Closed
2 tasks done
aykutkilic opened this issue Mar 29, 2021 · 1 comment
Closed
2 tasks done

TextField step setting needs inputProps instead of 'I'nputProps #25546

aykutkilic opened this issue Mar 29, 2021 · 1 comment
Labels
duplicate This issue or pull request already exists

Comments

@aykutkilic
Copy link

aykutkilic commented Mar 29, 2021

I need a TextField where the user enters a number with certain step size and also there's an adornment in the end. To get this working I need to set inputProps for step size and InputProps for the adornment which is confusing for the developer.

Is it possible to merge them or are there any good reasons to keep them separate?

  • The issue is present in the latest release.
  • I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Expected Behavior 🤔

Steps to Reproduce 🕹

Steps:

  1. Check out the CodeSandbox link

Context 🔦

Your Environment 🌎

System:
OS: Windows 10 10.0.18363
Binaries:
Node: 12.19.0 - C:\devtools\nodejs\12.19.0\node.EXE
Yarn: 1.22.5 - C:\devtools\yarn\1.22.5\bin\yarn.CMD
npm: 6.14.8 - C:\devtools\nodejs\12.19.0\npm.CMD
Browsers:
Chrome: Not Found
Edge: Spartan (44.18362.449.0)
npmPackages:
@material-ui/core: ^4.11.0 => 4.11.3
@material-ui/icons: ^4.9.1 => 4.11.2
@material-ui/lab: ^4.0.0-alpha.56 => 4.0.0-alpha.57
@material-ui/styles: 4.11.3
@material-ui/system: 4.11.3
@material-ui/types: 5.1.0
@material-ui/utils: 4.11.2
@types/react: 17.0.3
react: ^17.0.1 => 17.0.1
react-dom: ^17.0.1 => 17.0.1

`npx @material-ui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @material-ui/envinfo` goes here.
@aykutkilic aykutkilic added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Mar 29, 2021
@oliviertassinari oliviertassinari added duplicate This issue or pull request already exists and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Mar 29, 2021
@oliviertassinari
Copy link
Member

Duplicate of #11377

@oliviertassinari oliviertassinari marked this as a duplicate of #11377 Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants