Closed jeroenheijmans closed 2 years ago
I did a quick pass over this, and made a commit with some small fixes and improvements. Figured that'd be faster than pointing everything out, so I hope you don't mind ;)
If that looks okay to you, feel free to merge it!
Actually, one more thing pops to mind: the index.html
. Currently that's in Dutch, should the landing page also become the English version now?
If so, in build.js
we need to change the line:
const langMappings = { nl: 'index' };
to:
const langMappings = { en: 'index' };
and update the language links in the footer
of the template file.
Sounds good! Will add that last tweak and merge 👍🏻
Fixes #39