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

Add all input types? #205

Closed
tomzaragoza opened this issue Dec 14, 2014 · 9 comments
Closed

Add all input types? #205

tomzaragoza opened this issue Dec 14, 2014 · 9 comments
Milestone

Comments

@tomzaragoza
Copy link

Was using an input with type="url" but the input wasn't styled properly. Should we add support for styling all different types of inputs?

@luisrudge
Copy link

Yesssss!

@dhg dhg added this to the v2.0.2 milestone Dec 14, 2014
@dhg
Copy link
Owner

dhg commented Dec 14, 2014

Well, my intention was to only style the most common inputs, but I think not having a type="url" input actually is kind of discouraging to using the proper inputs, so I say yes this makes sense.

I'm going to go through the list of text input types and see if they all make sense or just an additional few.

Marking to be added in 2.0.2

@tomzaragoza
Copy link
Author

Awesome, thanks! Would be great to have type="url" styled because on mobile, when a user is going to enter a url, the phone keyboard will have the options to enter www. or .com with just one key. At least, on my phone.

Let me know how I can help!

@dhg
Copy link
Owner

dhg commented Dec 15, 2014

Just for reference - a couple old discussions from v1 with the same issue (just old code): #67
#91

This will be coming soon!

@dhg
Copy link
Owner

dhg commented Dec 16, 2014

Just packaged up and pushed v2.0.2 and it includes support for search, url and tel input types. Considered adding support for others like date, datetime, etc...but didn't seem to make sense to me since I've never used one and the styles for them as inputs are so inconsistent across browsers/devices.

Check out the commit with most of the code that resolves this issue(c34806e) or just go download v2.0.2

@dhg dhg closed this as completed Dec 16, 2014
@clrnd
Copy link

clrnd commented Jan 6, 2015

Oh, sad news. I really wanted "number" and "time" styled ...

@dan1elhughes
Copy link

Why not just style input, regardless of type? If there's a particular type that is specifically not being styled, that could be overridden after the general input declaration.

@r3id
Copy link

r3id commented Jan 7, 2015

I have added these to a version of the CSS. I got the list off W3School to make sure i had them all down correct.

/* Forms
–––––––––––––––––––––––––––––––––––––––––––––––––– */
input[type="date"],
input[type="datetime"],
input[type="datetime-local"],
input[type="email"],
input[type="month"],
input[type="number"],
input[type="password"],
input[type="range"],
input[type="search"],
input[type="tel"],
input[type="text"],
input[type="time"],
input[type="url"],
input[type="week"],
textarea,
select

@knorthfield
Copy link

Would be good to add input[type="date"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants