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

fix: alignment of secondary menu on mobile #4015 #4176

Merged
merged 1 commit into from
Dec 22, 2023
Merged

Conversation

GrigoreMihai
Copy link
Contributor

@GrigoreMihai GrigoreMihai commented Dec 15, 2023

Summary

Changes the spacing between the text and submenu toggle to keep the elements aligned to the center on mobile even when a submenu is not present.

It also accounts for the previous fix for the submenu alignment #3980 ie. it keeps the elements aligned when the submenu toggle is present.

Will affect visual aspect of the product

YES

Screenshots

Alignment on mobile when submenu is present

Alignment for the secundary menu when submenu's are defined for the primary menu

Test instructions

First we need to test that the secundary menu is aligned to center on mobile as in the issue description:

  1. Create a primary and a secondary menu, without any submenus
  2. Add the primary menu on desktop and the secondary menu in the mobile sidebar
  3. Set the mobile sidebar alignment to center and check the frontend
  4. Create a submenu for the primary menu and check again the mobile sidebar in frontend
  5. Check that the secundary menu is aligned to the center on mobile.

We also need to ensure we do not introduce a regression by breaking the fix for #3980.

For this the steps are:

  1. Create a primary menu with submenu's.
  2. Add the primary menu to the mobile and set the Mobile sidebar to the Center .
  3. Check the alignment between the menu elements.

Check before Pull Request is ready:

Closes #4015.

@pirate-bot pirate-bot added the pr-checklist-incomplete The Pull Request checklist is incomplete. (automatic label) label Dec 15, 2023
@pirate-bot
Copy link
Collaborator

Plugin build for 52fe778 is ready 🛎️!

@GrigoreMihai GrigoreMihai added the pr-checklist-skip Allow this Pull Request to skip checklist. label Dec 15, 2023
@pirate-bot pirate-bot added pr-checklist-complete The Pull Request checklist is complete. (automatic label) and removed pr-checklist-incomplete The Pull Request checklist is incomplete. (automatic label) labels Dec 15, 2023
@irinelenache
Copy link
Contributor

@GrigoreMihai Tested and the issue is fixed now, thank you 🚀

@preda-bogdan preda-bogdan merged commit 1d5a4ae into development Dec 22, 2023
28 of 35 checks passed
@preda-bogdan preda-bogdan deleted the fix/4015 branch December 22, 2023 14:31
@pirate-bot
Copy link
Collaborator

🎉 This PR is included in version 3.7.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Dec 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist. released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants