airbnb/babel-plugin-inline-react-svg

Need `includePattern` as option

Ouverte

#62 ouverte le 5 juin 2019

Voir sur GitHub
 (11 commentaires) (0 réaction) (0 personne assignée)JavaScript (90 forks)github user discovery
enhancementhelp wanted

Métriques du dépôt

Stars
 (474 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Familiarisez vous avec le code source du plugin, en particulier la façon dont les options sont actuellement gérées. Ajoutez une option 'includePattern' qui filtre les SVG à transformer. Utilisez une bibliothèque de correspondance glob comme 'micromatch'. Mettez à jour la documentation en conséquence.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70