-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix iPad Layouts #61
Comments
Hey, can you post issue ss, please? |
Hey there, my bad. I'll add screenshots to the original comment. |
Added screenshots and additional information. |
Hey @swapnalshahil , do you want to pick this up? Should I assign it to you? |
If no one is assigned , can you assign this issue to me ? |
Sure @IzacVrushabh ! |
Hey @IzacVrushabh , any updates on this? |
I'm getting some problems with installation . I am unassigning myself so others can work on it. |
Oh alright |
The layout of the app does not look good on iPads in portrait mode (768x1024 px).
This could be fixed by making the
CourseSelection
component wider and theSlotTable
narrower, and changing the column count for it.The
Navbar
also needs editing. A possible solution would be to allow it to collapse into a hamburger menu like on mobile screens.The
Timetable
component also overflows. It could be made to split into two parts like on Mobile Screens (eg: 360x740 px).This should be a quick fix by changing the media query here:
FFCSThingy2.0/client/src/components/Timetable/Timetable.tsx
Line 43 in 11d62d5
The timetable component on mobile screens:
The text was updated successfully, but these errors were encountered: