-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
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
Change :valid hack to class not-empty for inputs (also add email type) #170
Change :valid hack to class not-empty for inputs (also add email type) #170
Conversation
0278528
to
1b61b7a
Compare
@hai-cea done |
@@ -98,12 +98,12 @@ | |||
box-sizing: border-box; | |||
padding-bottom: 14px; | |||
|
|||
&[type='text'], &[type='password'] { | |||
&[type='text'], &[type='password'], &[type='email'] { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears we want to style a wide range of HTML5 inputs. For completeness sake, it would make sense to add type selectors for inputs like tel, number, url, search, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an issue #55
So I think it's better to discuss what exact type are going to be supported
Because for example I tried color picker and it looks no good
Would using a classSet be a better option? |
Yes you right its more cleaner |
@ButuzGOL Looks like some recent commits caused a conflict. Can you resolve please? |
Conflicts: src/less/components/input.less
@hai-cea done |
Change :valid hack to class not-empty for inputs (also add email type)
Thanks @ButuzGOL ! |
1b61b7a
Also fix for
Uncaught TypeError: Cannot read property 'getDOMNode' of undefined
focus: function() {
if (this.isMounted()) this.refs.input.getDOMNode().focus();
},