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

💄(frontend) updating the header and leftpanel for responsive #421

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

PanchoutNathan
Copy link
Collaborator

@PanchoutNathan PanchoutNathan commented Nov 13, 2024

Purpose

Previously we added a left panel. We now need to adapt the layout so that it becomes responsive.

We therefore add a burger menu on the left on mobile which, when clicked, deploys the left-panel over all the content.

Enregistrement.de.l.ecran.2024-11-18.a.09.56.02.mov

@PanchoutNathan PanchoutNathan self-assigned this Nov 13, 2024
@PanchoutNathan PanchoutNathan changed the title 💄(frontend) update header layout 💄(frontend) WIP : update header layout Nov 13, 2024
@PanchoutNathan PanchoutNathan force-pushed the new-ui/layout-header branch 2 times, most recently from 069b7b8 to 8268b26 Compare November 14, 2024 07:59
@PanchoutNathan PanchoutNathan force-pushed the new-ui/layout-header branch 4 times, most recently from 1457933 to d75d175 Compare November 18, 2024 08:53
@PanchoutNathan PanchoutNathan changed the title 💄(frontend) WIP : update header layout 💄(frontend) updating the header and leftpanel for responsive Nov 18, 2024
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

The header should be fixed I guess:

scrnli_tPgfH2z7JWGLfl.webm

I find that 1024px is quite large to switch the header in "mobile" state, you still have plenty of place to display the elements, to see with Robin I guess.

@PanchoutNathan PanchoutNathan force-pushed the new-ui/layout-header branch 2 times, most recently from 0abeaf4 to 12906e6 Compare November 19, 2024 11:03
@PanchoutNathan PanchoutNathan force-pushed the new-ui/layout-header branch 2 times, most recently from 5522232 to 4ddcbd7 Compare November 19, 2024 15:15
Previously we added a left panel. We now need to adapt the layout
so that it becomesresponsive.

We therefore add a burger menu on the left on mobile which,
when clicked, deploys the left-panel over all the content.
Deleted two components that were no longer needed following the
implementation of the new interface. This cleanup helps streamline
he codebase and avoid unnecessary maintenance.
Copy link
Collaborator

@AntoLC AntoLC left a comment

Choose a reason for hiding this comment

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

✅ ^^

@PanchoutNathan PanchoutNathan merged commit e101b74 into main-new-ui Nov 19, 2024
10 of 11 checks passed
@PanchoutNathan PanchoutNathan deleted the new-ui/layout-header branch November 19, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants