Closed Bug 24106 Opened 25 years ago Closed 23 years ago

Moving an item out of the visual area with reorder button should not be possible

Categories

(SeaMonkey :: Sidebar, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.2

People

(Reporter: bugzilla, Assigned: matt)

References

Details

(Whiteboard: will verify after 87168 is fixed.)

Attachments

(1 file)

When you go into the customize dialog and select a panel and presses the reorder button it's possible to move the panel out of the visual area. That should not be possible! The moved panel should always be visible. So when moving out of the visual the scroll bar has to move with it. Build 2000011508
layout problem, m14
Target Milestone: M14
Depends on: 26079
Move to M16 ...
Target Milestone: M14 → M16
spam: changing qa contact on sidebar bugs from paulmac to shrir@netscape.com (all 67 of them!)
QA Contact: paulmac → shrir
Target Milestone: M16 → M17
Move to M20 target milestone.
Target Milestone: M17 → M21
Gemal, I don't see this happening? Could u pls confirm? Thx
This can be reproduced: 1) Open the "Customize Sidebar" dialog 2) If you have fx 4 sidebar entries in the "My Sidebar" tree resize the dialog so that you can only see 3 of them. The vertical scrollbar will appear. 3) Now highlight the first entry (What's related) 4) Press the arrow down button and move the "What's Related" down 5) If you keep moving you can move it out of the visual of the area. That should not be the case.
True...I see it..thanks! (2000081105m18)
Reassigning 50 Sidebar bugs to Matt. I was told this was going to be done shortly about two months ago, but it clearly hasn't been. I think that's long enough for all these bugs to remain assigned to nobody. Feel free to filter all this spam into the trashcan by looking for this string in the message body: ducksgoquack
Assignee: slamm → matt
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9
spam : changing qa to sujay (New Sidebar QA)
QA Contact: shrir → sujay
Is this a sidebar bug or a more general XUL tree bug or something?
OS: Windows 98 → All
Hardware: PC → All
If there's a generic function for moving an item from one place in a tree to another, then this is an XP Toolkit/Widgets: Trees bug. If not, then this is a Sidebar bug.
Mass moving most of mozilla0.9 bugs to mozilla0.8.1
Target Milestone: mozilla0.9 → mozilla0.8.1
This looks like a tree problem. I have focus on the tree item and it is select. That item should come into focus without any special code. http://lxr.mozilla.org/seamonkey/source/xpfe/components/sidebar/resources/custom ize.xul
Assignee: matt → hyatt
You can use ensureElementIsVisible on the tree.
Assignee: hyatt → matt
My fathers got a great set of tools. I can fix it. .... Next mile stone.
Target Milestone: mozilla0.8.1 → mozilla0.9
nav triage team: Marking nsbeta1+, mozilla0.9.1
Keywords: nsbeta1nsbeta1+
Target Milestone: mozilla0.9 → mozilla0.9.1
Don't take this off my 9.1 radar!!!!!!
as discussed in team meeting, moving all Nav+ team members nsbeta1+ P3 bugs from mozilla0.9.1 to mozilla0.9.2.
Target Milestone: mozilla0.9.1 → mozilla0.9.2
day:1 risk:little
Whiteboard: 1 day, eta 6/22
sidebar+pdt triage: hyatt's comment is probably the answer to this bug.
Attached patch patch (deleted) — Splinter Review
Whiteboard: 1 day, eta 6/22 → posted patch, getting reviews
r=mcafee
corrected the damn tab!
Whiteboard: posted patch, getting reviews → have review...need A
a= asa@mozilla.org for checkin to the trunk. (on behalf of drivers)
Blocks: 83989
fixed
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Henrik, is this fixed for you now? please try on latest build. thanks..
hmmm... since resizing the customize sidebar tabs window doesn't produce any scrollbars it's impossible to test.
I opened a new bug on the scrollbar issue. I will cc: you on it... see bug 87168. This bug cannot be verified until that one is fixed.
Whiteboard: have review...need A → will verify after 87168 is fixed.
well I need a scrollbar to verify this one, but Matt says no scrollbar. I will mark this VERIFIED-FIXED.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: