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

swaps out the CSS2 :before with the CSS3 ::before #8630

Merged
merged 1 commit into from
Aug 7, 2018

Conversation

tomjn
Copy link
Contributor

@tomjn tomjn commented Aug 6, 2018

Attempts to address #8618 by swapping :before with ::before in SCSS

Copy link
Member

@noisysocks noisysocks left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Are we able to add a lint rule to catch this? I'm not sure that we have a CSS linter set up. cc. @tofumatt

@noisysocks noisysocks added the [Type] Code Quality Issues or PRs that relate to code quality label Aug 7, 2018
@ntwb
Copy link
Member

ntwb commented Aug 7, 2018

The 'selector-pseudo-element-colon-notation': 'double' rule is enabled in stylelint-config-wordpress, it's just a matter of setting stylelint up for use here with Gutenberg 😱

Previously #1151

@noisysocks
Copy link
Member

Nice. Let's go ahead and get this in then. Is setting up stylelint something you have time to do, @ntwb? If not I can take a crack at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants