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

Added !global to $class-type #73

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

emersonthis
Copy link

Silent classes weren't working in newer versions of Sass (3.2+) because variable scope changed. The intended behavior was repaired by overtly specifying !global.

Silent classes weren't working in newer versions of Sass (3.2+) because variable scope changed. The intended behavior was repaired by overtly specifying !global.
@csswizardry
Copy link
Owner

It still seems to work fine without this… are you using a particular version/flavour of Sass?

@emersonthis
Copy link
Author

I'm on Sass 3.3.8 (Maptastic Maple)

Harry Roberts wrote:

It still seems to work fine without this… are you using a particular
version/flavour of Sass?


Reply to this email directly or view it on GitHub
#73 (comment).

@KittyGiraudel
Copy link
Contributor

CSSWizardry is now using a ternary operator to define $class-type rather than a conditional statement so this is no longer an issue.

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.

3 participants