Opened 2 years ago
Closed 2 years ago
#56123 closed defect (bug) (fixed)
Plugin Update Message slightly moved out of place
Reported by: | Presskopp | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | minor | Version: | 5.0 |
Component: | Upgrade/Install | Keywords: | assigned-for-commit has-screenshots good-first-bug has-patch has-testing-info commit |
Focuses: | ui, css, administration | Cc: |
Description
Is it only me or is the text shown a bit too low in the plugin update messages?
Attachments (3)
Change History (12)
#1
@
2 years ago
- Component changed from General to Upgrade/Install
- Focuses ui css administration added
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
#2
follow-up:
↓ 3
@
2 years ago
This seems to relate to the vertical alignment of the icon. Toggling vertical-align:bottom
off so it falls back to top
—within the browser—fixes it at some screen widths and has no effect at other widths (not changing at CSS breakpoints).
#3
in reply to:
↑ 2
@
2 years ago
Replying to sabernhardt:
This seems to relate to the vertical alignment of the icon. Toggling
vertical-align:bottom
off so it falls back totop
—within the browser—fixes it at some screen widths and has no effect at other widths (not changing at CSS breakpoints).
To me deleting the vertical-align:bottom
looks like the solution for any screen size.
#4
@
2 years ago
- Keywords good-first-bug added
This looks like a good-first-bug
candidate.
The file in question: src/wp-admin/css/common.css (Line ~1577)
vertical-align: bottom;
should be removed from here and instead added with just the .import-php .updating-message:before
selector, unless the icon is misaligned in that element too.
#5
@
2 years ago
- Keywords has-patch added; needs-patch removed
- Version set to trunk
I've added a patch here with the following changes, which seems to resolve the alignment issue at all browser widths.
Changes were made to src https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-admin/css/common.css#L1577 as outlined by @costdev
.wrap .notice p:before, .import-php .updating-message:before { margin-right: 6px; vertical-align: bottom; }
to
.wrap .notice p:before, .import-php .updating-message:before { margin-right: 6px; } .import-php .updating-message:before { vertical-align: bottom; }
#6
@
2 years ago
- Keywords has-testing-info commit added
- Milestone changed from Future Release to 6.1
- Version changed from trunk to 5.0
Test Report
Patch tested: 56123.diff
Steps to Reproduce/Test
- Install an old version of a plugin.
- Navigate to
Appearance > Plugins
. - 🐞 The icon and text in the update notice are not aligned with each other.
- Apply 56123.diff.
- Run
npm run build:dev
. - Navigate to
Appearance > Plugins
. Perform a hard refresh. - 🎉 The icon and text in the update notice are aligned with each other.
Environment
- Server: Apache (Linux)
- WordPress: 6.1-alpha-53344-src
- Browser: Chrome 103.0.0.0
- OS: Windows 10
- Theme: Twenty Twenty-Two
- Language: LTR - English (US) and RTL - Arabic
- Plugins: None activated.
Actual Results
- 🐞 The icon and text are not aligned with each other.
- 🎉 The icon and text are aligned with each other.
- ✅ Issue resolved with patch.
Notes
- This appears to have existed since the update notices were added in 5.0. Changing
Version
to 5.0. - I attempted to check if
.import-php .updating-message::before
had the same issue, but can't seem to find any core reference toimport-php
except in the CSS file mentioned in this ticket. - Marking for
commit
consideration.
icon is aligned to the bottom instead of top