13456 - Clicking on down arrow in Bookmarks in Sidebar opens new window

Status: VERIFIED DUPLICATE of bug 9029

(SeaMonkey :: Sidebar, defect, P3)

Description

[cpratt]

Build ID: 1999090908
Platform: RH Linux 6, Windows NT

To reproduce:
- Launch apprunner
- Display the sidebar
- Scroll down to see the Bookmarks area
- Click a few times on the down arrow on the scrollbar in the Bookmarks area

Result: A new browser window is opened. In this window, the following files are
listed: bm-panel.xul, bm-props.js, bm-props.xul, bookmark-popup.js... etc.

Expected result: Nothing should happen if the list has already been scrolled all
the way down.

Comment 1

[Steve Lamm]

I see this too. Looks like a tree bug.
Reassigning to hyatt.

Comment 2

[Steve Lamm]

*** Bug 13379 has been marked as a duplicate of this bug. ***

Comment 3

[Steve Lamm]

*** Bug 13646 has been marked as a duplicate of this bug. ***

Comment 4

[Steve Lamm]

*** Bug 11508 has been marked as a duplicate of this bug. ***

Comment 5

[David Hyatt]

My hands have deteriorated to the point where I can no longer type.  I need
help.  If you think you can fix this bug on your own, please take it away from
me.  If you'd like to volunteer to be my hands for a specific bug, then I'll be
happy to come up to your cube and sit with you and fix the bug (assuming you
have the patience for that).

Comment 6

[David Hyatt]

*** This bug has been marked as a duplicate of 9029 ***

Comment 7

[cpratt]

Not being terrifically bright when it comes to DOM, how exactly is this a
duplicate of "DOM events with anonymous nodes as targets should not bubble up to
non-anonymous nodes"? Please help!

Comment 8

[David Hyatt]

The scrollbar click should be consumed before the handler that is installed on
the tree even sees it.  Because it is being allowed to bubble up from the
scrollbar to the tree, the handler on the tree mistakenly views the click on the
scrollbar as a click on a bookmark item in the tree.

The fix is to prevent the click event on the scrollbar from being seen by the
handler on the tree.

Comment 9

[cpratt]

Aha! Verifying this one as duplicate.