Creating event by dragging in calendar view: text edit of description is hanging - happens in cases where auto-scroll to create new fails
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 |
2.41 KB,
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:
Create an event by dragging in calender view (without need to scroll view). Then edit description text.
Actual results:
It is not possible to terminate editing of description text. Text cursor continues to blink. You cannot deselect the newly created event, by clicking elsewhere. You cannot select a different event by clicking on it. You cannot switch to view another week, etc.
In fact Calendar is hanging.
I did not find a way to solve this, except quitting Thunderbird and starting again.
Expected results:
Text editing should be terminated by clicking elsewhere, as in previous versions.
Comment 1•4 years ago
|
||
Seems to work find for me. Clicking elsewhere, or Enter, will finish the new event.
(In reply to Magnus Melin [:mkmelin] from comment #1)
Seems to work find for me. Clicking elsewhere, or Enter, will finish the new event.
I tried to reproduce the bug and indeed bug description above is incomplete.
The above effect occurs after you had tried to create an event by dragging where end time is out of view and therefore auto-scroll should occur (see bug 1683326).
Also by accident now i found, that text editing may be terminated by pressing TAB (instead of ENTER in earlier versions), however one still cannot select an existing event by clicking on it and you cannot switch to view another week.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 3•4 years ago
|
||
My colleague (Windows) and I (Linux) see a possibly related issue (shall I open separately? in this component?):
Be e.g. in week view. Drag a new entry. Old behavior was that I can edit its title, then enter to save it. New behavior (since December?) is that the new entry is auto-closed immdeately – and saved without title. I may then double-click and edit it, but the old way was way better IMHO.
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 4•3 years ago
•
|
||
Even with auto-scroll fixed (bug 1683326), you can still get an error if you move beyond the scroll area and move into the allday header.
Assignee | ||
Comment 5•3 years ago
|
||
We only end the drag if we are moving an event.
Depends on D130342
Assignee | ||
Comment 6•3 years ago
|
||
Note that the patch will fix this bug (dragging to create an event), but the same error can occur if you drag to move the event into the allday header, but I'll leave that to bug 1675056.
Assignee | ||
Comment 7•3 years ago
|
||
Please land the patch after D130342 in bug 1683326
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/cc92ad2ce71c
Stop ending a column mouse drag when resizing or creating events. r=darktrojan
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 9•3 years ago
|
||
Comment on attachment 9249208 [details]
Bug 1683328 - Stop ending a column mouse drag when resizing or creating events. r=darktrojan
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Dragging to create or extend an event can end earlier than necessary if the user accidentally leave the view. Moreover, it is converted into a drag session, which can cause errors on dropping.
Testing completed (on c-c, etc.): No tests.
Risk to taking this patch (and alternatives if risky): Low risk.
Comment 10•3 years ago
|
||
Comment on attachment 9249208 [details]
Bug 1683328 - Stop ending a column mouse drag when resizing or creating events. r=darktrojan
[Triage Comment]
Approved for beta
bug 1683326 uplift first
Comment 11•3 years ago
|
||
bugherder uplift |
Thunderbird 95.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/32ff7c300b53
Updated•3 years ago
|
Assignee | ||
Comment 12•3 years ago
|
||
Patch moved to calendar-event-column.js
in absence of bug 1731209 for esr91.
Assignee | ||
Comment 13•3 years ago
|
||
Comment on attachment 9252067 [details] [diff] [review]
bug1683328-esr91.patch
[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Dragging to create or extend an event can end earlier than necessary if the user accidentally leave the view. Moreover, it is converted into a drag session, which can cause errors on dropping.
Testing completed (on c-c, etc.): No tests.
Risk to taking this patch (and alternatives if risky): Low risk.
Comment 14•3 years ago
|
||
Comment on attachment 9252067 [details] [diff] [review]
bug1683328-esr91.patch
[Triage Comment]
Approved for esr91
Comment 15•3 years ago
|
||
bugherder uplift |
Thunderbird 91.4.0:
https://hg.mozilla.org/releases/comm-esr91/rev/c17f4bf0bdf2
Description
•