HomePhabricator
Diffusion Comm dc5e77bf2e0a

[web] Replace `TooltipButton` with `TooltipTextItem` in `MessageActionButton`

Description

[web] Replace TooltipButton with TooltipTextItem in MessageActionButton

Summary:
Context: https://linear.app/comm/issue/ENG-794#comment-a3728358

In the past, there was an ellipsis button that when clicked would reveal a menu with possible actions ("Create sidebar", "Reply"). In the future, the icons for each action will be displayed in a row. But, we still want to display an informative "alt text" tooltip when the icons are hovered over. In this diff we're making the tooltip content static and triggering it onMouseEnter instead of onClick

Test Plan:
Before: https://blob.sh/atul/messageActionButton_before.mov
After: https://blob.sh/atul/messageActionButton_after.mov

Reviewers: varun, palys-swm, def-au1t, benschac, ashoat

Reviewed By: palys-swm, ashoat

Subscribers: ashoat, Adrian, karol-bisztyga

Differential Revision: https://phabricator.ashoat.com/D3302

Details

Provenance
atulAuthored on Feb 25 2022, 9:32 AM
Reviewer
tomek
Differential Revision
D3302: [web] Replace `TooltipButton` with `TooltipTextItem` in `MessageActionButton`
Parents
rCOMM66ec123390b9: [web] Remove default stroke for `wrench` icon
Branches
Unknown
Tags
Unknown