Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Can't autocomplete css-next vars and mixins #56

Open
weichensw opened this issue Jul 8, 2016 · 4 comments
Open

Can't autocomplete css-next vars and mixins #56

weichensw opened this issue Jul 8, 2016 · 4 comments

Comments

@weichensw
Copy link

weichensw commented Jul 8, 2016

.class {
  background-color: var(--custom-color);
  @apply(--custom-mixin);
}

The default provider is at least collecting the part after --. However css provider disabled default provider.

atom/autocomplete-plus#740

@WoodyWoodsta
Copy link

Given that atom/language-css#99 was recently merged and is in the pipeline for Atom 1.15, what is the status of this issue? It would be great if custom variable auto-completion could accompany the added support for that syntax in language-css.

@50Wliu
Copy link
Contributor

50Wliu commented Jan 29, 2017

It may be feasible now, but someone will probably have to create a PR for it.

@WoodyWoodsta
Copy link

I don't mind having a go, after I understand whether or not #57 is a complete fix as was indicated.

@50Wliu
Copy link
Contributor

50Wliu commented Jan 29, 2017

I don't believe #57 actually fixes this. All it does is allow the default provider to also provide suggestions, which I think is undesirable since that introduces the possibility of nonsensical suggestions (such as display: background-image instead of display: block).

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 a pull request may close this issue.

3 participants