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

Added note to documentation about placeholders not working in IE7. #2842

Closed
wants to merge 2 commits into from

Conversation

joshuapinter
Copy link

As per issue: #2417

@mdo
Copy link
Member

mdo commented Mar 29, 2012

I don't feel this is something we need to put into the BS docs. If we went down this route, we'd set precedence for adding notes about compatibility for everything. This is different than the progress bars component, which doesn't work exactly the same in every browser, because it's a custom component we made from scratch. Placeholder is a browser feature.

@mdo mdo closed this Mar 29, 2012
@joshuapinter
Copy link
Author

Fair enough Mark.

I just think it would catch a number of people by surprise when they create forms with just placeholder labels and in IE it's completely useless.

Thanks for a the work, loving Bootstrap 2.

Joshua Pinter

On Wednesday, 28 March, 2012 at 8:46 PM, Mark Otto wrote:

I don't feel this is something we need to put into the BS docs. If we went down this route, we'd set precedence for adding notes about compatibility for everything. This is different than the progress bars component, which doesn't work exactly the same in every browser, because it's a custom component we made from scratch. Placeholder is a browser feature.


Reply to this email directly or view it on GitHub:
#2842 (comment)

@samography
Copy link

For anyone stumbling on to this pull request, the placeholder attribute does not work in IE9 either. There's a janky workaround using Modernizr, though.

@ctalkington
Copy link
Contributor

I would recommend github.com/jamesallardice/Placeholders.js, doesnt require jQuery and supports IE6+ along with a few other browsers. It doesn't kick in unless needed. Also, can be used with modernizer and a loader like this:

// HTML5 Placeholder Attribute Test
if (!Modernizr.input.placeholder) {
  require(['placeholders'],function() {
    Placeholders.init()
  })
}

@matiasurbano
Copy link

Thanks ctalkington for your reply.. that's works for me.. But It will be great if twitter boostrap include this issue for the next release.

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.

5 participants