Legg til et felles API for elm-ikonene #756
Conversation
|
packages/spor-icon-elm/generate.js
Outdated
| const moduleHeader = generateModuleHeader(module.name, svgNames); | ||
| const src = [moduleHeader].concat(svgImpls) | ||
| .join('\n\n\n{-|-}\n'); | ||
| // File names are structured as "name-[fill|stroke]-[size]x[size].svg" |
There was a problem hiding this comment.
"name-[fill|outline]" er det vel. Og alle følger ikke den standarden – noen har "colored" etc
|
Denne er good to go fra min side – bare å merge når du er i mål @FluidSense |
|
Flotters! For min del må det fort bli etter ferien, så i starten av august. Om noen derimot ønsker å plukke opp tråden her før det, så gjenstår det:
Konverterer dette PR-et til draft neste gang jeg er ved macen, i mellomtiden |
FluidSense
force-pushed
the
add-api-to-spor-icons-elm
branch
from
a007864
to
72e54d1
Compare
## Bakgrunn #756 la til et nytt API for elm-ikonene. Dessverre så jeg ikke at den krevde nyeste versjon av elm-css, mens alle prosjekter som bruker pakken har én major-versjon under. ## Løsning Nedgraderer til forrige major-versjon
Bakgrunn
Etter å ha testet Spor-ikonene i Elm en stund har vi avdekket noen smertepunkter.
Løsning
Ulempen med dette er at bruks-API-et til Elm og React fraviker i større grad. Samme endring er så vidt meg bekjent ikke like lett å gjøre for React pga. tree-shaking som kun kan gjøres på modulnivå.