Landing Page and Dashboard Mobile Responsiveness #110
Conversation
ZoeYieYeo
requested review from
du-dhartley and
romil-bijarnia
as code owners
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5d022e8ac1
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| // Track whether the current viewport is mobile or tablet | ||
| const [isMobile, setIsMobile] = useState(() => { | ||
| if (typeof window === "undefined") return false; | ||
| return window.innerWidth <= 1024; | ||
| }); |
There was a problem hiding this comment.
In Sidebar.jsx, the component treats widths ≤1024 as “mobile/tablet” and sets isHidden on nav clicks/close, but the CSS that actually hides the sidebar (.sidebar.sidebar--hidden { transform: translateX(-100%); }) only exists under @media (max-width: 480px) in Sidebar.css. On tablet widths (481–1024px) this means the state flips to hidden while the sidebar stays visible and the close button disappears, so users can’t actually dismiss the sidebar at those sizes. Consider matching the JS breakpoint to 480px or extending the hidden rule to cover ≤1024px.
Useful? React with 👍 / 👎.
romil-bijarnia
left a comment
romil-bijarnia
left a comment
There was a problem hiding this comment.
It can only be merged after the merge conflicts from the main and the errors get fixed
No description provided.