121180 - clicking scrollbar arrows in sidebar (bookmarks/history) triggers pageload

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Brian Landers]

Bookmarks sidebar open, bookmark selected (i.e. I just visited a bookmarked
page).  Click either the up/down arrows or in the scrollbar "channel" (i.e. to
scroll bookmarks by a page).  Causes the browser to reload the bookmark
currently selected.  Seems to act just as if I clicked on the bookmark again.

Running the 2002012108 build on Mandrake Linux 8.1 (2.4.16 kernel).

Comment 1

[R.K.Aa.]

Confirming.

-Single click on up/down arrows in bookmarks sidebar triggers
pageload of selected bookmark.
-Single clicking the focus-ring on either side of scrollbar also triggers a
pageload.

Likely unrelated but mentioning all the same:
Right-clicking a bookmark spawns a pagefull of this js error in console:
Warning: reference to undefined property aResource.QueryInterface
Source File:
chrome://communicator/content/bookmarks/bookmarks.xml#bookmarksBase.resolveType()
Line: 2

Comment 2

[R.K.Aa.]

bug 121184 seems to be a variant of this on MSWindows

Comment 3

[R.K.Aa.]

adding regression keyw. Can't find this filed, but user on Windows says week old.

Comment 4

[R.K.Aa.]

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

Comment 5

[Brian Landers]

Update: opening/closing a folder in the bookmark tree, i.e. clicking on the [+]
causes the same behavior.

Dragging the "thumb" does NOT cause the behavior.

Comment 6

[sujay]

--> bookmarks/claudius

Comment 7

[R.K.Aa.]

this isn't unique to bookmarks - it affects history-sidebar as well:
Double-clicking an url in sidebar history will normally load it from there.
And double-clicking scrollbar arrows in history sidebar will load a selected url
in history.

Comment 8

[R.K.Aa.]

Modifying summary.

This happens in both modern and classic theme, and widgets in both history and
bookmark sidebar are affected. Changing component to XP-apps/GUI features.

Comment 9

[Blake Ross]

Attachment: patch

Comment 10

[Blake Ross]

r=hewitt
sr=ben

Fixed.

Comment 11

[Jan Varga [:janv]]

sorry blake, another regression after outliner syntax change landing

Comment 12

[R.K.Aa.]

something weird happens to my local build with v 1.24 of 
mozilla/xpfe/components/history/resources/historyTreeOverlay.xul

History gets trunkated down to nothing but a weird row of blank columns.
All history items gone. Context menus also blank.
Sidebar and main history window both affected.

Comment 13

[Brian Landers]

Still broken in build 2002020222 (Linux).  Selecting a bookmark then clicking
the up/down arrows or the scroll "channel" in the sidebar reloads the bookmarked
page.  Note, collapsing or expanding an outliner [+] does NOT seem to trigger
the bug any longer.

Comment 14

[R.K.Aa.]

Brian Landers: The fix wasn't checked untill Feb 3rd.
Re-resolving as fixed.

Comment 15

[Chris Lyon]

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

Comment 16

[Oliver Jowett]

Still seeing a variant of this in 0.9.9:

- expand a folder in the bookmarks tree
- scroll the bookmarks sidebar all the way down
- collapse the folder, the currently selected page reloads