Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
When menu item has sub item then layout is break, see the attached file (menu-error.png). Also sub item text is not visible when it has no sub item, see the attached file (menu-level2.png). Sub menu layout also change when hover on that item. See the attached file (menu-level3.png).
To Reproduce
Steps:
1. Create menu item with sub items.
2. Hover on that item which has sub menu item.
Expected Result
Sub menu item should be visible and item should be align properly.
menu-error
menu-level2
menu-level3
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | belle-issue-2977424.png | 629.55 KB | saxenaakansha30 |
| #7 | issue-2977424-2977436.patch | 1.89 KB | saxenaakansha30 |
| #6 | Menu alignment is break.png | 494.03 KB | deepakkumar14 |
| #5 | Menu_alignment_break_2977424_5.patch | 1.84 KB | devshi.addweb |
| menu-level3.png | 188.32 KB | bandanasharma |
Comments
Comment #2
bandanasharma CreditAttribution: bandanasharma as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commentedComment #3
bandanasharma CreditAttribution: bandanasharma as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commentedComment #4
devshi.addweb CreditAttribution: devshi.addweb at AddWeb Solution Pvt. Ltd. commentedComment #5
devshi.addweb CreditAttribution: devshi.addweb at AddWeb Solution Pvt. Ltd. commented@bandanasharma, I review your issue mentioned above, I worked on it & attached the patch for the same. Kindly review my attached patch & let me know.
Comment #6
deepakkumar14 CreditAttribution: deepakkumar14 as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented@hiral.addweb i have reviewed your patch and the alignment issue still not better and I think the left side white space should be reduced as well. The child of sub child menu breaks on screen. See the attached screenshot for the reference.
Comment #7
saxenaakansha30I have worked on the issue and fixed it. Attaching the patch.
I found this issue and issue #2977436 relevant. So the patch I'm attaching would fix both issues together with minimum code modification.
Need reviews.
Comment #8
saxenaakansha30Comment #10
bandanasharma CreditAttribution: bandanasharma as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented@saxenaakansha30@gmail.com Please change the status "Needs Review" to "Fixed".
Comment #11
saxenaakansha30