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

Detect input/@placeholder and textarea/@placeholder separately #257

Closed
wants to merge 3 commits into from
Closed

Detect input/@placeholder and textarea/@placeholder separately #257

wants to merge 3 commits into from

Conversation

mathiasbynens
Copy link
Contributor

Add Modernizr.input.placeholder.input and Modernizr.input.placeholder.textarea, while still allowing the use of Modernizr.input.placeholder as a boolean.

I also noticed the unit tests were broken, so I fixed them.

….textarea, while still allowing the use of Modernizr.input.placeholder as a boolean. Kinda fixes #76.
@mathiasbynens
Copy link
Contributor Author

Let me know if you’d prefer Modernizr.textarea.placeholder, etc. It might make more sense to do so.

@paulirish
Copy link
Member

you had a circular reference in your commit btw. way to test. :p

you'll have to fix my whitespace another time.

patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this pull request Feb 22, 2015
patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this pull request Feb 22, 2015
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

Successfully merging this pull request may close these issues.

2 participants