STAPLE-verse / STAPLE

https://staple.science/
2 stars 0 forks source link

Remove dynamic page titles + include in Layout consistently #375

Open chartgerink opened 9 hours ago

chartgerink commented 9 hours ago

This PR does two things:

  1. Remove all dyanmic page titles in favor of static page titles. Fixes #205.
  2. Move all the page titles into the Layout component, where that component was already available.

(1) is a fix for #205, although dynamic page titles may need to be reintroduced later if people express benefit from this. If this is the case, note that we will need to use getServerSideProps to ensure the information is available upon page load, instead of a hook, which causes the flickering.

(2) was something I noticed, and I figured could help streamline the code. It is a minor thing but when adding up tens of these minor fixes across PRs, they start adding up 😊