Closed (fixed)
Project:
Belle
Version:
8.x-2.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
5 Jun 2018 at 06:54 UTC
Updated:
26 Jun 2018 at 04:59 UTC
Jump to comment: Most recent, Most recent file
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 commentedComment #3
bandanasharma commentedComment #4
devshi.addweb commentedComment #5
devshi.addweb 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 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 commented@saxenaakansha30@gmail.com Please change the status "Needs Review" to "Fixed".
Comment #11
saxenaakansha30