#56123 closed defect (bug) (fixed)
Plugin Update Message slightly moved out of place
| Reported by: | Presskopp | Owned by: | audrasjb |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.1 |
| Component: | Upgrade/Install | Version: | 5.0 |
| Severity: | minor | Keywords: | assigned-for-commit has-screenshots good-first-bug has-patch has-test-info commit |
| Cc: | Focuses: | ui, css, administration |
Description
Is it only me or is the text shown a bit too low in the plugin update messages?
Attachments (3)
Change History (13)
#1
@
4 years ago
- Component General → Upgrade/Install
- Focuses ui css administration added
- Keywords needs-patch added
- Milestone Awaiting Review → Future Release
#2
follow-up:
↓ 3
@
4 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
@
4 years ago
Replying to sabernhardt:
This seems to relate to the vertical alignment of the icon. Toggling
vertical-align:bottomoff 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
@
4 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
@
4 years ago
- Keywords has-patch added; needs-patch removed
- Version → 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
@
4 years ago
- Keywords has-testing-info commit added
- Milestone Future Release → 6.1
- Version trunk → 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
Versionto 5.0. - I attempted to check if
.import-php .updating-message::beforehad the same issue, but can't seem to find any core reference toimport-phpexcept in the CSS file mentioned in this ticket. - Marking for
commitconsideration.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
icon is aligned to the bottom instead of top