Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extra comma and " | Gatsby" in showcase details title #18571

Closed
rosslh opened this issue Oct 13, 2019 · 3 comments · Fixed by #18799
Closed

Extra comma and " | Gatsby" in showcase details title #18571

rosslh opened this issue Oct 13, 2019 · 3 comments · Fixed by #18799

Comments

@rosslh
Copy link
Contributor

rosslh commented Oct 13, 2019

Description

Showcase details pages have an extra , and | Gatsby in the document <title/>

Steps to reproduce

Expected result

  • The title should be MetaProjection: Showcase | GatsbyJS
  • The title should be ReactJS: Showcase | GatsbyJS

Actual result

  • The title is MetaProjection,: Showcase | GatsbyJS | GatsbyJS
  • The title is ReactJS,: Showcase | GatsbyJS | GatsbyJS

Environment

Reproduced on:

  • OS
    • Kubuntu 16.04
  • Browsers
    • Firefox 69.0.02
    • Chrome 77.0.3865.90
@fboechats
Copy link
Contributor

@rosslh How can I reproduce it locally??

@mrkutly
Copy link
Contributor

mrkutly commented Oct 15, 2019

It looks like this happens when you link directly to a showcase page. If you navigate to any of them from the showcase index, the title is fine. The extra comma only gets added if you go directly to a show page.

Click here to see error: https://www.gatsbyjs.org/showcase/reactjs.org
Click here and navigate to react page to see it behave as expected: https://www.gatsbyjs.org/showcase

Update: Oddly enough, the opposite is true when I run the website locally. Clicking to the page causes the bug, but refreshing or going directly to it works correctly.

I'll keep working on it and try to find a fix for it.

@mrkutly
Copy link
Contributor

mrkutly commented Oct 16, 2019

Digging around I found at least 2 react Helmet components that were tryin to set the title for each page. Setting the prop titleTemplate="" for the Helmet in the showcase-details.js component did the trick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants