On mobile, menu is broken

Created on 28 December 2022, almost 2 years ago
Updated 22 April 2024, 8 months ago


Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

🇮🇳India alok_singh

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • 🇮🇳India lokeshsahu Delhi

    I have created a patch to address the above issue. I am attaching a screenshot for reference, showing the state before and after applying the patch.

  • Status changed to Needs review over 1 year ago
  • Issue was unassigned.
  • Status changed to Needs work over 1 year ago
  • @lokeshsahu , I verified the #3 patch and the patch is working but the only issue I can see when we are closing the menu it's showing an old menu style.
    Added screenshot for reference.

  • Status changed to Needs review 9 months ago
  • Hello there,
    First of all the issue summary is not very much clear, I guess we are talking about the main menu only in this regard. If so, I think this is a duplicate issue, Issue #3314868 already exist for the same.
    https://www.drupal.org/project/showcase_lite/issues/3314868 🐛 No mobile menu displayed Needs review

    The patch provided on this ticket seems to be targeting menu class generally which will affect all the menu including but not limited to only main-navigation menu. Also I've checked that the issue still persist on the latest version of the theme which is compatible with D10. So I've addressed the issue on that ticket and raised an MR by incorporating the concepts of Drupal Behaviour and Once function. Please have a look:
    https://git.drupalcode.org/project/showcase_lite/-/merge_requests/10

    The issue is resolved now and incorporated a hamburger menu with a panel slider for the main menu on tablet and mobile devices.
    This is how it looks now on responsive screens

  • Hi , reviewed the MR 10, Applied cleanly and also the menu looks fine. Might look better if the top white space is reduced.

  • Hello, @bindu r,
    The hamburger menu will open up as a panel and the actual menu items are vertically center aligned with respect to the panel, so if we reduce the space from the top, it'll look asymmetrical.

  • Status changed to Needs work 8 months ago
  • 🇮🇳India Nupur Badola

    Still getting the same issue.

Production build 0.71.5 2024