Marking this as a draft PR since its validity is still in discussion in trello/discord.
I don't believe there is a trello card here, but there's been some discussion on discord
This PR changes the modal actions on the home page to links.
It also features a fair bit of tech debt:
the button component we're using on the home page to create pill buttons doesn't seem to style things correctly when as={Link} is used
when as={Link} is used, we should have the option to use a to={someLocalUrl} prop to take advantage of gatsby's Link component, which is far more performant than a standard anchor URL link
I left the code to support modals on the homepage intact. There will be some eslint yelling as a result
Pages/Interfaces that will change
/home
Screenshots / video of changes
n/a really, check the deploy preview - three buttons on the homepage are now links
Describe your PR
Marking this as a draft PR since its validity is still in discussion in trello/discord.
I don't believe there is a trello card here, but there's been some discussion on discord
This PR changes the modal actions on the home page to links.
It also features a fair bit of tech debt:
as={Link}
is usedas={Link}
is used, we should have the option to use ato={someLocalUrl}
prop to take advantage of gatsby'sLink
component, which is far more performant than a standard anchor URL linkPages/Interfaces that will change
/home
Screenshots / video of changes
n/a really, check the deploy preview - three buttons on the homepage are now links
Steps to test
1.
Additional notes