HomePhabricator
Diffusion Comm acd9728a0f6a

[web] Align `MessageTimestampTooltip` under `MessageTooltip`

Tags
None
Referenced Files
F123487: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123486: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123485: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123484: Screen Recording 2022-08-01 at 4.01.55 PM.mov
Aug 4 2022, 10:45 AM
File Not Attached
F123483: Screen Recording 2022-08-01 at 4.01.17 PM.mov
Aug 4 2022, 10:45 AM
File Not Attached
Subscribers
None

Description

[web] Align MessageTimestampTooltip under MessageTooltip

Summary:
Part of a stack of diffs to fix the timestamp bug, related Linear issue here.

This diff moves the timestamp tooltip below the message tooltip (with the reply and create thread buttons). Handling other front-end changes like color will be handled in subsequent diffs in this stack, this diff is just focused on aligning the tooltip correctly.

Depends on D4481.

Test Plan:
Tested on Chrome/Safari and looks as expected.
Chrome:

Safari:

Additional stills:

image.png (240×578 px, 19 KB)

image.png (172×396 px, 16 KB)

image.png (350×1 px, 68 KB)

Reviewers: atul, jacek, tomek

Reviewed By: atul

Subscribers: ashoat, adrian

Differential Revision: https://phab.comm.dev/D4704

Details

Provenance
aboshAuthored on Aug 1 2022, 12:55 PM
Reviewer
atul
Differential Revision
D4704: [web] Align `MessageTimestampTooltip` under `MessageTooltip`
Parents
rCOMM2c7dbf3314ec: [web] Reintroduce `MessageTimestampTooltip` in `ChatMessageList`
Branches
Unknown
Tags
Unknown