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

Implement handling of nested :not pseudo selectors #1046

Closed
wants to merge 1 commit into from

Conversation

mgreter
Copy link
Contributor

@mgreter mgreter commented Apr 5, 2015

They should not appear in the output. Ruby sass seems to
have a minor bug, as it inserts additional spaces before
the comma when the pseudo selector gets eliminated.

@mgreter mgreter self-assigned this Apr 5, 2015
@mgreter mgreter added this to the 3.2.1 milestone Apr 5, 2015
@mgreter mgreter force-pushed the feature/eliminate-nested-not branch 2 times, most recently from 0d86dec to b5dfefa Compare April 6, 2015 17:59
@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 80.64% when pulling b5dfefa on mgreter:feature/eliminate-nested-not into f82a41b on sass:master.

@mgreter mgreter mentioned this pull request Apr 30, 2015
9 tasks
They should not appear in the output. Ruby sass seems to
have a minor bug, as it inserts additional spaces before
the comma when the pseudo selector gets eliminated.
@mgreter mgreter force-pushed the feature/eliminate-nested-not branch from b5dfefa to 1bc35fa Compare April 30, 2015 01:43
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 80.31% when pulling 1bc35fa on mgreter:feature/eliminate-nested-not into 9ad775f on sass:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 80.31% when pulling 1bc35fa on mgreter:feature/eliminate-nested-not into 9ad775f on sass:master.

@xzyfer xzyfer modified the milestones: 3.2.1, 3.2.2, 3.2, 3.3 Apr 30, 2015
@mgreter mgreter closed this Jul 16, 2015
@mgreter mgreter deleted the feature/eliminate-nested-not branch July 28, 2015 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants