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

Fix .social-count underline #48

Merged
merged 2 commits into from
Mar 28, 2015
Merged

Conversation

flovan
Copy link

@flovan flovan commented Mar 25, 2015

.social-count elements get an underline when pressing down (:active), then dragging out. I didn't see this behaviour in any of the other buttons or text-buttons, but feel free to close this PR if this was a design-choice, rather than something that was overlooked.

Also, I did not know how to "add tests", but maybe that isn't needed for such a small change.

 `.social-count` elements get an underline when pressing down (`:active`), then dragging out of them.
@@ -250,9 +250,13 @@
border-top-right-radius: 3px;
border-bottom-right-radius: 3px;

&:hover,
&:active{

Choose a reason for hiding this comment

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

Opening curly brace { should be preceded by one space

Copy link
Author

Choose a reason for hiding this comment

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

Dang! My bad :(

@mdo mdo added the bug label Mar 25, 2015
@mdo mdo added this to the v2.0.3 milestone Mar 25, 2015
@mdo
Copy link
Contributor

mdo commented Mar 27, 2015

Howdy—thanks for the pull request, @flovan!

Before we can merge your changes, we need you to review our Contributor License Agreement (CLA). We think of CLAs as an open source software license between you and the project, letting us and other users know that we have permission to use the code you contribute.

At the bottom of the pull request, you'll see the note about a failed test. Just click the Details link, review the CLA, and sign off at the bottom.

Questions? Just holler—I'm here to help :).

<3

@flovan
Copy link
Author

flovan commented Mar 28, 2015

Done!

mdo added a commit that referenced this pull request Mar 28, 2015
@mdo mdo merged commit 98a492c into primer:master Mar 28, 2015
@mdo
Copy link
Contributor

mdo commented Mar 28, 2015

Thanks! <3

@mdo mdo mentioned this pull request Mar 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants