Skip to content

Conversation

@jcain82
Copy link
Contributor

@jcain82 jcain82 commented Oct 21, 2025

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation
  • Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)
https://hyland.atlassian.net/browse/AAE-38748

What is the new behaviour?
Instead of inserting an element in the container with the parent element we now show a matTooltip
image

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@jcain82
Copy link
Contributor Author

jcain82 commented Oct 21, 2025

@eromano, I made the switch to a floating tooltip but there are some accessibility issues with doing it this way and because we set title elements as you can see from the image there can be overlap. The way around this would be to instead update the title element to append the "Click to Copy" text or to abandon using unnecessary title elements since screen readers already know how to read tabular data from a grid so setting the title to the cell value doesn't make any sense.

@jcain82 jcain82 force-pushed the feature/AAE-38748-convert-clipboard-directive-tooltip branch 2 times, most recently from 43007fa to 2795386 Compare October 30, 2025 13:21
@jcain82 jcain82 force-pushed the feature/AAE-38748-convert-clipboard-directive-tooltip branch from 2795386 to d8d19c5 Compare November 3, 2025 16:16
@jcain82 jcain82 force-pushed the feature/AAE-38748-convert-clipboard-directive-tooltip branch from d8d19c5 to 76b6d61 Compare November 5, 2025 21:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

@jcain82 jcain82 merged commit e28e924 into develop Nov 6, 2025
18 checks passed
@jcain82 jcain82 deleted the feature/AAE-38748-convert-clipboard-directive-tooltip branch November 6, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants