BuilderIO/mitosis

BUG: Mitosis does not camelCase HTML attributes in React

Open

#526 ouverte le 28 juin 2022

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)TypeScript (411 forks)batch import
coregood first issuereact

Métriques du dépôt

Stars
 (9 803 stars)
Métriques de merge PR
 (Merge moyen 1j 1h) (6 PRs mergées en 30 j)

Description

Scope

  • This impacts ALL Mitosis outputs
  • This only impacts specific generators/outputs (please list them here): React

Describe the bug Attributes like srcset are camelCased in React (possibly other frameworks? Not sure). However, Mitosis does not account for that

To Reproduce

A link to a https://mitosis.builder.io/ fiddle containing the bug: link

Expected behavior scrset should become srcSet in the React output. Same goes for all HTML attributes that are multi-word: https://reactjs.org/docs/dom-elements.html

Additional context

  • I am not sure what the comprehensive list of camelCased DOM elements is, but we can probably look at React's JSX types to determine what to do
  • I am also not sure if other frameworks do this camelCasing. We might need to do this elsewhere too

Guide contributeur

Direction de recherche
Identifiez tous les attributs HTML composés de plusieurs mots et ajoutez un mappage camelCase pour la sortie React. Référez vous à la documentation JSX de React pour la liste des attributs camelCase.
Stack technique
typescriptreact
Domaine
compilers
Type d'issue
Bug
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
GitTypeScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75