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

Customize UI components: TextField. #864

Merged
merged 2 commits into from
Jun 18, 2015

Conversation

misha-panyushkin
Copy link
Contributor

This update targeted on UI components customization with developers styles pointed out in component theme. As the main part of the PR was introduced TextField component theme and bound it with TextField component itself.

Also there were added hint and label color themes (@hai-cea need your permit for the changes of this kind).

Also added customization of the textfield hint and float label lineHeight style property according to custom height.

@hai-cea
Copy link
Member

hai-cea commented Jun 16, 2015

@misha-panyushkin Yes, I think your PR is good - just have a small code cleanup suggestion. 👍

@misha-panyushkin
Copy link
Contributor Author

@hai-cea done.

@misha-panyushkin
Copy link
Contributor Author

Also made two steps:

  • corrected textField error state mode for manual setErrorText method call;
  • introduce textArea setValue method for correct textArea height calculation.

@misha-panyushkin
Copy link
Contributor Author

  • updated text field underline disabled mode to more flexible one.

- add text field component theme;
- bind text field component with its theme;
- introduce hint and label colors in text field theme (need author permit);
- customize textfield hint or float label with custom line height;
- caching method call result;
- corrected error mode state via setErrorText method call;
- introduced textarea setValue method for correct height calculation;
- updated disabled text field mode underline to more flexible one;
- syntax error fix.
@misha-panyushkin
Copy link
Contributor Author

@hai-cea syntax error fixed.

- renaming method.
hai-cea added a commit that referenced this pull request Jun 18, 2015
@hai-cea hai-cea merged commit 8d26add into mui:master Jun 18, 2015
@hai-cea
Copy link
Member

hai-cea commented Jun 18, 2015

Thanks @misha-panyushkin !

@zannager zannager added the component: text field This is the name of the generic UI component, not the React module! label Mar 14, 2023
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

3 participants