HomePhabricator
Diffusion Comm a1f7c72f4e9e

[web] introduce calculateReactionTooltipSize function

Tags
None
Referenced Files
F694584: Screenshot 2023-08-17 at 9.18.00 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694582: Screenshot 2023-08-17 at 9.17.18 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694585: Screenshot 2023-08-17 at 9.19.18 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694586: Screenshot 2023-08-17 at 9.18.52 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694583: Screenshot 2023-08-17 at 9.18.33 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694587: Screenshot 2023-08-17 at 9.18.17 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
F694576: Screenshot 2023-08-17 at 9.22.20 PM.png
Aug 27 2023, 10:04 PM
File Not Attached
Subscribers
None

Description

[web] introduce calculateReactionTooltipSize function

Summary:
This diff introduces a function to calculate the size of the reaction tooltip. In order to build a tooltip, we need to first know the size, so this function will handles that for our new reaction tooltip.

Here is a screenshot of the reaction for context:

Screenshot 2023-08-17 at 9.22.20 PM.png (482×486 px, 45 KB)

Depends on D8858

Test Plan:
logged out the values of width and height and confirmed that they matched the size of the tooltip that was being rendered

Screenshot 2023-08-17 at 9.18.17 PM.png (616×726 px, 84 KB)

Screenshot 2023-08-17 at 9.18.33 PM.png (170×3 px, 39 KB)

Screenshot 2023-08-17 at 9.18.52 PM.png (402×806 px, 56 KB)

Screenshot 2023-08-17 at 9.19.18 PM.png (152×3 px, 42 KB)

Screenshot 2023-08-17 at 9.17.18 PM.png (316×796 px, 39 KB)

Screenshot 2023-08-17 at 9.18.00 PM.png (148×2 px, 42 KB)

Reviewers: atul, kamil

Reviewed By: kamil

Subscribers: ashoat, tomek

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

Details

Provenance
ginsuAuthored on Aug 16 2023, 3:30 PM
Reviewer
kamil
Differential Revision
D8857: [web] introduce calculateReactionTooltipSize function
Parents
rCOMM2be07376adff: [web] rename calculateToolTipSize to calculateMessageTooltipSize
Branches
Unknown
Tags
Unknown