948807 - UITour: Fix highlight position and clipping when a dimension is smaller than the minimum

Status: RESOLVED FIXED

(Firefox :: General, defect, P1)

Description

[Matthew N. [:MattN]]

Attachment: Screen Shot of customize highlight

This fixes two problems related to the min-height/width:
A) The min-height of the html:div isn't taken into effect for the panel's instrinsic size. => Switch to <xul:box>.
** There was a reason why I didn't switch to a XUL box before but I couldn't remember or find notes on why. It seems to be working fine now.

B) When the target of a highlight is smaller than the minimum size (32x32px), the highlight won't be centered because the "overlap" position for panels anchors from the top-left which isn't right when the highlight is larger than the target. In this case, I offset the panel by half of the difference between it's minimum size and the target size (only when the highlight is smaller than the minimum).

Comment 1

[Matthew N. [:MattN]]

Attachment: v.1 Fix A and B

Comment 2

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Comment on attachment 8345705 [details] [diff] [review]
v.1 Fix A and B

Review of attachment 8345705 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/modules/UITour.jsm
@@ +465,4 @@
>           for the wobble animation */
> +      let highlightStyle = aTargetEl.ownerDocument.defaultView.getComputedStyle(highlighter);
> +      let offsetX = -4 - (Math.max(0, parseFloat(highlightStyle.minWidth) - targetRect.width) / 2);
> +      let offsetY = -4 - (Math.max(0, parseFloat(highlightStyle.minHeight) - targetRect.height) / 2);

Nit: The -4 should be a const.

Comment 3

[Matthew N. [:MattN]]

Attachment: v1.1 Calculate -4 with getComputedStyle [r=Unfocused]

(In reply to Blair McBride [:Unfocused] from comment #2)
> Nit: The -4 should be a const.

As discussed and reviewed on IRC, I made it calculate live with getComputedStyle like the other changes.

Comment 4

[Matthew N. [:MattN]]

https://hg.mozilla.org/integration/fx-team/rev/38a4aa93f793

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/38a4aa93f793