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

Missing carets in bootstrap v3.3.4 in IE8 #16172

Closed
simonbengtsson opened this issue Mar 27, 2015 · 3 comments · Fixed by #16258
Closed

Missing carets in bootstrap v3.3.4 in IE8 #16172

simonbengtsson opened this issue Mar 27, 2015 · 3 comments · Fixed by #16258

Comments

@simonbengtsson
Copy link

Bootstrap v3.3.4 made carets go missing in IE8. Here is a JSFiddle and here screenshots for IE8-IE10 (only IE8 is missing the caret). Reverting #15697 makes IE8 happily show carets again however.

In my projects I add a .legacy-ie class to the html element if <=IE9 so for me this can easily be fixed by adding this .legacy-ie .caret { border-top: @caret-width-base solid; }. However, I suppose bootstrap 3 supports IE8 and this should be fixed officially?

@twbs-lmvtfy
Copy link

Hi @someatoms!

You appear to have posted a live example (https://fiddle.jshell.net/DTcHh/5998/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

  • line 38, column 3: W007: Found one or more <button>s missing a type attribute.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!

(Please note that this is a fully automated comment.)

@simonbengtsson
Copy link
Author

Revised example with button type: https://jsfiddle.net/DTcHh/5999/

@cvrebert cvrebert added the css label Mar 27, 2015
@cvrebert
Copy link
Collaborator

Perhaps we should add something like border-top: @caret-width-base solid\9; to hack this for IE8.

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

Successfully merging a pull request may close this issue.

3 participants