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

Unknown custom properties inside @media #31

Closed
pepelsbey opened this issue Jan 19, 2017 · 4 comments
Closed

Unknown custom properties inside @media #31

pepelsbey opened this issue Jan 19, 2017 · 4 comments
Labels

Comments

@pepelsbey
Copy link

@pepelsbey pepelsbey commented Jan 19, 2017

In the following file --list-scale property is marked as unknown though it works fine in browsers:

.shower.list {

    position: absolute;
    clip: rect(0 auto auto 0);

    margin: 0;
    padding-top: calc(
        var(--slide-width) *
        var(--list-gutter) *
        var(--list-scale));
    padding-left: calc(
        var(--slide-width) *
        var(--list-gutter) *
        var(--list-scale));

    display: flex;
    flex-wrap: wrap;

    background: #666;

    }

/* Scale */

@media (min-width: 1120px) {
    .shower.list {
        --list-scale: 0.5;
        }
    }
@media (min-width: 2240px) {
    .shower.list {
        --list-scale: 1;
        }
    }

image

@lahmatiy
Copy link
Member

@lahmatiy lahmatiy commented Jan 19, 2017

Thank you for issue.
This is a problem not only @media but in general. Custom properties should be ignored by syntax checker.

@lahmatiy lahmatiy closed this in 91a13b8 Jan 19, 2017
@pepelsbey
Copy link
Author

@pepelsbey pepelsbey commented Jan 19, 2017

Thank you for the quick fix! Looking forward for Atom plugin update ;)

@smelukov
Copy link
Member

@smelukov smelukov commented Jan 19, 2017

@pepelsbey The Atom plugin has updated! 🎉

@pepelsbey
Copy link
Author

@pepelsbey pepelsbey commented Jan 20, 2017

Nice, thank you :)

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.