#42545 closed defect (bug) (fixed)
CSS-Grid on about.php needs polyfill for Internet Explorer
Reported by: | Clorith | Owned by: | ryelle |
---|---|---|---|
Milestone: | 4.9 | Priority: | normal |
Severity: | normal | Version: | 4.9 |
Component: | Help/About | Keywords: | has-patch dev-reviewed commit |
Focuses: | ui | Cc: |
Description
The about.php page for 4.9 uses CSS-grid, which is amazing, but unfortunately not available in IE 11 and below.
We still support IE 11, so we need to find a polyfill for this.
Attachments (5)
Change History (20)
#1
@
7 years ago
- Component changed from Administration to Help/About
- Focuses ui added; administration removed
- Milestone changed from Awaiting Review to 4.9
- Version changed from trunk to 4.9
#4
@
7 years ago
- Keywords has-patch added; needs-patch removed
As @melchoyce mentioned in slack, the issues here are more to do with the combo of the JS for docking the titles + grid compatibility issues. Technically grid is supported in IE11, so we're keeping the grid for the layout of header + section, but switching to a flex layout for the items themselves. This, plus some updates to the title-length-detection javascript seems to fix the layout issues in IE11 and Edge (by my testing on browserstack) - it would be great to confirm that with "real" use of IE :)
See 42545.diff for the patch. I also ran grunt postcss:core
to make sure the prefixes were in place.
#5
@
7 years ago
Looks good in IE11 on native Windows, I'm unable to test in the most widespread version of Edge, as I am on the Fall Creators version which didn't have issues originally.
This ticket was mentioned in Slack in #core by clorith. View the logs.
7 years ago
#7
@
7 years ago
Tested with Edge, titles don't go above images on smaller viewports, they instead overlay the section images, but the bigger issues from the initial screenshot are not there now on wider viewports.
#8
@
7 years ago
Thanks for all your testing @Clorith. Do you know at what browser width the titles start to overlap?
#9
@
7 years ago
It's when the 782px breakpoint kicks in and tries to lay the titles above the section
#10
@
7 years ago
42545.2.diff should take care of the overlapping headlines on the images.
#11
@
7 years ago
From 904px and down to the breakpoint at 782 I'm seeing text overlapping left to right from the headers (see screenshot above), but the headers above are good now!
#12
@
7 years ago
Chatted with @Clorith in Slack (https://wordpress.slack.com/archives/C02RQBWTW/p1510695911000002); we agree that the overlap in this specific size is pretty minor overall, so we're going to go ahead with the latest patch.
Can I get one more committer +1?
#13
@
7 years ago
- Keywords dev-reviewed commit added
Tested 42545.2.diff with Microsoft Edge 38.14393.1066.0 / Microsoft EdgeHTML 14.14393.
Reproduced the same overlap as in 42545-edge.2.PNG, looks good to me otherwise.
@ryelle is looking at this.