Opened 3 years ago
Last modified 3 years ago
#54217 new defect (bug)
Menu Screen has some UI inconsistency issue
Reported by: | juhise | Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | 5.8 |
Component: | Menus | Keywords: | has-patch has-screenshots |
Focuses: | ui, css, administration | Cc: |
Description (last modified by )
Bulk Select button on menu screen on top and bottom doesn't have the equal margin from top and bottom, Refer Screenshot bulk-select-margin.png
After applying the patch, the screen looks like this, refer bulk-select-after-fix.png
Made sure it doesn't break anything else, after applying the fix for it, menu items margin got disturbed so applied fix for that as well, refer menu-item-margin.png
Add menu items from the column on the left.
padding of this also got disturbed so fixed this by adding
.nav-menus-php #post-body-content .post-body-plain { margin-bottom: 0; padding: 0.1em 0; }
Refer Screenshots: menu-item-colum-left.png
& menu-item-colum-left-after-fix.png
Attachments (11)
Change History (16)
This ticket was mentioned in PR #1730 on WordPress/wordpress-develop by juhi123.
3 years ago
#1
Related Core Trac Ticket: https://core.trac.wordpress.org/ticket/54217
#2
@
3 years ago
- Component changed from General to Menus
- Description modified (diff)
- Focuses administration added
- Version changed from trunk to 5.8
#3
@
3 years ago
Hi there!
Thanks for the ticket and patch.
Can you please adjust the top space for Bulk select when there is a menu item in the menu?
#4
@
3 years ago
Hey @mukesh27
Thanks for sharing this issue, I have fixed this, hope it doesn't create any other issue.
#5
@
3 years ago
Added one more fixhttps://github.com/WordPress/wordpress-develop/pull/1730/commits/307b2094480d1cb0da3ad0b11a19c45abdb07d9f, for new menu menu when there was no title Give your menu a name, then click Create Menu.
message doesn't have proper spacing. Even in 5.8 it didn't have consistent spacing from top and bottom
Attaching few screenshots new-menu-with-no-title-5.8.png
new-menu-with-no-title-after-current-change.png
new-menu-with-no-title-after-fix.png
Initital Issue of Bulk Select margin