Conversation
There was a problem hiding this comment.
Hey Bryan, looks great overall, but I think one of the changes relating to the sidebar have caused a visual bug:
main |
81-implement-30-minute-skills-page |
|---|---|
![]() |
![]() |
I used the FOI guide page to demonstrate this since it exists on both branches, but this also affects the 30 Minute Skills page and others that use the sidebar template. The title is getting padded too much from the left, so it is no longer aligned with the rest of the page. I think the px-10 added here is what's causing that. Could you please take a look? Also, until the navbar and footer are updated to be mobile responsive, you can temporarily comment out those components in pages/_app.tsx before testing on mobile so they don't interfere with the responsiveness of the rest of the site (but don't commit that change).


Most changes made in existing components were made to match the Figma page
EducationInitiativeWrapper.tsx
PastRecordingDisplay.tsx
Sidebar.tsx
SidebarWrapper,.tsx
thirty-minute-skills.tsx
globals.css