Closed
Bug 106325
Opened 23 years ago
Closed 23 years ago
Drag bookmarks into folders
Categories
(SeaMonkey :: Bookmarks & History, enhancement)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: trudelle, Assigned: bugzilla)
References
Details
(Whiteboard: [ADT2 rtm])
Attachments
(1 file)
(deleted),
patch
|
bugs
:
superreview+
|
Details | Diff | Splinter Review |
This is a feature tracking bug for the ability to file into specific location
within folders in popup menus. This is pretty strange UI but some Windows users
are used to it, and expect us to go the next step. Blake is reportedly working
on it at the moment. Need estimate of task duration.
Comment 1•23 years ago
|
||
Bugs this might depend on:
bug 18052 Ability to D&D the proxy icon into the bookmarks menu or submenu
bug 96504 [linux] after dragging proxy icon onto bookmarks button, bookmarks
menu refuses to close
bug 50505 d&d bookmarks in menu
bug 41888 [RFE] Quickfile: 'File' bookmarks to cascading submenus, to file in
folders (on the fly)
bug 19437 [RFE] Move, delete, and edit bookmarks inline (in menu)
bug 50504 Context menu for bookmarks menus
bug 100479 Dragging link over Personal Toolbar folder should open menu
Keywords: meta
Comment 2•23 years ago
|
||
What's the target and priority for this bug? Are we planning to have this done
for MachV?
--> Trudelle for reassignement
Assignee: pchen → trudelle
Comment 4•23 years ago
|
||
This is currently not on my set of assigned features, so I'm assuming it has
been cut.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
Reporter | ||
Comment 5•23 years ago
|
||
No, it hasn't been cut, and you haven't identified it as being at risk. Is it?
Target Milestone: Future → ---
Reporter | ||
Comment 7•23 years ago
|
||
Cool!, but it has to be done in ->099, or made a candidate for cutting now.
Target Milestone: --- → mozilla0.9.9
Comment 8•23 years ago
|
||
actually this worked fine before bookmarks-outliner landed, there are several
bugs about this feature being broken, dragging between folders, bookmarks list,
from url bar to personal toolbar folders, etc.
Comment 9•23 years ago
|
||
that is what is bug is about is it not?
Assignee | ||
Comment 10•23 years ago
|
||
No, this is about dragging bookmarks into folders on the personal toolbar. It
will get done for 099 as I have a patch.
I will note, however, that as of now this works in the same way as it does for
the bookmarks button: you can't currently drop into a specific location of a
nested folder.
That is, given
[ Button ]
[ Bookmark 1 ]
[ Folder 1 ] > [ Bookmark 2 ]
[ Folder 2 ]
Folder 2 will not expand to allow specific placement (however, you can drop on
it to append it to the end -- the folder icon opens and the item highlights to
indicate this.)
This was originally deemed okay for a first-cut implementation because most
users presumably don't have such nested folders, and because anything more
complicated -- trying to keep track of the open nested folders -- would probably
be buggy at best.
If it's deemed a priority that nested folder opening work as well, I can
certainly post a patch, but I think it's even less important for this bug. I
don't consider a folder within a folder within a personal toolbar folder the
common usage scenario (a folder within a personal toolbar folder will open as
expected).
Status: NEW → ASSIGNED
Reporter | ||
Comment 11•23 years ago
|
||
I think you're exactly right Blake; even my worst-case Bookmark folder doesn't
have that scenario. IMO, it is not worth handling, though if someone wanted to
contribute a rock-solid patch for it, that might be worth considering.
Assignee | ||
Comment 12•23 years ago
|
||
this is fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 13•23 years ago
|
||
Arg? Are you sure? It doesn't work on my win32 build from Feb. 21. Isn't this
bug for expanding personal toolbar folders when one drags a link over them, and
letting the user place the bookmark in a particular place in that folder (like
the windows98 and later start menu)?
Comment 14•23 years ago
|
||
Hi, this still doesn't work on Feb. 25 builds. I see no patch attached to this
bug. I really think it is a mistake to mark this fixed.
Comment 15•23 years ago
|
||
From Blake's comment 10, I'd say this is a dup of bug 100479.
Comment 16•23 years ago
|
||
reopening, this bug as reported is not fixed in linux build 2002 02 25 08
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 17•23 years ago
|
||
Odd...works in my homegrown build. I'll look into this later, on my way out.
Assignee | ||
Comment 18•23 years ago
|
||
Comment 19•23 years ago
|
||
Comment on attachment 72890 [details] [diff] [review]
missed xpfe/browser/resources/content changes.
sr=ben@netscape.com
Attachment #72890 -
Flags: superreview+
Assignee | ||
Comment 20•23 years ago
|
||
Pierre, give it another go; this should work now.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Comment 21•23 years ago
|
||
verified fixed on windows build 2002 03 16 03
I filed bug 133600, 133602 and 133604 for the remaining issues and especially
bug 133601 and bug 133621.
However I can not verify this bug with linux build 2002032608, since the
behavior is horked:
- no feedback line
- DND does not seem to perform
- there is a persistent DND cursor icon above th menu
- and once, I could not close the folder menu (same as bug 96504 still waiting
for reviews)
I don't know what is the best thing to do: reopening this bug or file some pp
linux bugs.
Comment 22•23 years ago
|
||
*** Bug 135091 has been marked as a duplicate of this bug. ***
Comment 23•23 years ago
|
||
Sorry about the duplicate report, it's pretty hard to tell what is what without
being familiar with the component.
I just got build 2002031104
I still cannot drag links onto the toolbar icons and have them open. The link
goes to the bottom of the list in the toolbar icon, cannot be placed in a
folder, etc. in this build. Works the same as the older build I was using.
So in windows 98, not fixed.
Comment 24•23 years ago
|
||
Works for me on win98. Try getting a more recent build, 0311 is more than half
a month old!
Comment 25•23 years ago
|
||
The weird thing is, I got that build from the stub installer last night, off the
FTP.
Wonder how that could have happened. I am looking for a newer build now.
Comment 26•23 years ago
|
||
thanks for the excellent QA work pierre (comment 21)
I wanted to mark this bug verified and simply go forward with the other 'small'
bugs, but it's a disgrace to fixed bugs to call this one fixed as well.
4 seconds of testing shows numerous errors with this feature on a Win98
2002041003 trunk build.(I just tried to drag the url proxy icon to a folder via
the bookmarks button on the toolbar and couldn't)
Did trying to cover the case in comment 10 lead to introducing all these bugs?
at this stage in the game I only see three options with the following cons:
1. Leave things as is and ship with obvious, broken, and quirky behavior.(even
worse on OSX for me)
-obvious ramifications
2. Try to fix all of these little bugs.
-probably risky to muck around with XPMenus at this point. Plus, will likely
involve several iterations until we hit a 'good enough' point.
3. Rollback the most recent changes and pare down the functionality.
-probably only slightly less risky than (2)
I'd be pleased to know if there is another answer.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Comment 27•23 years ago
|
||
ADT2. cc ben & bryner for possible help on this.
Whiteboard: [ADT2]
Assignee | ||
Comment 28•23 years ago
|
||
Huh??? Please point out the specific bugs you see. I don't see any of the
problems Pierre points out. And dragging and dropping the proxy icon works fine
for me, unless I already have a bookmark to that url, in which case it does
nothing (there's nothing I can do about that, it's a bookmarks problem).
Assignee | ||
Comment 29•23 years ago
|
||
Okay, I see that the issues Pierre listed are pp Linux. I thought we had
disabled this feature on Linux long ago because of issues with that platform. If
not, that is an easy fix, if we want to.
I also see that he linked to a few Windows bugs. The most important of them
seems to be that dragging over multiple folders in the personal toolbar opens
them. That is simple to fix -- but that doesn't seem to be what you're
complaining about.
None of this has to do with XP menu code.
Comment 30•23 years ago
|
||
As I see it the issue with bookmarks and windows is this: You cannot drag a
link into a folder on the personal toolbar, except to just drop it onto the
folder. Iis then at the end of the list on that folder. There is no real way
to get it anywhere except at the end. There could be organized folders in
there, but you can't put a link into one. You gotta go in and rearrange it.
Even dragging a link into the bookmarks list (crtl-B) is difficult because as
soon as you click the link in the browser URL bar the bookmarks window goes to
back. In netscape 4.7 this does not occur.
Assignee | ||
Comment 31•23 years ago
|
||
Steve, are you using a nightly build or a milestone?
Comment 32•23 years ago
|
||
I am using last night's nightly build. Which has stuff broken which was fixed
last week. Heh.
Reporter | ||
Comment 33•23 years ago
|
||
The worst thing I see is the folder/menus not closing on mouseOut, which bites
me several times a day when I overshoot dragging to a tab. I'm using NS
2002041011 on Win2K. Is there a separate bug on that? We should get separate
defects into their own bug reports.
Comment 34•23 years ago
|
||
trudelle: the bug you are seeing is bug 133601 (onDragOver) and bug 133621
(after onDrop). See the dependencies in bug 133604 for the other issues.
Btw, I tested this feature on Linux with the patch from bug 96504: all the good
and all the bad were similar to the Windows builds. No more, no less.
Comment 35•23 years ago
|
||
Worksforme.
Windows 98 build 2002041103
I drag a link from the url bar to the toolbar icons, and they open, revealing
their folders, etc. Perfect!
However, the ctrl-b bookmark window still pops to back if I click on the link
icon in the browser window. Unlike 4.77
Comment 36•23 years ago
|
||
*** Bug 138159 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 37•23 years ago
|
||
Well, I'm re-resolving this FIXED since I have nsbeta1+ bugs on the most
important and reproducible issues.
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•23 years ago
|
Whiteboard: [ADT2] → [ADT2 rtm]
Comment 38•22 years ago
|
||
rs vrfy --able to d'n'd proxy icon to bookmarks folder in PT.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•