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

[Input] Made the labels for adorned elements not shrink on end adornment #8882

Merged
merged 2 commits into from Oct 28, 2017

Conversation

kf6kjg
Copy link
Contributor

@kf6kjg kf6kjg commented Oct 27, 2017

This provides more consistency.

This was not done with the startAdornment for two reasons:

  1. I don't yet know how, and I don't think it's possible to do generically due to indeterminate adornment width. Could be wrong due to ignorance.
  2. "$ Amount" just doesn't feel right.

Thus I'm open to suggestions.

Closes #8881

As per mui#8881 this provides more consistency.

This was not done with the startAdornment for two reasons:
1) I don't yet know how, and I don't think it's possible to do generically due to indeterminate adornment width.  Could be wrong due to ignorance.
2) "$ Amount" just doesn't feel right.

Thus I'm open to suggestions.
@kf6kjg
Copy link
Contributor Author

kf6kjg commented Oct 27, 2017

And the Argos difference is expected :)

@oliviertassinari oliviertassinari changed the title Made the labels for adorned elements not shrink on end adornment [Input] Made the labels for adorned elements not shrink on end adornment Oct 28, 2017
@oliviertassinari oliviertassinari added component: text field This is the name of the generic UI component, not the React module! PR: accepted labels Oct 28, 2017
@oliviertassinari oliviertassinari merged commit 1678930 into mui:v1-beta Oct 28, 2017
@oliviertassinari
Copy link
Member

@kf6kjg Thank you :)

@kf6kjg kf6kjg deleted the shrinkray branch October 29, 2017 18:25
the-noob pushed a commit to the-noob/material-ui that referenced this pull request Nov 17, 2017
…ent (mui#8882)

* Made the labels for adorned elements not shrink on end adornment

As per mui#8881 this provides more consistency.

This was not done with the startAdornment for two reasons:
1) I don't yet know how, and I don't think it's possible to do generically due to indeterminate adornment width.  Could be wrong due to ignorance.
2) "$ Amount" just doesn't feel right.

Thus I'm open to suggestions.

* remove some dead code
@eps1lon eps1lon mentioned this pull request Jan 11, 2019
1 task
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!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants