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 it does not contain all the text of the label #26781

Closed
2 tasks done
Angelk90 opened this issue Jun 16, 2021 · 8 comments
Closed
2 tasks done

TextField it does not contain all the text of the label #26781

Angelk90 opened this issue Jun 16, 2021 · 8 comments
Labels
component: text field This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists

Comments

@Angelk90
Copy link
Contributor

  • 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 😯

Hi @eps1lon ,

As you can see from the image with very long labeled texts this happens:

Schermata 2021-06-16 alle 13 09 20

Expected Behavior 🤔

That the text of the label does not leave the input field.

Steps to Reproduce 🕹

Link: https://codesandbox.io/s/material-demo-forked-ipsgy?file=/demo.js

@Angelk90 Angelk90 added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Jun 16, 2021
@siriwatknp
Copy link
Member

Thanks for raising this up but to me this does not sound practical. Label should be concise and not too long. If you need long label then the TextField should be long enough to cover the label.

@siriwatknp siriwatknp added the support: question Community support but can be turned into an improvement label Jun 17, 2021
@Angelk90
Copy link
Contributor Author

@siriwatknp :
In some cases a long label is needed, it would not be possible to think of one of these two solutions:

  • That through a prop, if it is true (usually false), then the size of the label that is seen when there is no text, case of the screen, the text is adapted considering the size of the input field and the text.

  • That through a prop, if it is true (usually false), the size of the input field increases its length, trying to cover all the text, as you say.

@eps1lon
Copy link
Member

eps1lon commented Jul 13, 2021

Duplicate of #12255 (resolution: #12255 (comment))

@eps1lon eps1lon closed this as completed Jul 13, 2021
@eps1lon eps1lon added component: text field This is the name of the generic UI component, not the React module! duplicate This issue or pull request already exists and removed support: question Community support but can be turned into an improvement status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jul 13, 2021
@csantos-nydig
Copy link

Duplicate of #12255 (resolution: #12255 (comment))

This issue should be re-opened and evaluated.
#12255 was closed, then opened, then finally fixed by #25012

the comment you referenced is pretty old, keep reading the whole thread:
#12255 (comment)
#12255 (comment)
#12255 (comment)

@Angelk90
Copy link
Contributor Author

@csantos-nydig , @eps1lon , @siriwatknp :

I tried to update several packages, after various problems I managed to get what you see in the video: codesandbox

Registrazione.schermo.2021-07-20.alle.14.04.14.mov

Eventuality counselor can have a solution like this if it is possible.
Link: #12255 (comment)

@eps1lon
Copy link
Member

eps1lon commented Jul 20, 2021

This issue should be re-opened and evaluated.
#12255 was closed, then opened, then finally fixed by #25012

I've re-opened #12255

@Angelk90
Copy link
Contributor Author

@eps1lon : What kind of solution do you have in mind to undertake?

@eps1lon
Copy link
Member

eps1lon commented Jul 20, 2021

@eps1lon : What kind of solution do you have in mind to undertake?

I don't have any solution in mind. I just recognize the problem. Please continue any discussion in the re-opened issue.

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! duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants