Closed Bug 271891 Opened 20 years ago Closed 20 years ago

[FIX]Cannot drop to subfolder of the personal toolbar's folder

Categories

(SeaMonkey :: Bookmarks & History, defect, P1)

Product:
SeaMonkey
Component:
Bookmarks & History
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.8beta1

People

(Reporter: masayuki, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

Patch
1.04 KB, patch
dbaron
: review+
dbaron
: superreview+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a5) Gecko/20041123
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a5) Gecko/20041123

I cannot drop bookmark item to subfolder on personal toolbar's folder.
It is not reproduced on 2004112306.
After 20041124xx builds are bad.

Reproducible: Always
Steps to Reproduce:
1. Create folder 'A' on personal toolbar
2. Create folder 'B' in folder 'A'
3. Drag the bookmark item, and drop to 'B'

Actual Results:  
The folder 'B' is not opened, and feedback line is shown above and bottom the
folder. not hilight the folder.

Expected Results:  
The folder 'B' should be opened, and we should be able to drop to the folder.

http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=MozillaTinderboxAll&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2004-11-23&maxdate=2004-11-24+08%3A00%3A00&cvsroot=%2Fcvsroot
Is this regression caused by bug 268576?
This worksforme in a Linux debug build pulled from CVS at "Mon Nov 29 16:02:35
CST 2004" (as well as another build I have from Nov 25).
I can reproduce with 2004113004/WinXP.
Perhaps I am misunderstanding the steps to reproduce?

I create a folder "A" on the personal toolbar and a folder "B" inside it.  Then
I grab the url in the urlbar, drag it to "A" and don't let go.  After half a
second or so "A" opens and I can drag down to "B", which  opens too (it's empty,
so shows an empty popup).  If I release over "B", the bookmark is dropped into "B".

I just retested this with a 2004-11-30-08 mozilla.org nightly (still on Linux).
 It works the same exact way as a 2004-11-22-08 build.

Perhaps this is a Windows-only issue?
Keywords: qawanted
Sorry, my steps are wrong.

1. Create folder 'A' on personal toolbar
2. Create folder 'B' in folder 'A'
3. Create folder 'C' bottom 'B'(in 'A')
3. Drag the bookmark item, and drop to 'C'

I can drop 'B', but cannot drop 'C'.

Please re-test on Linux.
Personal Toolbar 'A'
                  +-'B'      <- B is opened and can be dropped
                  +-'C'      <- C is not opened and can not be dropped
Is this the same problem that occurs in bug 276126 ?
Status: UNCONFIRMED → NEW
Ever confirmed: true
*** Bug 276126 has been marked as a duplicate of this bug. ***
Bug 276126 was a Firefox bug. If bug 276126 ist duped against this one, should
product not be changed to core? 
In addition this seems not to be only a problem for the personal toolbar. As
well dragging an url via the bookmark menu item does not open the sub-folders.
This is a problem for me and others I have installed FF for. I use 20041228
while others use a mix of nightlies after 20041228. All plugins had been
disabled/removed with no success.
Using XP sp1.
Attached patch Patch — — Splinter Review 
Nakano-san, thank you for the steps in comment 5!  With those, I can indeed
reproduce the bug, and you're correct that it's a regression from bug 268576. 
I'm sorry it took me so long to respond to that comment; I was out of town and
had very spotty computer access...

David, the patch is pretty simple.  We were double-counting the offset of
mRect, since it was getting included both in the offset from the view and in
ourRect.  As a result the screen coordinates this method returned were off by
mRect.TopLeft().
Assignee: p_ch → bzbarsky
Status: NEW → ASSIGNED
Attachment #170721 - Flags: superreview?(dbaron)
Attachment #170721 - Flags: review?(dbaron)
Keywords: qawanted
OS: Windows XP → All
Priority: -- → P1
Hardware: PC → All
Summary: Cannot drop to subfolder of the personal toolbar's folder → [FIX]Cannot drop to subfolder of the personal toolbar's folder
Target Milestone: --- → mozilla1.8alpha6
A similar issue is that you cannot drag and place a bookmark as the last item in
a list just as you cannot drag into a folder that is last in a list.
That's the same issue.
Blocks: 273226
Bug 276017, bug 273466 and bug 271891 are essentially all the same bug. Could we
just pick one and dupe the others to get some focus? This is an important bug.
Peter, please read bugs before making inane comments like that.  For example,
this bug and bug 273466 are very clearly NOT the same bug.
The bugs are similar, but may or may not be the same
Bug 276017 is about dropping a bookmark in a folder in the Bookmarks folder
bug 273466 is about dropping a bookmark in the bookmark sidebar
bug 271891 (this one) is about dropping a bookmark in a personal toolbar subfolder
(In reply to comment #16)
> Peter, please read bugs before making inane comments like that.  For example,
> this bug and bug 273466 are very clearly NOT the same bug.

Boris: Please try to get *something* positive out of a comment before making
accusations like that. I did read the bugs. I mainly wanted to point out the
*existence* of the other very similar bugs ("dragging URLs into bookmarks") that
*might* have the same or a similar solution, in *case* some of the developers
were not aware of the other bugs. Sorry for not being more clear (s/same/similar).
Peter, there is no "the developers."  For example, I really could not care less
about bugs that are specific to the bookmarks front-end implementation in
Firefox.  I do care about core bugs like this one.  I'm sure the Firefox
developers feel the same way, in reverse.
Attachment #170721 - Flags: superreview?(dbaron)
Attachment #170721 - Flags: superreview+
Attachment #170721 - Flags: review?(dbaron)
Attachment #170721 - Flags: review+
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: mozilla1.8alpha6 → mozilla1.8beta
Another bug seems to be fixed by this (was perhaps a dupe): Bug 277070.
Version: unspecified → Trunk
*** Bug 277070 has been marked as a duplicate of this bug. ***
*** Bug 276017 has been marked as a duplicate of this bug. ***
*** Bug 282668 has been marked as a duplicate of this bug. ***
For lack of any drag/drop into subfolder functionality in OSX, I opened a new
bug: bug 299185

"Toolbar folders don't open when a url is dragged onto them - prevents placement
of bookmarks in subfolders or where desired in order in folders"


*** Bug 314981 has been marked as a duplicate of this bug. ***
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: