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

Colons for pseudo-classes and pseudo-elements #662

Closed
gtramontina opened this issue Apr 29, 2016 · 4 comments · Fixed by #663
Closed

Colons for pseudo-classes and pseudo-elements #662

gtramontina opened this issue Apr 29, 2016 · 4 comments · Fixed by #663
Labels

Comments

@gtramontina
Copy link
Contributor

Copied verbatim from sds/scss-lint#592:


CSS3 introduces a distinction between using single or double colons for pseudo-classes and pseudo-elements. Per MDN:

Sometimes you will see double colons (::) instead of just one (:). This is part of CSS3 and an attempt to distinguish between pseudo-classes and pseudo-elements. Most browsers support both values.

It’d be nice to have a linter to check that:

  • Pseudo-classes have one colon (:)
  • Pseudo-elements have two colons (::)

https://github.com/brigade/scss-lint/pull/634/files

@bgriffith
Copy link
Member

Thanks for the request. Would be a good rule to have!

@bgriffith bgriffith added the rule label Apr 29, 2016
@gtramontina
Copy link
Contributor Author

Cool... I'll take a stab at it .... maybe tonight.

@anpa
Copy link

anpa commented May 19, 2016

Thanks for this, this is very useful! But what happened to this rule? I can see that it was merged but it doesn't seem to be available.

@bgriffith
Copy link
Member

It'll be released with version 1.8 (the next release) - so pretty soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants