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: restore missing marketing padding utilities #695

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

shawnbot
Copy link
Contributor

I flubbed the merge of master into #636, so this is my attempt at restoring those changes. (I'm so sorry, Tyson!)

Running script/selector-diff-report locally shows a bunch of new classnames. @trosage and/or @gladwearefriends, do you mind double-checking the code and this list to make sure I got it right? 🙇

> .p-7
> .pr-7
> .pl-7
> .p-8
> .pr-8
> .pl-8
> .p-9
> .pr-9
> .pl-9
> .p-10
> .pr-10
> .pl-10
> .p-11
> .pr-11
> .pl-11
> .p-12
> .pr-12
> .pl-12
> .p-sm-7
> .pr-sm-7
> .pl-sm-7
> .p-sm-8
> .pr-sm-8
> .pl-sm-8
> .p-sm-9
> .pr-sm-9
> .pl-sm-9
> .p-sm-10
> .pr-sm-10
> .pl-sm-10
> .p-sm-11
> .pr-sm-11
> .pl-sm-11
> .p-sm-12
> .pr-sm-12
> .pl-sm-12
> .p-md-7
> .pr-md-7
> .pl-md-7
> .p-md-8
> .pr-md-8
> .pl-md-8
> .p-md-9
> .pr-md-9
> .pl-md-9
> .p-md-10
> .pr-md-10
> .pl-md-10
> .p-md-11
> .pr-md-11
> .pl-md-11
> .p-md-12
> .pr-md-12
> .pl-md-12
> .p-lg-7
> .pr-lg-7
> .pl-lg-7
> .p-lg-8
> .pr-lg-8
> .pl-lg-8
> .p-lg-9
> .pr-lg-9
> .pl-lg-9
> .p-lg-10
> .pr-lg-10
> .pl-lg-10
> .p-lg-11
> .pr-lg-11
> .pl-lg-11
> .p-lg-12
> .pr-lg-12
> .pl-lg-12
> .p-xl-7
> .pr-xl-7
> .pl-xl-7
> .p-xl-8
> .pr-xl-8
> .pl-xl-8
> .p-xl-9
> .pr-xl-9
> .pl-xl-9
> .p-xl-10
> .pr-xl-10
> .pl-xl-10
> .p-xl-11
> .pr-xl-11
> .pl-xl-11
> .p-xl-12
> .pr-xl-12
> .pl-xl-12

Originally added in 55e7fc by @trosage

Co-authored-by: Tyson Rosage <trosage@github.com>
@shawnbot shawnbot mentioned this pull request Feb 27, 2019
12 tasks
Copy link
Contributor

@gladwearefriends gladwearefriends left a comment

Choose a reason for hiding this comment

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

all looks right to me!

@gladwearefriends
Copy link
Contributor

is there a reason why pb- and pts aren't in that diff list though?

@shawnbot
Copy link
Contributor Author

@gladwearefriends it's because those were already in there. ✨

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.

2 participants