474451 - Dragging the scrollbar grip in the calendar list doesn't scroll the list

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Stefan Sitter [:ssitter]]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090120 Calendar/1.0pre (BuildID: 20090120031333)

Steps to Reproduce:
1. Create new calendars until the scrollbar is visible in the calendar list
2. Scroll the list by dragging the scrollbar grip up and down

Actual Results:
The list doesn't scroll. Instead a drag and drop operation is started to reorder the calendars.

Expected Results:
Dragging the scrollbar grip works like in any other scrollbar.

Most probably regressed by Bug 266249.

Comment 1

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Attachment: Fix - v1

This patch fixes the issue described and also a few other DND problems we've been having:

* Dragging of calendars anywhere doesn't cause errors or view drop shadows
* Dragging of events doesn't allow drop except in
  - view (body)
  - richlistbox in today pane

I've noticed the drop shadows don't get removed when leaving the view but this is a bug that also exists without my patch.

Comment 2

[Berend Cornelius]

Comment on attachment 357935 [details] [diff] [review]
Fix - v1

>I've noticed the drop shadows don't get removed when leaving the view but this
>is a bug that also exists without my patch.
This is the reason why Bug 416138 -  Moving of all-day-events in weekview not possible, but shows possibility was reopened. I will take care of this in the next days.

>+               if (!session.sourceNode || !session.sourceNode.sourceObject) {
>+                  // No source item? Then this is not for us.
>+                  return;
>+               }
We have this query also in the "dragdrop" event of the binding unfortunately after session.sourceNode.sourceObject has been used. Maybe you could place it at the beginning of the event. This causes an error when you try to drag a calendar into the calendar-view.

Dropping a task on the agenda-listbox throws an error, regressed by bug 471973 - Make use of alarm interface in backend code. We are using an "aEvent" object where only a "aTask" object is available.

The patch looks very good! r=berend

Comment 3

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

(In reply to comment #2)
> We have this query also in the "dragdrop" event of the binding unfortunately
> after session.sourceNode.sourceObject has been used. Maybe you could place it
> at the beginning of the event. This causes an error when you try to drag a
> calendar into the calendar-view.
I see, there is a task dnd patch up for review where I touch more code in this area. I'll fix that there.


> 
> Dropping a task on the agenda-listbox throws an error, regressed by bug 471973
> - Make use of alarm interface in backend code. We are using an "aEvent" object
> where only a "aTask" object is available.
I fixed this in a different patch thats up for review.

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/ba01c954461c>

-> FIXED