-
Notifications
You must be signed in to change notification settings - Fork 20
Description
Hi,
I've found another problem while trying to use @glyphs/wc-core.
At https://github.com/gorango/glyphs/blob/main/cli/lib/generate/wc.js#L30, there is a missing $:
- <span style={computedStyle}>
+ <span style=${computedStyle}>The "computedStyle" is not interpolated, which results in a broken attribute style="{computedStyle}"
This means that the icons are enormous and cannot be customized by setting the "size" attribute.

Unfortunately, due to issue #43 I cannot provide a codepen for easy reproduction, as it does not allow importing a sub-file. Using a bundler such as webpack, the issue can be reproduced as follow:
-
npm install @glyphs/wc-core -
Import a component directly to bypass issue Duplicate export in
@glyphs/wc-core#43:import "@glyphs/wc-core/src/icons/FolderOpen";
-
Include an icon component in HTML
<core-folder-open variant="path" stroke-width="3" ></core-folder-open>
I could try submitting a PR to fix that if you need me to but I'm not familiar with @glyphs's build system. It's unclear to me if I need to use the glyphs CLI first and then npm commands to rebuild web components.