Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Tooltip: migrate remaining tooltips from AccessibleTooltipButton to AccessibleButton #12522

Merged
merged 37 commits into from
May 17, 2024

Conversation

florianduros
Copy link
Contributor

@florianduros florianduros commented May 14, 2024

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

Part of element-hq/element-web#26762
Migrate the last usage of AccessibleTooltipButton and delete it.

florianduros and others added 29 commits April 26, 2024 16:20
# Conflicts:
#	src/accessibility/roving/RovingAccessibleTooltipButton.tsx
# Conflicts:
#	test/components/views/spaces/__snapshots__/SpacePanel-test.tsx.snap
@florianduros florianduros added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label May 14, 2024
@florianduros florianduros marked this pull request as ready for review May 15, 2024 16:10
@florianduros florianduros requested a review from a team as a code owner May 15, 2024 16:10
@florianduros florianduros requested review from richvdh and robintown May 15, 2024 16:10
@florianduros
Copy link
Contributor Author

The merge of develop decreased the code coverage check in my PR. I updated the tests to reach it.

@richvdh richvdh changed the title Tooltip: migrate remaining tootltip Tooltip: migrate remaining tooltips May 17, 2024
@richvdh richvdh changed the title Tooltip: migrate remaining tooltips Tooltip: migrate remaining tooltips from AccessibleTooltipButton to AccessibleButton May 17, 2024
@florianduros florianduros enabled auto-merge May 17, 2024 13:50
@florianduros florianduros added this pull request to the merge queue May 17, 2024
Merged via the queue into develop with commit 75562b1 May 17, 2024
30 checks passed
@florianduros florianduros deleted the florianduros/tooltip/misc branch May 17, 2024 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants