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

Alias with -st-extend shouldn't be possible #1792

Open
idoros opened this issue May 9, 2021 · 0 comments
Open

Alias with -st-extend shouldn't be possible #1792

idoros opened this issue May 9, 2021 · 0 comments
Labels
bug Unexpected behavior or exception core Processing and transforming logic

Comments

@idoros
Copy link
Collaborator

idoros commented May 9, 2021

Currently when adding -st-extends to an alias it:

  1. warns for unscoped class, because we used it without any root scoping
  2. creates a local class symbol because it's a simple class with -st-extends

I think it should allow the local definition first and error with Import declaration conflicts with local declaration of 'symbol name'

example:

@st-import [icon] from "./button.st.css";
@st-import OtherIcon from "./other-icon.st.css";

.icon { /* is this a local definition or an import alias? */
    -st-extends: OtherIcon;
} 
@idoros idoros added bug Unexpected behavior or exception core Processing and transforming logic labels May 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected behavior or exception core Processing and transforming logic
Projects
None yet
Development

No branches or pull requests

1 participant