-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
Button hover & .btn-default styling #8786
Comments
Filed a separate issue for the navbar: #8790. Refocusing this issue on the button stuff. |
There should be a button class that is clearly secondary, based on color saturation. That's the point of having primary and secondary buttons. Light gray looks secondary versus blue. Black vs blue is not so obvious. |
@pastur Maybe something like this? It's just the .btn class with border-color: #AAAAAA. It'd need a better hover-state too, since the default hover state changes the text color to white, which is obviously unreadable on that button. |
Related issue from #8827:
|
@sslepian I think the border makes the button look smaller. I was thinking in something more like this: |
Need a secondary button for sure. I like that one @pastur but with dark grey text and a very slightly darker background color. |
+1 |
+1 I like @pastur's second try. We need to get this done. |
I preferred the idea of having a "secondary" style, but I guess this sticks more with the conventions throughout the rest of the code. Also, this is now the only button style that has a visible border. Not necessarily a bad thing, but it doesn't seem to fit in. |
@joezimjs I've addressed that in another commit ;). |
I understand the changes made to buttons with RC2, but now I want the dark button back for certain purposes. I can make this myself, but feel a dark button is useful to have as a default option. Could be .btn-inverse or .btn-dark |
We specifically dropped |
+1 for btn-dark |
@mdo In my opinion the contrast in the red button isn't enough. Good improvement on the rest of them though. |
+1 for btn-dark |
+1 for btn-dark We're switching back to rc1 because the new default button doesn't fit in with the rest of the buttons. Accessibility is also an issue, the border is too thin for some people to see, and so they do not realize it's a button until this fact is pointed out to them. |
@shaund Wouldn't it be better to just tweak the CSS than reverting back to a broken RC? |
@quasipickle |
I suppose I could have posted in the BS3 thread, but at some point individual topics about BS3 need to have their own thread.
Buttons
Navbar
.navbar-text
and.navbar-btn
aren't vertically aligned with.brand
. You can see this here: http://twbs.github.io/bootstrap/components/#navbar-buttonsList groups & Panels
Love these. Great addition.
The text was updated successfully, but these errors were encountered: