patternfly-elements icon indicating copy to clipboard operation
patternfly-elements copied to clipboard

fix(tooltip): make content inert when hidden

Open bennypowers opened this issue 6 months ago • 8 comments

What I did

  1. make tooltip content inert when hidden. Fixes #2902

Testing Instructions

  1. view repro instructions in linked issue

Notes to Reviewers

  1. Pay attention to a11y / SR use

bennypowers avatar May 12 '25 14:05 bennypowers

🦋 Changeset detected

Latest commit: 63375c15a3e2d35f14ba6770168fff1f3fd9aa25

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@patternfly/elements Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar May 12 '25 14:05 changeset-bot[bot]

✅ Commitlint tests passed!

More Info
{
  "valid": true,
  "errors": [],
  "warnings": [],
  "input": "fix(tooltip): make content inert when hidden"
}

github-actions[bot] avatar May 12 '25 14:05 github-actions[bot]

✅ Commitlint tests passed!

More Info
{
  "valid": true,
  "errors": [],
  "warnings": [],
  "input": "fix(tooltip): make content inert when hidden"
}

github-actions[bot] avatar May 12 '25 14:05 github-actions[bot]

Deploy Preview for patternfly-elements ready!

Name Link
Latest commit e1a9f6399a0c72fd9fe9d2a59628c9f0c87253b1
Deploy Preview https://deploy-preview-2908--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar May 12 '25 14:05 netlify[bot]

SSR Test Run for 19509b9d174286c315b4aff2f65cc9ab3130d1f1: Report

github-actions[bot] avatar May 12 '25 14:05 github-actions[bot]

Deploy Preview for patternfly-elements ready!

Name Link
Latest commit 63375c15a3e2d35f14ba6770168fff1f3fd9aa25
Latest deploy log https://app.netlify.com/projects/patternfly-elements/deploys/68851a0170061b000846f660
Deploy Preview https://deploy-preview-2908--patternfly-elements.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar May 12 '25 14:05 netlify[bot]

SSR Test Run for e25a38bc4eb0b94406a1efbda465c4dae31d246f: Report

github-actions[bot] avatar May 12 '25 14:05 github-actions[bot]

I have requested more detailed instructions from Tejas about what words/characters to specifically hightlight and copy/paste.

In my initial review, it appears this PR doesn't solve Tejas's issue. Results appear to be identical in this PR's DP and on pfe.org.

Interestingly, you get different clipboard results when copy/pasting into a text document versus Google Docs. Not sure if this is a PFE or Google Docs thing. 🤷‍♂️

adamjohnson avatar May 14 '25 20:05 adamjohnson

Likely we need a user-select: none; on the tooltip text.

zeroedin avatar Jul 25 '25 16:07 zeroedin

SSR Test Run for e1a9f6399a0c72fd9fe9d2a59628c9f0c87253b1: Report

github-actions[bot] avatar Jul 26 '25 18:07 github-actions[bot]

SSR Test Run for a7b2a5b8cc6e689e1618d96db6a52c7e077b8295: Report

github-actions[bot] avatar Jul 26 '25 18:07 github-actions[bot]