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

Add language options on mobile #95

Merged
merged 13 commits into from
Jul 31, 2024
Merged

Add language options on mobile #95

merged 13 commits into from
Jul 31, 2024

Conversation

Calinator444
Copy link
Member

@Calinator444 Calinator444 commented Jul 30, 2024

Description

This PR updates the mega menu so you can switch between different versions (i.e. French, Chinese) of the site while on a mobile device. I've based the design on a concept designed by @bettybondoc. The search bar for mobile devices has been moved into the side bar so that the language options are immediately visible from the top of the screen.

TLDR: I converted the search box into a dumb component, moved it into the side navigation bar for mobile devices and moved the form state and logic into the main layout component.

Screenshots

state preserved between mobile and desktop view

Figure: Search window state preserved between desktop and mobile view

image

Figure: New resized window layout

image

Figure: New sidebar layout

image

Figure: New sidebar open layout


search test navbar

Figure: Performing a search using the responsive layout

@Calinator444 Calinator444 requested a review from JackDevAU July 30, 2024 11:22
Copy link
Member

@tkapa tkapa left a comment

Choose a reason for hiding this comment

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

LGTM - After clarification from @jaydenalchin I think this solution has turned out well 👍

@Calinator444 Calinator444 merged commit cb460b3 into main Jul 31, 2024
1 check passed
@Calinator444 Calinator444 deleted the 93-add-flag-mega-menu branch July 31, 2024 00:28
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.

✨ SSW.com.au should have French and China flags in the mega menu (smaller screens)
2 participants