hasmin icon indicating copy to clipboard operation
hasmin copied to clipboard

Some support for unicode-range and @font-face{font-weight}.

Open galenhuntington opened this issue 4 years ago • 2 comments

Ref #8.

I don't know or really understand this codebase, but I gave it a shot.

  • unicode-range now at least doesn't get mangled, although the parser is primitive and doesn't try to minify.

  • font-weight mostly works, but font-weight: normal bold gets changed to font-weight:700, which is wrong, and I haven't yet figured out why normal gets eaten. One question is whether it's better to leave this bug or defer support.

galenhuntington avatar May 10 '20 03:05 galenhuntington

Thanks! My apologies for the silence, you caught me in a very busy time in my life unfortunately :/ I'll try to work off this to add support while fixing the bug you mentioned. Do you need this fast, or can you wait for me a bit longer?

On a side note, do you mind me asking in which context are you using this project (is it the library or the stand alone executable?). Just curious as to how people are using this.

contivero avatar May 23 '20 09:05 contivero

There's no particular hurry from my end. The above code changes sufficed, and anyway I gave up trying for pure Haskell solutions for that project, and went with Node libraries, largely because I needed Node in the stack anyway for Stylus. I was using the Hasmin library, called from a Shake build file.

galenhuntington avatar May 23 '20 09:05 galenhuntington