Revert #15697 — Change CSS Caret border-top from dashed to solid #16658
Conversation
| @@ -10,7 +10,7 @@ | |||
| height: 0; | |||
| margin-left: 2px; | |||
| vertical-align: middle; | |||
| border-top: @caret-width-base dashed; | |||
| border-top: @caret-width-base solid; | |||
There was a problem hiding this comment.
This makes the next line unnecessary; please delete the now-redundant next line.
There was a problem hiding this comment.
Also, please apply this fix to line 189 too.
There was a problem hiding this comment.
Fixed redundant line(s) (one on this and one at 189) and made the change to line 189. Sorry about missing those
|
That other issue had a JS Bin example, was this tested in Windows and OS X versions of all browsers? |
|
I don't have access to a windows machine to test this on, I have tested this in OS X on Firefox, Safari, Opera, and Chrome. In OS X though there is no noticeable difference between the two caret styles that I could tell in the latest versions of each of these browsers. |
|
This is on hold due to the lack of confirmation in #16657. |
|
Closing since we haven't been able to confirm/reproduce #16657. |
Fix #16657
Reverts #1597 which fixed #15682
EDIT:
Is it worth adding a short inline comment why this is solid not dashed?