airbnb/babel-plugin-inline-react-svg

Need `includePattern` as option

Aperta

#62 aperta il 5 giu 2019

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)JavaScript (90 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (474 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

My issue is that we want to use this babel transform just for icons that we want to style - inline icons that should have the same color as surrounding text, etc.

On the other hand, we'd like to use webpack's url-loader to handle all larger "illustration" SVGs. We would rather put those in img tags and let them be cached by the browser and not have them in our javascript bundle.

To do so, an includePattern could be used that would only use this babel transform on (for example):

  • *.icon.svg
  • */icons/*.svg

Thoughts? I'm about to make a PR.

Guida contributor

Direzione di ricerca
Familiarizzati con il codice sorgente del plugin, in particolare come vengono attualmente gestite le opzioni. Aggiungi un'opzione 'includePattern' che filtra quali SVG trasformare. Usa una libreria di corrispondenza glob come 'micromatch'. Aggiorna la documentazione di conseguenza.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70