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

Tooltip: Use AccessibleButton in dialogs #12460

Merged
merged 3 commits into from
Apr 29, 2024

Conversation

florianduros
Copy link
Contributor

@florianduros florianduros commented Apr 29, 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
Following #12443, AccessibleButton includes the new tooltip. We are migrating step by step the usage of the deprecated AccessibleTooltipButton to AccessibleButton.

Update the dialogs.

@florianduros florianduros added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Apr 29, 2024
@florianduros florianduros changed the title Tooltip: update dialogs Tooltip: Use AccessibleButton in dialogs Apr 29, 2024
@florianduros florianduros marked this pull request as ready for review April 29, 2024 13:56
@florianduros florianduros requested a review from a team as a code owner April 29, 2024 13:56
@florianduros florianduros added this pull request to the merge queue Apr 29, 2024
Merged via the queue into develop with commit 9aeaa38 Apr 29, 2024
30 checks passed
@florianduros florianduros deleted the florianduros/tooltip/dialogs branch April 29, 2024 22:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants