Details

web/utils/tooltip-utils.js
51–52 ↗	(On Diff #16244)	What do you think about having a single structure with `x` and `y` props, so that it's more obvious that these are a part of a single entity?

This revision is now accepted and ready to land.Sep 2 2022, 1:36 PM

• jacek added inline comments.Sep 2 2022, 1:46 PM

web/utils/tooltip-utils.js
51–52 ↗	(On Diff #16244)	Makes sense

address feedback

• jacek requested review of this revision.Sep 2 2022, 2:48 PM

• jacek retitled this revision from [web] Rename `x/yCoord` to `x/yAnchorPoint` in `TooltipPositionStyle` to [web] Rename `x/yCoord` to `AnchorPoint` in `TooltipPositionStyle`.

Harbormaster completed remote builds in B11848: Diff 16253.Sep 2 2022, 2:57 PM

tomek accepted this revision.Sep 5 2022, 8:14 AM

This revision is now accepted and ready to land.Sep 5 2022, 8:14 AM

Closed by commit rCOMMd89c8f499c0c: [web] Rename `x/yCoord` to `AnchorPoint` in `TooltipPositionStyle`. · Explain WhySep 5 2022, 10:13 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMMd89c8f499c0c: [web] Rename `x/yCoord` to `AnchorPoint` in `TooltipPositionStyle`.