Closed Bug 1683328 Opened 1 year ago Closed 2 months ago

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)

Thunderbird 78
defect

Tracking

(thunderbird_esr91+ fixed, thunderbird95+ fixed)

RESOLVED FIXED
96 Branch
Tracking Status
thunderbird_esr91 + fixed
thunderbird95 + fixed

People

(Reporter: MozBug20, Assigned: henry)

References

Details

Attachments

(2 files)

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.

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.

Depends on: 1683326
Summary: Creating event by dragging in calendar view: text edit of description is hanging → Creating event by dragging in calendar view: text edit of description is hanging - happens in cases where auto-scroll to create new fails
Status: UNCONFIRMED → NEW
Ever confirmed: true

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: nobody → henry
Status: NEW → ASSIGNED

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.

We only end the drag if we are moving an event.

Depends on D130342

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.

See Also: → 1675056

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

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch

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.

Attachment #9249208 - Flags: approval-comm-beta?

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

Flags: needinfo?(rob)
Attachment #9249208 - Flags: approval-comm-beta? → approval-comm-beta+
Flags: needinfo?(rob)

Patch moved to calendar-event-column.js in absence of bug 1731209 for esr91.

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.

Attachment #9252067 - Flags: approval-comm-esr91?

Comment on attachment 9252067 [details] [diff] [review]
bug1683328-esr91.patch

[Triage Comment]
Approved for esr91

Attachment #9252067 - Flags: approval-comm-esr91? → approval-comm-esr91+
You need to log in before you can comment on or make changes to this bug.