Make WordPress Core

Opened 2 years ago

Last modified 2 years ago

#44719 new defect (bug)

Hide this update button placement is wrong on update-core.php

Reported by: subrataemfluence Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Upgrade/Install Keywords: needs-patch
Focuses: ui Cc:


The button Hide this update in update-core.php is wrongly placed. There should be a small space after Update Now button.

Please see screenshot

Attachments (4)

screenshot.png (9.3 KB) - added by subrataemfluence 2 years ago.
44719.diff (563 bytes) - added by subrataemfluence 2 years ago.
Proposed patch
with-patch.png (11.8 KB) - added by subrataemfluence 2 years ago.
After applying proposed patch
44719.2.diff (950 bytes) - added by chetan200891 2 years ago.
Created patch to fix space issue.

Download all attachments as: .zip

Change History (8)

2 years ago

Proposed patch

2 years ago

After applying proposed patch

#1 @subrataemfluence
2 years ago

  • Keywords has-patch added; needs-patch removed

#2 @swissspidy
2 years ago

  • Keywords needs-patch added; has-patch removed
  • Version 4.9.8 deleted

I wonder where these styles got lost...

In any case, we shouldn't fix this with inline styles but by putting some proper selectors in a CSS file.

2 years ago

Created patch to fix space issue.

#3 @chetan200891
2 years ago

I have created patch 44719.2.diff to fix space issue.

Last edited 2 years ago by chetan200891 (previous) (diff)

#4 @subrataemfluence
2 years ago

@chetan200891 there are many areas in core where inline styles have been used specially where elements are not being reused anywhere else.

Since Hide Update button is only available on update-core.php and not being used anywhere else I thought its better to have it inline rather than introducing another CSS class.

I may be wrong!

Note: See TracTickets for help on using tickets.