DnD month view: Error if event is dragged out of displayed calendar date range

VERIFIED FIXED

Status

Calendar
Calendar Views
VERIFIED FIXED
12 years ago
12 years ago

People

(Reporter: Stefan Sitter, Assigned: Joey Minta)

Tracking

Trunk
x86
Windows 2000

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
Fall out from Bug 312736:
DnD month view: Error if event is dragged out of displayed calendar date range

Steps to Reproduce:
0. Default settings, Sunday is first day of week
1. Switch to Month view, e.g. June 2006
2. Create event at end of month view, 
   e.g. 2006-06-29 12:00 to 2006-06-30 12:00
3. Drag event to 2006-07-01, so that the second drop shadow would be
   displayed on 2006-07-02 (but that is out of displayed calendar date range)

Actual Results:
Drag and Drop stops working. Drop shadow is displayed on last position.
JavaScript console shows:

    Error: this.monthView.findBoxForDate(shadowStart) has no properties
    Source File: chrome://calendar/content/calendar-month-view.xml
    Line: 458

Same results when doing that steps at begin date of calendar view.

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060629
Sunbird/0.3a2+ (my own trunk build)
(Assignee)

Comment 1

12 years ago
Created attachment 227699 [details] [diff] [review]
check for box

Patch checks to see whether we actually get a box back from findBoxForDate.
Assignee: base → jminta
Status: NEW → ASSIGNED
Attachment #227699 - Flags: first-review?(dmose)
(Assignee)

Comment 2

12 years ago
Comment on attachment 227699 [details] [diff] [review]
check for box

r=dmose
Attachment #227699 - Flags: first-review?(dmose) → first-review+
(Assignee)

Comment 3

12 years ago
patch checked in with some merging fun.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

12 years ago
Works fine with Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060721 Calendar/0.3a2+ --> VERIFIED
Status: RESOLVED → VERIFIED
Component: Internal Components → Calendar Views
QA Contact: base → views
You need to log in before you can comment on or make changes to this bug.