Closed Bug 1521417 Opened 11 months ago Closed 11 months ago

Some items of calendar list context menu are permanently disabled


(Calendar :: Calendar Views, enhancement)

Lightning 6.8
Not set


(Not tracked)



(Reporter: MakeMyDay, Assigned: MakeMyDay)



(1 file)

In calendar list context menu, some items (Remove calendar, Publish calendar, properties,...) are permanently disabled, while accessing e.g. the calendar properties dialog is still possible using the main menu.

This is a fallout from bug 1482389, which changed the behaviour of getCellAt to not support out parameters anymore but return values instead.

The only other caller of getCellAt in calendar was already converted in bug 1491269.

Attachment #9037863 - Flags: review?(philipp)
Attachment #9037863 - Flags: review?(philipp) → review+
Keywords: checkin-needed

Pushed by
Use getCellAt return values instead of output parameters; r=philipp

Closed: 11 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 6.8
Comment on attachment 9037863 [details] [diff] [review]

Review of attachment 9037863 [details] [diff] [review]:

::: calendar/base/content/widgets/calendar-list-tree.xml
@@ +648,5 @@
>                  aRow = aRow || {};
> +                aCol = aCol || {};
> +                let { col, row } = this.treebox.getCellAt(event.clientX, event.clientY);
> +                aCol.value = col;
> +                aRow.value = row;

Are you sure that this works?
Most other callers use let cell = tree.getCellAt() and then use cell.row and cell.col. See also .

Why do you assign to aCol.value and aRow.value if you could use col/row or (cell.col/cell.row) directly? aCol and aRow isn't passed anywhere as an object. I see they are used as out-arguments. The 2 callers could be updated easily to not access them as objects anymore (aCol.value and aRow.value).
You need to log in before you can comment on or make changes to this bug.