No auto-scroll when creating event via dragging in calendar tab
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(thunderbird_esr91+ fixed, thunderbird95+ fixed)
People
(Reporter: MozBug20, Assigned: henry-x)
References
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details | Review |
923 bytes,
patch
|
wsmwk
:
approval-comm-esr91+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:83.0) Gecko/20100101 Firefox/83.0
Steps to reproduce:
Try to create an event in calendar tab by dragging from start time down. When bottom of window is reached, no auto-scroll occurs as in earlier versions, but instead selected rectangle for new event disappears.
Actual results:
No event created. Selected rectangle disappears.
Expected results:
View should auto-scroll down to allow creating event with end time out of view.
For further actual results also see related bug 1683328
Comment 2•5 years ago
|
||
In the console I get:
Uncaught TypeError: can't access property "hashId", item is undefined
mSelectedItems chrome://calendar/content/calendar-multiday-base-view.js:1509
doDeleteItem chrome://calendar/content/calendar-multiday-base-view.js:1508
get chrome://global/content/customElements.js:696
onModifyItem chrome://calendar/content/calendar-base-view.js:77
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onModifyItem resource:///modules/CalCompositeCalendar.jsm:50
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onModifyItem resource:///modules/CalICSCalendar.jsm:869
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
modifyItem resource:///modules/CalMemoryCalendar.jsm:274
processQueue resource:///modules/CalICSCalendar.jsm:558
modifyItem resource:///modules/CalICSCalendar.jsm:495
doTransaction resource:///modules/CalTransactionManager.jsm:137
createAndCommitTxn resource:///modules/CalTransactionManager.jsm:25
doTransaction chrome://calendar/content/calendar-item-editing.js:669
modifyOccurrence chrome://calendar/content/calendar-views-utils.js:97
stopEditing chrome://calendar/content/calendar-editable-item.js:428
onkeypress chrome://calendar/content/calendar-editable-item.js:269
addEventNameTextboxListener chrome://calendar/content/calendar-editable-item.js:266
connectedCallback chrome://calendar/content/calendar-multiday-view.js:334
relayout chrome://calendar/content/calendar-event-column.js:637
mEventMapTimeout chrome://calendar/content/calendar-event-column.js:457
setTimeout handler*addEvent chrome://calendar/content/calendar-event-column.js:457
doAddItem chrome://calendar/content/calendar-multiday-base-view.js:1490
get chrome://global/content/customElements.js:696
onAddItem chrome://calendar/content/calendar-base-view.js:62
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onAddItem resource:///modules/CalCompositeCalendar.jsm:46
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onAddItem resource:///modules/CalICSCalendar.jsm:866
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
adoptItem resource:///modules/CalMemoryCalendar.jsm:174
addItem resource:///modules/CalMemoryCalendar.jsm:112
processQueue resource:///modules/CalICSCalendar.jsm:553
adoptItem resource:///modules/CalICSCalendar.jsm:482
addItem resource:///modules/CalICSCalendar.jsm:475
doTransaction resource:///modules/CalTransactionManager.jsm:133
createAndCommitTxn resource:///modules/CalTransactionManager.jsm:25
doTransaction chrome://calendar/content/calendar-item-editing.js:669
createNewEvent chrome://calendar/content/calendar-views-utils.js:42
onEventSweepMouseUp chrome://calendar/content/calendar-event-column.js:1632
connectedCallback chrome://calendar/content/calendar-event-column.js:128
calendar-multiday-base-view.js:1509:9
mSelectedItems chrome://calendar/content/calendar-multiday-base-view.js:1509
filter self-hosted:238
doDeleteItem chrome://calendar/content/calendar-multiday-base-view.js:1508
get chrome://global/content/customElements.js:696
onModifyItem chrome://calendar/content/calendar-base-view.js:77
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onModifyItem resource:///modules/CalCompositeCalendar.jsm:50
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onModifyItem resource:///modules/CalICSCalendar.jsm:869
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
modifyItem resource:///modules/CalMemoryCalendar.jsm:274
processQueue resource:///modules/CalICSCalendar.jsm:558
modifyItem resource:///modules/CalICSCalendar.jsm:495
doTransaction resource:///modules/CalTransactionManager.jsm:137
createAndCommitTxn resource:///modules/CalTransactionManager.jsm:25
doTransaction chrome://calendar/content/calendar-item-editing.js:669
modifyOccurrence chrome://calendar/content/calendar-views-utils.js:97
stopEditing chrome://calendar/content/calendar-editable-item.js:428
onkeypress chrome://calendar/content/calendar-editable-item.js:269
(Async: EventHandlerNonNull)
addEventNameTextboxListener chrome://calendar/content/calendar-editable-item.js:266
connectedCallback chrome://calendar/content/calendar-multiday-view.js:334
relayout chrome://calendar/content/calendar-event-column.js:637
mEventMapTimeout chrome://calendar/content/calendar-event-column.js:457
(Async: setTimeout handler)
addEvent chrome://calendar/content/calendar-event-column.js:457
doAddItem chrome://calendar/content/calendar-multiday-base-view.js:1490
get chrome://global/content/customElements.js:696
onAddItem chrome://calendar/content/calendar-base-view.js:62
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onAddItem resource:///modules/CalCompositeCalendar.jsm:46
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
onAddItem resource:///modules/CalICSCalendar.jsm:866
notify resource:///modules/calendar/utils/calDataUtils.jsm:38
notify resource:///modules/calendar/utils/calDataUtils.jsm:65
adoptItem resource:///modules/CalMemoryCalendar.jsm:174
addItem resource:///modules/CalMemoryCalendar.jsm:112
processQueue resource:///modules/CalICSCalendar.jsm:553
adoptItem resource:///modules/CalICSCalendar.jsm:482
addItem resource:///modules/CalICSCalendar.jsm:475
doTransaction resource:///modules/CalTransactionManager.jsm:133
createAndCommitTxn resource:///modules/CalTransactionManager.jsm:25
doTransaction chrome://calendar/content/calendar-item-editing.js:669
createNewEvent chrome://calendar/content/calendar-views-utils.js:42
onEventSweepMouseUp chrome://calendar/content/calendar-event-column.js:1632
(Async: EventListener.handleEvent)
connectedCallback chrome://calendar/content/calendar-event-column.js:128
Updated•5 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
It seems that setupMagicScroll
is broken https://searchfox.org/comm-central/rev/bc0b58e63e7fc62dd4c7c0adb83d1a2732719600/calendar/base/content/calendar-multiday-view.js#973
I'll just create a new method to do the same thing once I finish bug 1738689
Assignee | ||
Comment 4•4 years ago
|
||
(In reply to Henry Wilkes [:henry] from comment #3)
It seems that
setupMagicScroll
is broken https://searchfox.org/comm-central/rev/bc0b58e63e7fc62dd4c7c0adb83d1a2732719600/calendar/base/content/calendar-multiday-view.js#973I'll just create a new method to do the same thing once I finish bug 1738689
Actually, the fix for the magic scroll is quite simple. I might revise the method again though.
Assignee | ||
Comment 5•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/954c06bf686f
Fix multiday auto-scroll. r=darktrojan
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 7•4 years ago
|
||
Comment on attachment 9249197 [details]
Bug 1683326 - Fix multiday auto-scroll. r=darktrojan
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Multiday-view auto-scroll does not work.
Testing completed (on c-c, etc.): No
Risk to taking this patch (and alternatives if risky): Low risk. One line fix.
Comment 8•4 years ago
|
||
Comment on attachment 9249197 [details]
Bug 1683326 - Fix multiday auto-scroll. r=darktrojan
[Triage Comment]
Approved for beta
Comment 9•4 years ago
|
||
bugherder uplift |
Thunderbird 95.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/7ff87d7f1e1a
Assignee | ||
Comment 10•4 years ago
|
||
Patch moved to calendar-event-column.js
in absence of bug 1731209 for esr91.
Assignee | ||
Comment 11•4 years ago
|
||
Comment on attachment 9252066 [details] [diff] [review]
bug1683326-esr91.patch
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Multiday-view auto-scroll does not work.
Testing completed (on c-c, etc.): No
Risk to taking this patch (and alternatives if risky): Low risk. One line fix.
Comment 12•4 years ago
|
||
Comment on attachment 9252066 [details] [diff] [review]
bug1683326-esr91.patch
[Triage Comment]
Approved for esr91
Comment 13•4 years ago
|
||
bugherder uplift |
Thunderbird 91.4.0:
https://hg.mozilla.org/releases/comm-esr91/rev/80c70243ba52
Description
•