515979 - The menu of the upper hierarchy is scrolled by a mouse wheel on sub-menu, and The select position of the menu slips off.

Status: RESOLVED WORKSFORME

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Alice0775 White]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a2pre) Gecko/20090911 Firefox/3.5.1
Build Identifier: http://hg.mozilla.org/mozilla-central/rev/ecb355c31e14

When use overflow:auto for scrollbox,
The menu of the upper hierarchy is scrolled by a mouse wheel on the sub menu, and The select position of the sub menu slips off.

Reproducible: Always

Steps to Reproduce:
1.Start Minefield with new profile
2.Open TestCase
3."MENU-B" > Select "BB001" > Wheel scroll on "BBB0**", and Move mouse top/bottom on the "BBB0**".

Actual Results:  
The menu of the upper hierarchy is scrolled by a mouse wheel on the sub menu, and The select position of the sub menu slips off.

Expected Results:  
The menu of the upper hierarchy should not be scrolled
And the select position of the menu properly.

Comment 1

[Alice0775 White]

Attachment: testCase

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

There is some regression here (the selected menu item handling).
Though, I think mouse scroll on BBB0** shouldn't scroll BB***, and that is not a
regression from 1.9.0.

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #2)
> There is some regression here (the selected menu item handling).
Regression range could be very useful.

Comment 4

[Alice0775 White]

Regression window is as follows:

Works (as same as Firefox3.0.*):
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080903034741 Minefield/3.1b1pre ID:20080903034741

Broken:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080904035000 Minefield/3.1b1pre ID:20080904035000

Pushlog(from the above build Id, because I cannot identify changesets of the above build):
http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2008-09-03+03%3A47%3A41&enddate=2008-09-04+03%3A50%3A00

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Attachment: possible patch

Needs tests, but I think something like could work..

Comment 6

[eyal gruss]

is this related to Bug 530504 ?

Comment 7

[Alice0775 White]

(In reply to comment #6)
> is this related to Bug 530504 ?

I think so, because Created an attachment (id=400292) fixes the issue of Bug 530504 too.

Comment 9

[Alice0775 White]

Attachment: revised testcase (due to Bug 658949)

For your record