Closed Bug 127437 Opened 22 years ago Closed 22 years ago

[BRANCH ONLY] Moving bookmarks in personal toolbar places them in wrong position

Categories

(SeaMonkey :: Bookmarks & History, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.1alpha

People

(Reporter: bugzilla, Assigned: p_ch)

References

Details

Attachments

(1 file)

Using build 2002022203, dragging and dropping bookmarks in the personal toolbar
puts them one place to the right of where they should be.
Confiming
Status: UNCONFIRMED → NEW
Ever confirmed: true
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.1
*** Bug 130453 has been marked as a duplicate of this bug. ***
*** Bug 130836 has been marked as a duplicate of this bug. ***
*** Bug 132390 has been marked as a duplicate of this bug. ***
Blocks: 133604
*** Bug 134656 has been marked as a duplicate of this bug. ***
Didn't see this when I created my duplicate.  Not entirely the same though;
looks like this happens in Win2k and Win98 as well.
taking.
The indices of the elements of a RDF start from 1.
So there is no need to increment the index when calling InsertElementAt in
CreateBookmarkWithDetails and CreateFolderWithDetails.
The two methods (with an insertion index != -1) are called in bookmarkOverlay.js
(no need to modify it) and navigatorDD.js.
I modified the latter so that the insertion correctly performs for both:
- dragging bookmarks from the PT
- proxy icon
Requesting r=, sr=
Assignee: ben → pierrechanial
Status: ASSIGNED → NEW
Attached patch patch v1.0Splinter Review
*** Bug 137117 has been marked as a duplicate of this bug. ***
Attachment #78830 - Attachment is obsolete: true
Depends on: 139471
*** Bug 140071 has been marked as a duplicate of this bug. ***
*** Bug 140342 has been marked as a duplicate of this bug. ***
fix is now in bug 139471
*** Bug 140971 has been marked as a duplicate of this bug. ***
Being annoyed by Bookmarks always appearing where I didn't expect them after
drag-and-drop (in Personal toolbar and in "Manage Bookmarks" - mostly they
appear one line above from where I placed them - this is one of my most hated
bugs nowadays) I looked around in bugzilla - and found a real jungle (including
this bug here) of open bugs describing the same:

bug 114256 (bookmark gets into wrong folder after dnd) although related, this
particular aspect is fixed now, but at least one of the dups is more general and
not adressed yet (bug 114878).

bug 114606 (bookmark goes to position above) describes exactly what I see and
hate, but in the meanwhile this bug has been somewhat changed into a tracking
bug and now also adresses other bugs related to bookmark d'n'd. (however,
component is not changed to "Tracking")
Many bugs that also describe the exact behaviour are duped to this one (I'll
list them later) because of its desription, but now nobody cares anymore about
the described problem because this one is now considered a tracking bug by some
people...

bug 119879 (bookmark appears one line above after dnd) is 100% what I see and hate.

bug 127437 (wrong position after dnd in personal toolbar) is about the same
problem just at another place (still mostly off-by-one after dragging bookmark).
Has *lots* of duplicates.

bug 137119 (bookmark positioned one line too high after dragging) is about
bookmark manager again, still the same problem. Was duped against bug 114606
already (same summary) but later reopened (bug 114606 is supposed to be tracking).

bug 139471 is finally about general dnd cleanup and is also supposed to fix this
particular issue I'm talking about. Very recently worked on, but now
unfortunately removed from RC2 list.


So it looks like (almost!) all of these six open bugs are duplicates of one
another, but correct duping and resolving was prevented by halfheartedly
changing bug 114606 into a tracking bug.
In addition the underlying bug I speak about (bookmark d'n'd results in wrong
position) was reported in many other bugs which are now duped to one of the six
bugs above: bug 115420, bug 116076, bug 117134, bug 120245, bug 122599, bug
130702, bug 137119, bug 130453, bug 130836, bug 132390, bug 134656, bug 137117,
bug 140071, bug 140342 and bug 140971 are the ones I found.


As you can see from the dup count or simply from imagining a browser where
bookmarks do not materialize where they are dropped, this is a quite serious
problem (and in my humble opinion should not go into 1.0). 

I think it would be a rewarding task for someone with the appropriate
permissions to clean up this jungle a bit. Maybe by clearly (and correctly)
labeling a general tracking bug and a separate one for the one particular d'n'd
problem I'm talking about (and duping the others against it).
This cleanup would get us rid of several ASSIGNED and NEW bugs and prepare the
way for fixing this one annoying d'n'd issue. 
Bug 139471 unfortunately was taken off the RC2 list (bug 138000) by Asa because
of its complexity. A simple "fix this one bug" wouldn't have been, I think.
*** Bug 143765 has been marked as a duplicate of this bug. ***
*** Bug 144233 has been marked as a duplicate of this bug. ***
*** Bug 146849 has been marked as a duplicate of this bug. ***
fixed by the checkin of bug 139471
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Reopening this bug for the branch.
Trunk is already patched.
Summary: Moving bookmarks in personal toolbar places them in wrong position → [BRANCH ONLY] Moving bookmarks in personal toolbar places them in wrong position
Really reopening
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #78830 - Attachment is obsolete: false
If this is fixed on the trunk then the correct thing to do is mark it
RESOLVED/FIXED. To get it on the branch, you add the adt1.0.1 keyword.
Status: REOPENED → RESOLVED
Closed: 22 years ago22 years ago
Keywords: adt1.0.1
Resolution: --- → FIXED
ADT has nothing to do with non-NS contributors. email drivers@mozilla.org to ask
the patch be taken under consideration for the branch.
Keywords: adt1.0.1
I erroneously marked this bug nsbeta1+ instead of nsbeta1, removing the
nomination per Christopher comment if I understand correctly the logic. I will
mail the drivers.
Keywords: nsbeta1+
*** Bug 153719 has been marked as a duplicate of this bug. ***
*** Bug 161937 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
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: