Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

In CSSUtils, detect when in an @ rule correctly #12397

Merged
merged 1 commit into from
May 13, 2016

Conversation

marcelgerber
Copy link
Contributor

Followup for #12393.
I could have sworn it worked the way I had it, but as it turned out, it doesn't. CM has also changed the context of @import from "at" to "atBlock", so now when we're inside, we deal with a "atBlock_parens".

cc @nethip @redmunds

@redmunds
Copy link
Contributor

redmunds commented May 2, 2016

This looks correct according to description above, but I haven't run the code. Can someone verify that this works? Also need to verify that broken tests are now fixed.

@ficristo
Copy link
Collaborator

ficristo commented May 3, 2016

I see the autocomplete with this PR (I've tested against the brackets_shared.less file)
There are still 4 error on Url Code Hinting tests (on master 5 do not pass).
The CSS Code Hinting tests pass both on master and on this PR.

@marcelgerber
Copy link
Contributor Author

The other 4 test failures are caused by #12396.

@ficristo
Copy link
Collaborator

ficristo commented May 3, 2016

The other 4 test failures are caused by #12396.

Good to know!

Then LGTM

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

Successfully merging this pull request may close these issues.

3 participants