1287074 - Icons in devtools don't loose hover effect until tooltip appears

Status: VERIFIED DUPLICATE of bug 1287122

(Core :: Layout, defect, P3)

Description

[arni2033]

>>>   My Info:   Win7_64, Nightly 50, 32bit, ID 20160714030208 (2016-07-14)
STR:
1. Open debugger (Ctrl+Shift+S)
2. Hover some buttons in devtools that have icons
  (you should move mouse fast enough so that tooltip didn't appear)
3. Hover mouse over some button in devtools and wait until tooltip appears

AR:  All buttons hovered in Step 2 keep "hovered" look until tooltip appears in Step 3
ER:  All buttons from Step 2 should immediately loose hover state when mouse is moved outside

This is regression from bug 1283827. Regression range:
> https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=067521ccd17907843e1db644ca0f02a1f5a6520a&tochange=eac0c056235ee1174fc092fe159d2d5710331fbd

Comment 1

[Andrew Overholt [:overholt]]

I don't think I can reproduce anymore (or I'm just doing it wrong). Can you?

Comment 2

[arni2033]

I can still reproduce the bug with the old styling.
Devtools developers have made a workaround for this in pushlog [1], but the underlying bug isn't fixed
Meanwhile, my bug (this bug) was ignored AGAIN, and somebody filed duplicate of this (bug 1287122), so I think it makes no sense to create a testcase.

> [1] https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=664bd6aa11e1c9e53f5e6e7bca4990265b563802&tochange=7ba94e0c5daa48d7667519c930eca31d98648512

Comment 3

[Andrew Overholt [:overholt]]

(In reply to arni2033 from comment #2)
> I can still reproduce the bug with the old styling.
> 
> [...]
> 
> Devtools developers have made a workaround for this in pushlog [1], but the
> underlying bug isn't fixed

Oh, you mean it's fixed with the dark theme but will "re-appear" once 50 goes to beta? Is that what you mean by "old styling" above?

Comment 4

[Andrew Overholt [:overholt]]

I tried switching from Dark to Light theme in devtools and can't reproduce. Maybe bug 1287122 did indeed fix this?

Comment 5

[Marcia Knous [:marcia]]

ni on Andrei - Can you please try to reproduce this using the steps in Comment 0 - thanks!

Comment 6

[Andrei Vaida [:avaida]]

(In reply to Marcia Knous [:marcia - use ni] from comment #5)
> ni on Andrei - Can you please try to reproduce this using the steps in
> Comment 0 - thanks!

I was unable to reproduce this bug on an affected build - 50.0a1 (2016-07-14), using the str from Comment 0 (both dark and light theme variant) with Windows 7 x64, Windows 7 x86 and Windows 10 x64.

Perhaps I'm missing something here? arni2033, are you still able to reproduce this bug?

Comment 8

[arni2033]

Attachment: testcase 1 - bug 1287074.XUL

(In reply to Andrew Overholt [:overholt] (back Sept 14) from comment #3)
> (In reply to arni2033 from comment #2)
> > I can still reproduce the bug with the old styling.
> > 
> > [...]
> > 
> > Devtools developers have made a workaround for this in pushlog [1], but the
> > underlying bug isn't fixed
> 
> Oh, you mean it's fixed with the dark theme but will "re-appear" once 50 goes to beta? 
I was very busy this month. No, I meant that this was Core::Layout bug, that exists regardless of what
styling is used in devtools. Once devtools changed styling, method of testing described in comment 0
no longer lead to the bug (because testcase from comment 0 was altered), but the bug was still present

> Is that what you mean by "old styling" above?
CSS that was used in devtools before pushlog [1] (comment 2)

Originally I just applied old styling via browser toolbox; but apparently I have to post "testcase 1"
(reproducible as of 2016-08-04). This testcase was fixed by pushlog [2], most likely by bug 1287122.
> [2] https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=79b31ab92faa&tochange=906df3f2cce7