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

Tooltip: Use AccessibleButton in reusable elements #12461

Merged
merged 5 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 reusable elements in the views.

@florianduros florianduros added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Apr 29, 2024
@florianduros florianduros changed the title Tooltip: Update reusable elements Tooltip: Use AccessibleButton in reusable elements Apr 29, 2024
@florianduros florianduros force-pushed the florianduros/tooltip/views-elements branch from 3c338c3 to b16b717 Compare April 29, 2024 14:05
@florianduros florianduros marked this pull request as ready for review April 29, 2024 14:37
@florianduros florianduros requested a review from a team as a code owner April 29, 2024 14:37
@florianduros florianduros requested review from richvdh and t3chguy April 29, 2024 14:37
src/components/views/elements/SSOButtons.tsx Outdated Show resolved Hide resolved
@florianduros florianduros added this pull request to the merge queue Apr 29, 2024
Merged via the queue into develop with commit 44e2a6d Apr 29, 2024
30 checks passed
@florianduros florianduros deleted the florianduros/tooltip/views-elements branch April 29, 2024 17:40
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