Skip to content

Problem in computedStyle interpolation in @glyphs/wc-core  #44

@Hyzual

Description

@Hyzual

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.
BrokenStyle

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:

  1. npm install @glyphs/wc-core

  2. Import a component directly to bypass issue Duplicate export in @glyphs/wc-core #43:

    import "@glyphs/wc-core/src/icons/FolderOpen";
  3. 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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions