978048 - Customization mode outlines don't appear when moving the mouse over an empty space

Status: VERIFIED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

I'm seeing this issue on Windows 8.1 and Windows XP:

When moving the mouse over an empty part of the tab strip, I don't see an outline.
When moving the mouse over an icon in the tab strip (or the tabs themselves), the outline is there.
When dragging an icon to the tab strip, the outline is visible, regardless of the distinction above.

Comment 1

[:Gijs (he/him)]

Over in bug 978091, Tim said he can't reproduce this on a clean profile. Can you? :-)

Comment 2

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

(In reply to :Gijs Kruitbosch from comment #1)
> Over in bug 978091, Tim said he can't reproduce this on a clean profile. Can
> you? :-)

Yes, it also happens in a brand new profile on both Windows 8.1 and XP

Comment 3

[:Gijs (he/him)]

Attachment: track mouse on tabstoolbar and menubar in customize mode to get outlines to show up,

So the window dragging utils is canceling mozmousehittest and I suspect that that's causing this kind of stuff to happen. Which is interesting. In any case, I'm just adding/removing this attribute thing and then maybe this will work. Right? Testing this on Windows in a bit.

Comment 4

[:Gijs (he/him)]

Attachment: Patch

That didn't work, but this does, surprisingly. :-)

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8387838 [details] [diff] [review]
Patch

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

Felipe, can you review this? I think you're more familiar with MozMouseHitTest than I am.

Comment 6

[:Gijs (he/him)]

Attachment: Remove drag area highlighting

This is sad, but I think this is the best fix considering timeline.

Comment 7

[:Felipe Gomes (needinfo for replies!)]

On IRC we talked about this, the gist is that due to how the Aero API from Windows works, to activate Aero dragging in the titlebar parts of the UI we need to forfeit receiving normal mouse events on those areas.
Some possible solutions, which none are perfect, are:

 - opt out of Aero dragging inside the outline in customize mode, and no dragging would occur
 - opt out of Aero dragging inside the outline in customize mode, and implement manual JS dragging (by tracking mousedown/mousemoves and calling window.moveTo)
 - Hack around by applying the outline styling using JS instead of :hover. I tried something like that, and it works kinda well, but there's no way to make it reliable on the edge of the window (i.e. when the mouse quickly moves from the toolbar to outside of the window)
 - Come up with some workaround in widget code, which I think could be doable but not upliftable for beta

With all these sucky options, Gijs opted for removing the outline for now. FWIW, if [1] would be acceptable it would be an easy fix.

Comment 8

[:Gijs (he/him)]

(In reply to :Felipe Gomes from comment #7)
>  - opt out of Aero dragging inside the outline in customize mode, and no
> dragging would occur
>  - opt out of Aero dragging inside the outline in customize mode, and
> implement manual JS dragging (by tracking mousedown/mousemoves and calling
> window.moveTo)

> With all these sucky options, Gijs opted for removing the outline for now.
> FWIW, if [1] would be acceptable it would be an easy fix.

Don't know which of these you meant, but the problem is also that there are still more bugs specifically with these elements, in particular bug 979630. So I think that outright removing them is a lot simpler than trying to hack around the multitude of issues. :-(

Comment 9

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8387892 [details] [diff] [review]
Remove drag area highlighting

Philipp, are you ok with removing the outline?

Comment 10

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

Comment on attachment 8387892 [details] [diff] [review]
Remove drag area highlighting

Looks good!

Comment 11

[:Gijs (he/him)]

remote:   https://hg.mozilla.org/integration/fx-team/rev/997a18bcc2d6

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/997a18bcc2d6

Comment 13

[:Gijs (he/him)]

Comment on attachment 8387892 [details] [diff] [review]
Remove drag area highlighting

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Australis / bug 963576
User impact if declined: weird behaviour of tabstrip/menubar outline
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): none
String or IDL/UUID changes made by this patch: none

Comment 14

[:Gijs (he/him)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/5885c5c71e10

Comment 15

[Matthew N. [:MattN]]

Comment on attachment 8387892 [details] [diff] [review]
Remove drag area highlighting

As a said in-person, we should have added new rules to hide the pseudo elements "content: none;" in the cases where this is a problem. This would have also avoided making the selectors more complex. We could have left the outlines as they were on OS X AFAICT.

Comment 16

[Ioana (away)]

Unfortunately, qa can't reproduce this issue on builds with this bug (and no fix), so we can't really verify it reliably either.

Philipp, can you please check if this is fixed for you in Firefox 29?

Comment 17

[(Currently slow to respond) Philipp Sackl [:phlsa] (Firefox UX) please use needinfo]

(In reply to Ioana Budnar, QA [:ioana] from comment #16)
> Unfortunately, qa can't reproduce this issue on builds with this bug (and no
> fix), so we can't really verify it reliably either.
> 
> Philipp, can you please check if this is fixed for you in Firefox 29?

Yes, 29 uses the intended behavior.