element-web icon indicating copy to clipboard operation
element-web copied to clipboard

Use Compound tooltip component for all tooltips in EW

Open langleyd opened this issue 1 year ago • 1 comments

Your use case

What would you like to do?

Replace all existing tooltip occurrences with the tooltip in Compound

Why would you like to do it?

The existing tooltip is is accessible and the compound tooltip is being made accessible.

How would you like to achieve it?

Replace tooltip occurrences 1 by 1, improving the compound tooltip or adapting the site of it's usage with support from design/product, as necessary.

### Tasks
- [x] https://github.com/element-hq/compound-web/pull/156
- [x] https://github.com/matrix-org/matrix-react-sdk/pull/12416
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12443
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12456
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12455
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12458
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12459
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12460
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12461
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12462
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12487
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12489
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12490
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12493
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12497
- [ ] https://github.com/element-hq/compound-web/pull/165
- [ ] https://github.com/matrix-org/matrix-react-sdk/pull/12448 (will be replaced by smaller ones)
- [ ] Use kbd dom element for shortcut in tooltip
- [ ] Migrate `RovingAccessibleTooltipButton` to `RovingAccessibleButton`
- [ ] Replace `TooltipTarget` by compound tooltip
- [ ] Replace `Tooltip` usage by compound tooltip - Keeping for form field, deprecate old `Tooltip` 

Have you considered any alternatives?

No response

Additional context

No response

langleyd avatar Dec 14 '23 13:12 langleyd

@nadonomy This is the issue we're discussing with @t3chguy

gaelledel avatar Dec 20 '23 10:12 gaelledel