#58829 closed defect (bug) (fixed)
Miss alignment of "Copied!" message when clicking "Copy URL to clipboard" after uploading a media item
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.4 | Priority: | normal |
Severity: | normal | Version: | 6.2.2 |
Component: | Upload | Keywords: | has-patch has-testing-info commit |
Focuses: | Cc: |
Description
In the media section, once after uploading a new media file, when clicking "Copy URL to clipboard", the message "Copied!" displayed is misaligned.
Steps to reproduce:
- Select the media library
- Upload new media from "Add New"
- After uploading, select the "Copy URL to clipboard" from the uploaded media file displayed.
- The "Copied!" message displayed is misaligned.
Attachments (9)
Change History (22)
This ticket was mentioned in Slack in #core by sujichandran14. View the logs.
4 months ago
This ticket was mentioned in Slack in #core by nicolefurlan. View the logs.
2 months ago
#8
@
2 months ago
- Keywords has-testing-info needs-testing added
During our bug scrub today, @joemcgill mentioned that we should make sure the proposed CSS change isn't too broad, which could lead to other side-effects.
#9
@
2 months ago
Tested this out today and seems to work fine. I do agree though with @joemcgill that the selector we're changing here is fairly broad and as such, could impact other areas. I did look into when this code was introduced (#51754) and from looking at that, seems fairly safe to modify the CSS as it was introduced as part of the copy to clipboard functionality (and thus I'm assuming it's scoped to that).
All that said, we already have some CSS that is scoped directly to the copy to clipboard container so feels safest to me to make changes there instead.
I've attached a patch above that changes from modifying the .edit-attachment
container and instead modifies the .copy-to-clipboard-container
container. We add display: flex
there, which in essence is the same as what the previous patch was doing (it removed the display: block
property and in doing, it falls back to display: flex
).
I've tested on various breakpoints and on RTL and all seems fine.
This ticket was mentioned in PR #5414 on WordPress/wordpress-develop by @oglekler.
2 months ago
#10
Both suggested fixes in one patch
Trac ticket: https://core.trac.wordpress.org/ticket/58829
@
2 months ago
Thank you, @oglekler ! I've confirmed that both issues have been resolved after applying your patch.
@joedolson commented on PR #5414:
2 months ago
#13
In r56801
I think, we need to fix also this issue on this ticket.
