Opened 5 years ago
Closed 4 years ago
#49245 closed defect (bug) (fixed)
Menu admin screens: button issue on small screens
Reported by: | passoniate | Owned by: | sabernhardt |
---|---|---|---|
Milestone: | 5.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Menus | Keywords: | has-screenshots has-patch commit |
Focuses: | ui, css | Cc: |
Description
Menu admin screens: button issue on small screens
Attachments (13)
Change History (28)
#2
@
5 years ago
- Keywords has-patch added
- Version 5.3.2 deleted
@passoniate I can't replicate the checkbox issue in windows 10 responsive design.
#3
@
5 years ago
The focus outline on the checkboxes is cut-off on the left because one of the ancestor elements (.menu-settings-group
) has overflow: hidden
.
This has always been the case in all previous WP versions. If used to contain floated elements, overflow: hidden
should be replaced with less invasive methods e.g. by using the CSS class wp-clearfix
(needs investigation).
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
5 years ago
#5
@
5 years ago
- Focuses accessibility administration removed
This ticket was discussed during today's accessibility bug-scrub: not sure this issue relates to web accessibility. Seems more related to UI consistency / design. Updating the ticket properties accordingly.
This ticket was mentioned in Slack in #core-css by sabernhardt. View the logs.
4 years ago
#7
@
4 years ago
- Keywords needs-testing added
- Milestone changed from Awaiting Review to Future Release
With Windows browsers in English, I don't see any need to hide the overflow, even on larger screens. 49245.show-overflow-always.patch simply removes overflow: hidden
from the container.
If there is a problem with floating, however, floats are removed for mobile screens anyway. 49245.show-overflow-on-mobile.patch continues to hide the overflow at larger sizes and shows it for smaller sizes.
#8
@
4 years ago
- Milestone changed from Future Release to 5.7
- Owner set to sabernhardt
- Status changed from new to assigned
This ticket was mentioned in Slack in #core-css by sabernhardt. View the logs.
4 years ago
#11
@
4 years ago
- Keywords commit added; needs-testing removed
49245.show-overflow-on-mobile.patch
for @sabernhardt still applies cleanly and fixes the issue (see above screenshots). Marking this for commit
.
@
4 years ago
before 49245.show-overflow-on-mobile.patch: checkbox focus cutoff on left, Firefox also cuts off top border
@
4 years ago
after 49245.show-overflow-on-mobile.patch: checkbox focus outline and borders in Firefox
#12
@
4 years ago
Screenshots added for the checkbox portion of the patch.
By not hiding the overflow, this would also fix the similar Firefox border issue reported on #51706.
A patch that fixed button alignment.