HomePhabricator
Diffusion Comm 33347778d029

[web] Finish styling `MessageTimestampTooltip`

Tags
None
Referenced Files
F124488: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123663: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123662: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123661: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123660: image.png
Aug 4 2022, 10:45 AM
File Not Attached
F123659: Screen Recording 2022-08-01 at 4.49.49 PM.mov
Aug 4 2022, 10:45 AM
File Not Attached
F123658: Screen Recording 2022-08-01 at 4.48.55 PM.mov
Aug 4 2022, 10:45 AM
File Not Attached
Subscribers

Description

[web] Finish styling MessageTimestampTooltip

Summary:
Relevant Linear issue here.

This diff adds styling changes to MessageTimestampTooltip, specifically changing the color of the tooltip, increasing the width of the tooltip (so it is at minimum the width of the MessageTooltip with the reply/create thread buttons), some padding, and makes the border radius match the MessageTooltip.

Depends on D4704.

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

Safari:

Additional stills:

image.png (312×702 px, 24 KB)

image.png (180×380 px, 16 KB)

image.png (250×1 px, 45 KB)

image.png (508×1 px, 128 KB)

Finally, the timestamp does not appear behind photos, as @ashoat noted on Linear:

image.png (1×2 px, 4 MB)

Reviewers: atul, jacek, tomek

Reviewed By: atul, jacek

Subscribers: ashoat, adrian

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

Details

Provenance
aboshAuthored on Aug 1 2022, 1:45 PM
Reviewer
atul
Differential Revision
D4705: [web] Finish styling `MessageTimestampTooltip`
Parents
rCOMMacd9728a0f6a: [web] Align `MessageTimestampTooltip` under `MessageTooltip`
Branches
Unknown
Tags
Unknown