Move Escape Room breadcrumbs from hero to header#559
Conversation
…o for escape room pages.
✅ Deploy Preview for accessiblecommunity ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
If this approach is approved, I can go ahead an remove the HeroWithBreadcrumbs component, as it would no longer be used. |
brian-montgomery
left a comment
brian-montgomery
left a comment
There was a problem hiding this comment.
Hey Alexis. Your concept looks good, although all of the breadcrumbs are showing up a bit incorrectly (see below).
Let me run this by Rachael and see what she thinks about it.
| href: "/escape-room/our-story", | ||
| }]; | ||
|
|
||
| const breadCrumbs: Breadcrumbs = |
There was a problem hiding this comment.
For these breadcrumbs, we want to Escape Room Home (/escape-room/) to be the "Home" link.
| import ThemedSection from "@components/ThemedSection.astro"; | ||
| --- | ||
|
|
||
| <Layout {metadata} {title}> |
There was a problem hiding this comment.
You didn't update the Layout to pass the crumbs into the ERHeader, so they are all showing up as the default. You need to pass the crumbs to the Layout and in the Layout, pass the crumbs to the HeaderER component.
2 participants
Fixes #404