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 .border-dashed #824

Merged
merged 7 commits into from Jun 20, 2019
Merged

Fix .border-dashed #824

merged 7 commits into from Jun 20, 2019

Conversation

simurai
Copy link
Contributor

@simurai simurai commented Jun 17, 2019

This fixes the .border-dashed utility.

Before After
image image
image image

Problem

It seems that .border-dashed gets overridden by the responsive variant.

Fix

Moving .border-dashed below the responsive variants fixes the issue.


Fixes #822

Emily Plummer and others added 6 commits May 7, 2019 14:45
Co-Authored-By: Shawn Allen <shawn.allen@github.com>
This is not needed anymore since it gets added with the responsive mixin
Below responsive variants
@vercel
Copy link

vercel bot commented Jun 17, 2019

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

@@ -2,9 +2,6 @@
// stylelint-disable primer/selector-no-utility
// stylelint-disable block-opening-brace-space-before, comment-empty-line-before

/* Add a gray border on all edges */
.border { border: $border !important; }
Copy link
Contributor Author

@simurai simurai Jun 17, 2019

Choose a reason for hiding this comment

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

This can be removed since it already gets added part of the responsive variants below.

@simurai simurai requested a review from shawnbot June 17, 2019 01:02
@simurai simurai moved this from Release backlog to Ready to release in 📦 Primer CSS release tracking Jun 17, 2019
Copy link
Contributor

@shawnbot shawnbot left a comment

Choose a reason for hiding this comment

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

Nice. Let's include this in #810.

@shawnbot shawnbot changed the base branch from master to release-12.4.1 June 20, 2019 17:52
@shawnbot shawnbot mentioned this pull request Jun 20, 2019
13 tasks
@shawnbot shawnbot merged commit 9e54555 into release-12.4.1 Jun 20, 2019
📦 Primer CSS release tracking automation moved this from Ready to release to 💜 Done Jun 20, 2019
@shawnbot shawnbot deleted the fix-border branch June 20, 2019 18:03
@slothelle
Copy link

Follow up - looks to be related: https://github.com/github/github/issues/117836 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants