#48859 closed enhancement (fixed)
Plugin and Theme upload form is misaligned
Reported by: | seedsca | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | 5.3 |
Component: | Upload | Keywords: | has-patch has-screenshots commit |
Focuses: | ui, css, administration | Cc: |
Description
When uploading a plugin or theme file, the Install Now button is not correctly aligned and the button drops to the next line if filename is long. Changing the .wp-upload-form
class to display: flex
and max-width: max-content
fixes this.
Also added right padding of 30px to input[type="file"]
The following CSS fixes the issue when added to a browser's dev tools.
.upload-plugin .wp-upload-form, .upload-theme .wp-upload-form { /* max-width: 380px; */ max-width: max-content; display: flex; } .wp-admin input[type="file"] { /* padding: 3px 0; */ padding: 3px 30px 3px 0; }
I couldn't find where to change the CSS since it's being created from a load-styles.php file... Happy to create a pull request if anybody would give me some guidance on where the CSS is to be found :)
Attachments (9)
Change History (24)
#1
@
5 years ago
- Keywords has-patch has-screenshots added
- Type changed from defect (bug) to enhancement
Hi @seedsca,
Welcome to WordPress Trac! Thanks for the ticket.
You can please refer - https://make.wordpress.org/core/handbook/tutorials/trac/submitting-a-patch/ how to submit a patch in WordPress Trac
#2
@
5 years ago
- Milestone changed from Awaiting Review to 5.4
- Summary changed from Plugin and Theme upload form is miss aligned to Plugin and Theme upload form is misaligned
#3
@
5 years ago
- Keywords needs-refresh added
Hi there, thanks for the patches!
Unfortunately, max-content
is not supported by IE11 and Edge browsers, @seedsca.
See another patch below for this issue.
#4
@
5 years ago
- Keywords needs-refresh removed
- Owner set to audrasjb
- Status changed from new to accepted
#5
@
5 years ago
- Milestone changed from 5.4 to 5.3.3
The last patch looks good on my side.
@SergeyBiryukov moving this one to 5.3.3 as it's a regression from branch 5.3 ;)
#6
@
5 years ago
@audrasjb Wouldn't the max-content
be ignored by browsers that don't support it? There by leaving my patch to work pretty much as yours. But when the browsers do support it, they get to show the full width of the file name?
#7
@
5 years ago
- Milestone changed from 5.3.3 to 5.4
Moving all unfixed tickets from 5.3.3 to milestone 5.4, as there is no plan for a 5.3.3 maintenance release for now.
This ticket was mentioned in Slack in #core by david.baumwald. View the logs.
5 years ago
#9
@
5 years ago
Based on https://core.trac.wordpress.org/ticket/48859#comment:6, marking this for feedback.
#11
@
5 years ago
- Keywords commit added; dev-feedback removed
48859.3.diff
still applies cleanly.
This first approach is the safer for now, and handles well the issue raised by this ticket. Another ticket could be opened later to make some improvements when max-content` property will have better browser’s support.
Marking this for commit
.
Install Now button misaligned