Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

Approach refresh breaks page #122

Closed
moeriki opened this issue Dec 10, 2019 · 2 comments
Closed

Approach refresh breaks page #122

moeriki opened this issue Dec 10, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@moeriki
Copy link
Collaborator

moeriki commented Dec 10, 2019

Steps

  1. Go to https://foursevens.be/nl/approach/
  2. Refresh

Expected result

Page looks the same.

Actual result

Text is gone.
@moeriki moeriki added the bug Something isn't working label Dec 10, 2019
@moeriki
Copy link
Collaborator Author

moeriki commented Jan 8, 2020

Now it even seems to break by default, very shortly after the initial render.

@moeriki
Copy link
Collaborator Author

moeriki commented Jan 22, 2020

Bug found.

gatsbyjs/gatsby#11108
facebook/react#5479

Sollution: don't user {{dangerouslySetInnerHTML}} on P tags.

@moeriki moeriki closed this as completed Jan 22, 2020
moeriki added a commit that referenced this issue Jan 22, 2020
@moeriki
fix(#122): don't accidentally nest paragraphs
600bc7a
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@moeriki