Opened 5 years ago
Closed 5 years ago
#49375 closed defect (bug) (fixed)
Menu accordion content on small screen doesn't match widths with the accordion handle
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Menus | Keywords: | has-screenshots has-patch commit |
Focuses: | ui, css, administration | Cc: |
Description
While on half-screen on a laptop I noticed that menu items added to the menu have their accordion display slightly off. Their accordion content is set to a fixed width and it doesn't span the same width as the accordion heading/handle. *Screenshot uploading
Attachments (7)
Change History (14)
#1
@
5 years ago
- Keywords needs-patch has-screenshots added
- Milestone changed from Awaiting Review to 5.6
- Owner set to audrasjb
- Status changed from new to accepted
Hi Garrett,
I was able to reproduce the issue. Moving to milestone 5.6.
Thanks!
#2
@
5 years ago
- Keywords has-patch added; needs-patch removed
49375.diff
is a first approach, based on fixed max-width
for the two containers.
@
5 years ago
Refresh to make the description-[wide/thin] percent based to avoid overflow and ensure consistent space on the right.
#3
@
5 years ago
- Keywords needs-testing added
Thanks for the patch @audrasjb testing I found the inputs spilled beyond the bounds. I initially was going to use max-width there as well but found it left inconsistent padding on the right depending if it's percent or px based that's in effect. Along with those changes I fixed some minor css coding standards providing a space between the selector and the opening {
brace.
#4
@
5 years ago
- Keywords commit added; needs-testing removed
@garrett-eclipse your last patch looks perfect to me! Thanks!
Marking this for commit
as it's now tested against each endpoint 👊
(see video above, but I also tested on a real tablet to get intermediate views)
Screenshot illustrating the issue