Opened 5 years ago
Closed 5 years ago
#47536 closed defect (bug) (fixed)
More actions not aligned correctly
Reported by: | Presskopp | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.3 | Priority: | normal |
Severity: | minor | Version: | 5.3 |
Component: | Administration | Keywords: | needs-patch has-screenshots dev-feedback |
Focuses: | Cc: |
Description
"Manage ..." has a leading space, see screenshot
Attachments (1)
Change History (6)
Note: See
TracTickets for help on using
tickets.
This is a product of the unique HTML structure of that specific line. In the first list element, a
<div class="welcome-icon welcome-widgets-menus">
wraps text containing multiple<a>
tags. Therefore, the icon pseudo-element::before
for that row is applied to the div.Conversely, the other lines on the
Next Steps
andMore Actions
sections contain only onea
, and in those cases, the::before
is applied to that element.Here's the relevant section from wp-admin/includes/dashboard.php, starting on line 1716:
Coding Standards dictate that opening/closing PHP tags shall appear on their own lines. With CSS adding the
::before
pseudo-element for the icon to<div class="welcome-icon welcome-widgets-menus">
, the whitespace created between lines 1717 and 1718 is rendered.Line 1726 and 1728 have PHP rendering singular
<a>
tags and there's no whitespace because Coding Standards allow for single-line mixtures of HTML and PHP.Hopefully this breaks down the root cause of the issue. This first appeared from [42343] stemming from ticket #41057.