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

Recompiled version of bootstrap adds property which breaks placeholder text #630

Closed
andrewryno opened this issue Nov 13, 2011 · 9 comments
Closed
Milestone

Comments

@andrewryno
Copy link

Downloaded bootstrap and had to recompile it to use the grid system we are using for the site. Only changed variables.less. Once I recompiled, placeholder text on all inputs no longer works. Did some testing and found it was due to the "-webkit-transform-style: preserve-3d;" property on inputs and selects. Looked in the forms.less and the pre-compiled bootstrap.css files and didn't see the rule.

Honestly, I'm not sure if this is a problem with the compilers I'm using (Less.app and SimpLESS.app on Mac OS X) but considering they should be using the same compiler, not sure why this property is appearing in my compiled versions but not the pre-compiled version (http://twitter.github.com/bootstrap/1.4.0/bootstrap.min.css). Any ideas? Removing that line fixes placeholder text but I'd rather not have to touch the file beyond editing variables.less.

@mdo
Copy link
Member

mdo commented Nov 17, 2011

Have a jsfiddle to share to document this?

I know we added it in 1.4, but we won't likely take care of this until 2.0 (which is well under way). This particular change isn't in the 2.0-wip branch yet, but we'll address it once we merge all of 1.4 in.

@jozefizso
Copy link

I have issue with the Bootstraip 1.4.0 downloded as ZIP (twitter-bootstrap-v1.4.0-0-gf92759b.zip). The bootstrap.css and bootstrap.min.css both contains the "-webkit-transform-style: preserve-3d;" which causes problems with inputs in Chrome.
These properties are not in this 1.4.0 version: http://twitter.github.com/bootstrap/1.4.0/bootstrap.css

@kevinrenskers
Copy link

I've also found an issue with this added property: close buttons on alerts are no longer visible.

Doesn't seem to be Less.app, as the node.js version has the same problem.

@andrewryno
Copy link
Author

http://jsfiddle.net/28KWN/

Has my recompiled version of bootstrap (grid changed, maybe link colors too, but that's it) and some basic HTML. It doesn't work in Chrome. Remove -webkit-transform-style: preserve-3d; from "input, textarea" and it works. Looks like that is also applied to ".topbar input".

@blanchardjeremy
Copy link

I'm also having the problem with placeholder attributes not being displayed in Chrome.

Just so I understand what's actually going on here: what happens if I set -webkit-transform-style: flat;? What am I breaking or losing out on?

@mdo
Copy link
Member

mdo commented Dec 11, 2011

I believe this isn't part of 2.0-wip, so closing this out.

@mdo mdo closed this as completed Dec 11, 2011
@blanchardjeremy
Copy link

@markdotto, can you explain more? Is this something that won't be fixed? Or is this a duplicate issue? I'm confused by your comment.

Thanks!

@jonaphin
Copy link

Let's reopen this issue. It's very real.

@clacote
Copy link

clacote commented Jan 23, 2012

I faced the same issue with version 1.4. I manually applied this commit (not yet released) which fixed it : a560eb6#lib/mixins.less

@twbs twbs locked and limited conversation to collaborators Jul 28, 2014
DocX pushed a commit to DocX/bootstrap that referenced this issue Sep 16, 2014
in Sass, the vararg form handles both cases
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants