Skip to content
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

Mobile View rework #897

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

bryantran24
Copy link

@bryantran24 bryantran24 commented Nov 26, 2024

Issue
Fix the mobile view for the YACS pages so it is usable in mobile view.
closes #896

Photos

Before
The sidebar is overlapping with the schedule which makes things hard to see and pushing it to the side.
image
The color assistance button and export data are not placed on the right and is taking too much unnecessary space.
image
Filter button pushes professors too far to the right so its hard to read.
image

After
Moved sidebar to the bottom and removed the open and close sidebar when in mobile view so efficiently use space.
image
image
Fixed professor and pathways pages by moving the filter buttons to the top to avoid pushing the info and making it hard to read.
image
image

Additional Info

Use ctrl-shift-i and control-shift-m to see mobile view or just move the website to the side

@bryantran24 bryantran24 changed the title Front end rework Mobile View rework Nov 26, 2024
Copy link
Contributor

@dorian451 dorian451 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works pretty well so far; I would just make some adjustments to the component that displays what schedule you are on, which currently bugs out on mobile

Made it so you cant scroll screen vertically and removed duplicate schedule navigators.
YACS-RCOS#897 Added final commits to help future developers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request — Improve Mobile View
2 participants