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

Update _buttons.scss #20641

Closed
wants to merge 1 commit into from
Closed

Update _buttons.scss #20641

wants to merge 1 commit into from

Conversation

kevinruscoe
Copy link
Contributor

The hover/active color is defaulted to white. If the user adds a lightly coloured button (such as yellow), then the white text gets lost. This conditionally checks the color passed to return either white, or the $body-color. Fixes #20609

The hover/active color is defaulted to white. If the user adds a lightly coloured button (such as yellow), then the white text gets lost. This conditionally checks the color passed to return either white, or the $body-color. Fixes #20609
@mdo mdo added this to the v4.1 ideas milestone Sep 7, 2016
beausmith added a commit to beausmith/bootstrap that referenced this pull request Sep 16, 2016
Fixes twbs#20609

PR twbs#20641, another solution to this issue, hard-codes use of $body-color, which may not be the desired color for the text when hovering.
Copy link
Member

@bardiharborow bardiharborow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason you aren't putting spaces around the @else?

@mdo
Copy link
Member

mdo commented Oct 28, 2016

Punting on this after merging in #20734 just now. Thanks though!

@mdo mdo closed this Oct 28, 2016
@mdo mdo removed this from the v4.1 ideas milestone Oct 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants