Skip to content

Conversation

@BinaryMuse
Copy link
Contributor

@BinaryMuse BinaryMuse commented Oct 2, 2019

This PR increases the specificity of the ButtonGroup child styles so that components styled from Button all render with the correct border styles.

Before

image

After

image

Fixes #570

Merge checklist

  • Changed base branch to release branch package.json version
  • Add or update TypeScript definitions (index.d.ts) if necessary
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari (ButtonGroup is already broken in Safari; see Button group + Safari = unexpected left/right margins #528)
  • Tested in Edge

@BinaryMuse BinaryMuse self-assigned this Oct 2, 2019
@vercel
Copy link

vercel bot commented Oct 2, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://primer-components-git-mkt-fix-button-group-for-all-buttons.primer.now.sh

@vercel vercel bot temporarily deployed to staging October 2, 2019 20:43 Inactive
@BinaryMuse BinaryMuse marked this pull request as ready for review October 2, 2019 20:43
@BinaryMuse BinaryMuse requested a review from emplums October 2, 2019 20:44
@BinaryMuse
Copy link
Contributor Author

@emplums Still need to glance at this on Edge but think it's simple and safe enough for some 👀. Let me know if I did anything wrong here!

@vercel vercel bot temporarily deployed to staging October 2, 2019 20:45 Inactive
Copy link

@emplums emplums left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks! :)

@BinaryMuse BinaryMuse merged commit 34c1cfb into master Oct 2, 2019
@BinaryMuse BinaryMuse deleted the mkt/fix-button-group-for-all-buttons branch October 2, 2019 21:24
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.

ButtonPrimary and ButtonOutline styled incorrectly inside ButtonGroup

3 participants