nfl/react-helmet

Unexpected comma in title

Open

#408 ouverte le 9 oct. 2018

Voir sur GitHub
 (9 commentaires) (6 réactions) (0 assignés)JavaScript (17 167 stars) (706 forks)batch import
bughelp wanted

Description

Similar to #286, I continue to see a comma in the title of pages of my Gatsby app and seems to only be a problem if there's an expression in the title. A reproduction of this issue can be seen in the following branch.

https://github.com/corydeppen/gatsby-8655-example/tree/react-helmet-comma

Guide contributeur

Stack technique
reactjavascript
Domaine
frontend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ReactJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Investigate the reproduction branch at https://github.com/corydeppen/gatsby 8655 example/tree/react helmet comma to reproduce the issue. Examine the React Helmet source code, particularly how the title prop is processed and concatenated, to identify where an extra comma is inserted. Check if the bug is related to how expressions in the title are handled, possibly in the side effects or rewrite methods.