wixplosives / stylable-intelligence

Stylable extension for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=wix.stylable-intelligence
BSD 3-Clause "New" or "Revised" License
17 stars 6 forks source link

Support 3rd party as a source #200

Closed tomrav closed 6 years ago

tomrav commented 6 years ago

Stylable Intelligence should offer support for all existing features when consuming a 3rd party package (from node_modules).

hepiyellow commented 6 years ago

Does this issue include the error I am getting when importing a stylesheet from an external package like here:

:import {
  -st-from: "wix-ui-core/index.st.css";
  -st-named: Tooltip;
}

The error:

[stylable] Imported file 'wix-ui-core/dist/src/baseComponents/Popover/Popover.st.css' not found

FYI: The extension shows so many errors in wix-ui-backoffice that I have to disable it )-:

tomrav commented 6 years ago

Actually sounds like that error originates from Stylable, and the intelligence just displays it. Are you running in a mono-repo project, by any chance?

hepiyellow commented 6 years ago

No. Wix-ui-backoffice is not a monorepo.

On 2018. Apr 11., at 18:36, Tom Raviv notifications@github.com wrote:

Actually sounds like that error originates from Stylable, and the intelligence just displays it. Are you running in a mono-repo project?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

hepiyellow commented 6 years ago

Is there any config that has to be done after installing the stylable-intelligence extension?

On 2018. Apr 11., at 18:36, Tom Raviv notifications@github.com wrote:

Actually sounds like that error originates from Stylable, and the intelligence just displays it. Are you running in a mono-repo project?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

tomrav commented 6 years ago

No, the intelligence extension should not require any configuration. It does relate to what I'm doing now, will take a look.

tempit commented 6 years ago

Removed :vars, @namesapce etc. from todo list, as they have no relation to this feature.

tomrav commented 6 years ago

Resolved in #220