Make WordPress Core

Opened 4 months ago

Last modified 6 weeks ago

#60082 new defect (bug)

"Copied" tooltip should be hidden when another attachment URL is copied (in list mode)

Reported by: jayadevankbh's profile jayadevankbh Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 6.0
Component: Media Keywords: needs-patch
Focuses: ui, javascript Cc:

Description

In the media list view, clicking the "Copy URL" action link displays previously clicked "Copy URL". The previously clicked row showing "Copied!" when hover the media again.

Environment

  • WordPress: 6.4.2
  • PHP: 8.2.0
  • Server: Apache/2.4.54 (Unix) OpenSSL/1.0.2u PHP/8.2.0 mod_wsgi/3.5 Python/2.7.18 mod_fastcgi/mod_fastcgi-SNAP-0910052141 mod_perl/2.0.11 Perl/v5.30.1
  • Database: mysqli (Server: 5.7.39 / Client: 8.2.0)
  • Browser: Firefox 119.0 (macOS)
  • Theme: Twenty Twenty-Four 1.0
  • MU-Plugins: None activated
  • Plugins:
    • WordPress Beta Tester 3.5.5

Steps to Reproduce

  1. Go to media library
  2. Click "Copy URL" of one of the media.
  3. Click "Copy URL" of another media.
  4. The previously clicked row showing "Copied!" when hover the media again.

Expected Results

  1. The previously "Copied!" tooltip should be close after clicking the next one.

Actual Results

  1. The previously clicked row showing "Copied!" when hover the media again.

Change History (4)

#2 @sabernhardt
4 months ago

  • Focuses javascript added
  • Keywords needs-patch added
  • Severity changed from critical to normal
  • Summary changed from Media: "Copied" tooltip should be closed to "Copied" tooltip should be hidden when another attachment URL is copied (in list mode)
  • Version set to 6.0

Thanks for the report!

The row action was added in #54426.

Clicking a second attachment's "Copy URL to clipboard" button less than 3 seconds after the clicking the first allows the success message to persist indefinitely in the first attachment's row.

Maybe the script should restore the hidden class on all of the .success elements whenever another copy button is activated.

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


6 weeks ago

#4 @joedolson
6 weeks ago

  • Focuses accessibility removed
Note: See TracTickets for help on using tickets.