Bookmark Menu won't scroll with Dragged URL (also Personal Toolbar)

NEW
Unassigned

Status

()

Core
XUL
--
minor
16 years ago
6 years ago

People

(Reporter: Ed Lana, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.7 -
blocking1.9 -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: workaround comment 21)

Attachments

(1 attachment, 4 obsolete attachments)

(Reporter)

Description

16 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.3b) Gecko/20030217
Build Identifier: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.3b) Gecko/20030217

Neither the Bookmark Menu or the Personal Toolbar menu scroll when a URL is
dragged to their respective drop down menus.

Reproducible: Always

Steps to Reproduce:
How to Reproduce

Step #1 Have a book mark folder or personal bookmark folder with enough entries
to require you to scroll the bookmark list to see them all.

Step #2 Drag the URL handle from the URL Address Bar to the bookmark drop down
menu and attempt to scroll down the bookmark list.
Actual Results:  
Result: You can't scroll


Expected Results:  
The Bookmark list would scroll when the dragged object was over the up or down
arrow on the drop down list.

OS: Win98SE
Mozilla: 1.3b 2003021704

This may be related to bug 139403 but because it is outside of the original
description of the problem I am submitting it as a separate bug.

Comment 1

15 years ago
The same with post-1.4a Trunk build on Win2k.
-> NEW
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

15 years ago
QA Contact: kasumi → petersen

Comment 2

15 years ago
I'm using the same OS as the original report with identical problems and
frequency in versions 1.0 through 1.4.

Bookmark can't be dragged up or down from the presently viewable area in a
personal toolbar folder drop down.  If I try to pre-scroll the drop down before
dragging a URL, I can only drop the bookmark by itself, not in a folder.

If it matters, I moved the bookmarks from NN4.74.  I can't recall what I did but
I remember having issues setting up the personal toolbar folder in version 1.0
Is there a guide somewhere for installing NN4 bookmarks so that I can try a
clean install?
Mass reassign of my non-Firefox bugs to ben_seamonkey@hotmail.com
Assignee: bugs → ben_seamonkey

Comment 4

15 years ago
Reproduced in 1.6 for Windows.

I suggest a nomination as possible 1.7 final blocker.
Flags: blocking1.7?

Comment 5

15 years ago
When did this regress? If it's never worked then it's unlikely to make it for
1.7 final. If a well tested and fully reviewed patch happens in time for final,
please request approval to land. Not going to block the release for this change.
Flags: blocking1.7? → blocking1.7-

Comment 6

14 years ago
*** Bug 238459 has been marked as a duplicate of this bug. ***

Updated

14 years ago
OS: Windows 98 → All

Comment 7

14 years ago
Created attachment 150120 [details] [diff] [review]
patch

This patch should be applied also to 
xpfe/global/resources/content/bindings/scrollbox.xml

this is solvable even if you do not have a compiler.
just unzip your mozilla/toolkit.jar, edit scrollbox.xml
as well as this patch, and zip it again.

Updated

14 years ago
Attachment #150120 - Flags: review?

Comment 8

14 years ago
Comment on attachment 150120 [details] [diff] [review]
patch

hachiro:
Well, this bug is not about Firefox, but the Mozilla suite. But let's not get
too bureaucratic, at least right now...
However, you have to ask a specific person for review or you'll never get it.
I'll choose mconnor for that.
(note: I have not tested this patch)
Attachment #150120 - Flags: review? → review?(mconnor)

Comment 9

14 years ago
Created attachment 150200 [details] [diff] [review]
equivalent patch for seamonkey

This is the same patch for the seamonkey file. Works, but not fully tested.
Scrolling is pretty fast... maybe even as fast as the CPU allows??
Autoscrolling in trees e.g. uses a scroll callback and timer...

hachiro:
did you have a specific reason for letting the event bubble (which is prevented
when the user clicks the scroll arrow) - or is it more like "well, it does
work..." ?
(I'm not saying you should not, just wondering)
And am I wrong when I think
   ondragover="this.parentNode.scrollByIndex(1);"
makes it scroll as fast as possible? What about faster machines in two years?
Won't they possibly scroll too fast?

Comment 10

14 years ago
(In reply to comment #9)
> hachiro:
> did you have a specific reason for letting the event bubble (which is prevented
> when the user clicks the scroll arrow) - or is it more like "well, it does
> work..." ?
> (I'm not saying you should not, just wondering)
> And am I wrong when I think
>    ondragover="this.parentNode.scrollByIndex(1);"
> makes it scroll as fast as possible? What about faster machines in two years?
> Won't they possibly scroll too fast?

I should have tested it in more cases.
There's no specific reason why I didn't add preventBubble.
(maybe, I didn't know about event mechanism well)

What is worse, several bugs surfaced after posting a patch here.

1.Folders never open unless bookmarks-menu show its head portion.
2.bookmarks-menu closes soon after few scrolling in my Firefox(0.8 linux). 

Now, I'm trying to fix these new bugs (and too fast scrolling).
I'm hoping to post a patch again that fixes these bugs.
Comment on attachment 150120 [details] [diff] [review]
patch

based on subsequent comments, I'm going to minus this for now.	let me know if
you get a better solution.
Attachment #150120 - Flags: review?(mconnor) → review-
Product: Browser → Seamonkey
*** Bug 213094 has been marked as a duplicate of this bug. ***
Created attachment 169682 [details] [diff] [review]
Add preventBubble (Firefox)

Adding the preventBubble, I see none of the effects mentioned earlier. The fast
scrolling can be tackled in another bug if needed, it shouldn't hold this one
up since regular scrolling is that fast now anyways.
Attachment #150120 - Attachment is obsolete: true
Attachment #150200 - Attachment is obsolete: true
Attachment #169682 - Flags: review?(mconnor)
Created attachment 169683 [details] [diff] [review]
Add preventBubble (Seamonkey)

Same for the Suite. I have no clue who I should ask for review.
Oh, and Andreas, hachiro, I'd appreciate if you could test this version as you
did the others. It seems to work fine for me on Windows.
*** Bug 276169 has been marked as a duplicate of this bug. ***

Comment 17

14 years ago
In addition to fixing this Bug,  how about taking the initiative and either
adding new Bookmarks at the top of the list or adding an item to the right click
menu for a selected bookmark entry to "Move To Top" and "Move To Bottom", or both ?
A feature you would have, that is Important, that Microsoft does not have in
Internet Explorer 6.

Comment 18

14 years ago
(In reply to comment #14)
> Add preventBubble (Seamonkey)
> 
> Same for the Suite. I have no clue who I should ask for review.

Ask neil.parkwaycc.co.uk or timeless (if not they can help I guess).
Assignee: ben_seamonkey → p_ch
QA Contact: chrispetersen → bookmarks
Attachment #169683 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #169683 - Flags: review?(neil.parkwaycc.co.uk)

Comment 19

14 years ago
I'd prefer a tweak to nsScrollBoxObject.cpp's nsAutoRepeatBoxFrame::HandleEvent
although I don't know the difference between NS_DRAG_EXIT and NS_DRAG_EXIT_SYNTH
Attachment #169682 - Attachment is obsolete: true
Attachment #169682 - Flags: review?(mconnor)
Attachment #169683 - Attachment is obsolete: true
Attachment #169683 - Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #169683 - Flags: review?(neil.parkwaycc.co.uk)
Assignee: p_ch → nobody
Component: Bookmarks → XP Toolkit/Widgets: Menus
Keywords: helpwanted
Product: Mozilla Application Suite → Core
QA Contact: bookmarks
Hardware: PC → All
Note that an eventuel solution that uses timers, won't work in windows, because
of bug 203573.
Maybe a workaround would be to check for how much time has elapsed?
Depends on: 203573

Comment 21

13 years ago
Created attachment 201034 [details] [diff] [review]
patch in nsAutoRepeatBoxFrame::HandleEvent

This patch does not work in Windows because of bug 203573, but I think the logic is sound.  Could someone please test with Linux and/or Mac?

P.S. NS_DRAG_EXIT and NS_DRAG_EXIT_SYNTH are defined to have the same value in http://lxr.mozilla.org/mozilla/source/widget/public/nsGUIEvent.h#262 (revision 3.125).

P.P.S. I have written an extension to fix this problem without timers.  It's a hack and doesn't really belong in the source, but should tide everyone over until this bug is fixed (and bug 203573, in the case of Windows users).  It will be available from https://addons.mozilla.org/extensions/moreinfo.php?id=1411 shortly, pending approval there.

Comment 22

12 years ago
How does the patch in bug 240709 fit in?
Whiteboard: workaround comment 21
> P.S. NS_DRAG_EXIT and NS_DRAG_EXIT_SYNTH are defined to have the same value in
> http://lxr.mozilla.org/mozilla/source/widget/public/nsGUIEvent.h#262 (revision
> 3.125).

richard, because of the same value issue, I had to remove the NS_DRAG_EXIT_SYNTH case in order for your patch to build on mac os x.

I'm rebuilding my tree now to see if your patch fixes an issue I was running into (on the mac) for my patch in bug #318168

Comment 24

12 years ago
I don't know whether this will be of any consequence, but I applied patch 169683 [Add preventBubble (Seamonkey)] to SeaMonkey 1.5a nightly 20060830 on WinXP Pro, and it seems to be working great for me.

Updated

12 years ago
Duplicate of this bug: 367058

Comment 26

11 years ago
Andy in comment #24
> I applied patch 169683 [Add preventBubble (Seamonkey)] to SeaMonkey 1.5a nightly 20060830 on
> WinXP Pro, and it seems to be working great for me.

still fails 
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a3pre) Gecko/20070314 SeaMonkey/1.5a


Comment 27

11 years ago
In the previous release I could work around this bug by pre-scrolling the menu and then dragging the URL to the desired folder in the pre-scrolled menu. But release 2.0.0.6 now prevents pre-scrolling of the menu (it always returns to its original state.) So in 2.0.0.6 there is no work-around to this bug, thereby *making it worse.*

I'm not sophisticated enough to figure out how to install a patch. I hope that this *bug made worse by a new release* (the first Mozilla bug I have ever reported) will get some higher priority.

As requested, on Aug 25 2007 I installed the latest overnight to see if the bug has been fixed, - but the current version of Minefield is worse still: dragging a URL to the bookmark folder does not cause the folder to open at all!
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022601 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

Bug still there.


[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022702 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

This has probably been fixed by bug 389931,
but bug 419740 regression prevents checking this for now.
Blocks: 381699
Severity: normal → minor
Depends on: 389931, 419740
No longer depends on: 203573
Flags: blocking1.9?
Keywords: helpwanted
Depends on: 420341
As I also mentioned in bug 240709, comment 13, this is actually worksforme, except for the main bookmarks menu popup. For folder popups in the bookmarks menu, and sub-folders in the bookmarks menu, this is actually working now in current trunk build.
(In reply to comment #28)
> but bug 419740 regression prevents checking this for now.

Let me be more explicit:

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022704 Minefield/3.0b4pre] (nightly) (W2Ksp4)

Bug 419740 is Firefox specific.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022702 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

SeaMonkey has "similar" bug 420341.

*****

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008022604 Minefield/3.0b4pre] (nightly) (W2Ksp4)

Before bug 389931 regressions (in Gecko/2008022704),
trying to drag a bookmark within a folder of the Personal Toolbar does scroll the folder,
as if the current bug had been already fixed, or worked around, (in Firefox) before that Core patch !
Does anyone know more about this ?
So you're saying that you think this bug is fixed, but you can't test because of another bug, and yet you're nominating *this* bug to block 1.9?  Why?
(In reply to comment #29)
> As I also mentioned in bug 240709, comment 13, this is actually worksforme,

(Yes, I could confirm this in comment 30.)

> except for the main bookmarks menu popup. For folder popups in the bookmarks
> menu, and sub-folders in the bookmarks menu, this is actually working now in
> current trunk build.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9b4pre) Gecko/2008030305 Minefield/3.0b4pre] (nightly) (W2Ksp4)

Confirming:
*Folder in Personal Toolbar: scrolls.
*Bookmarks menu itself: does not scroll. (even after bug 418156, which fixed bug 419740)


[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.1.13pre) Gecko/2008030203 BonEcho/2.0.0.13pre] (nightly) (W2Ksp4)

FWIW, in the Bookmarks menu, when hovering the up/down arrows, FF v2 displays the forbidden pointer then closes the menu :-(
No longer depends on: 419740
(In reply to comment #31)
> So you're saying that you think this bug is fixed, but you can't test because
> of another bug, and yet you're nominating *this* bug to block 1.9?  Why?

Maybe it would be better to move bug 240709 from "Blocks" to "Depends on",
and make *it* "blocking1.9=?" !?
Flags: blocking1.9? → blocking1.9-

Updated

10 years ago
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: xptoolkit.widgets

Comment 34

9 years ago
works for me on firefox 3.6rc / 3.7a1pre
Same bug affects Thunderbird- so its definitely a Mozilla Core problem.

In my extension QuickFolders...
https://addons.mozilla.org/en-US/thunderbird/addon/3254
...you can drag emails onto popup menus that represent the folder tree, if the
list exceeds the screen the drag chevron does not scroll down the menu list.

See also https://www.mozdev.org/bugs/show_bug.cgi?id=21054
(In reply to comment #34)
> works for me on firefox 3.6rc / 3.7a1pre

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.3a1pre) Gecko/20100116 Minefield/3.7a1pre] (nightly) (W2Ksp4)

Same as comment 32: can't test Bookmarks menu due to bug 428411.

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.3a1pre) Gecko/20100116 SeaMonkey/2.1a1pre] (nightly) (W2Ksp4)

Still doesn't work at all.
Depends on: 428411

Comment 37

8 years ago
Fixing this bug would greatly improve the usefulness of the QuickFolders Add-on for Thunderbird.
I have integrated Richard's code from:
https://bugzilla.mozilla.org/show_bug.cgi?id=194319#c21

extension:
https://addons.mozilla.org/extensions/moreinfo.php?id=1411

...into QuickFolders, which now makes the bookmarks scroll as expected (TB3.0.3 / WinXP). It will be released with the next version (QF 1.8.9 or 1.9). 

In the meantime, a test version for QuickFolders users can be downloaded here:
https://www.mozdev.org/bugs/show_bug.cgi?id=21054#c26

please leave some comments there if it works for you.
Fwiw, this bug is still worksforme.

Comment 40

6 years ago
I hadn't tried this for a long time. It's now a worksforme.

User agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:15.0) Gecko/15.0 Firefox/15.0a1 SeaMonkey/2.12a1
Build identifier: 20120602003006
You need to log in before you can comment on or make changes to this bug.