1538789 - TypeError: treeCellInfo.col is null 2 foldersummary.js:210:9

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[:aceman]

When you carefully hover the mouse to the right of the Size column in the folder pane (under the tree column picker widget) and wait a moment so that a tooltip menu coul appear, I get:
TypeError: treeCellInfo.col is null 2 foldersummary.js:210:9

The code is:
let treeCellInfo = gFolderTreeView._tree.getCellAt(event.clientX, event.clientY);
if (treeCellInfo.col.id == "folderNameCol") {

There may not be any column at this place.

Comment 1

[:aceman]

Attachment: 1538789.patch

This should do it. The other change is that if a cell has truncated text (e.g. Size) and the subfolders have zero in that same quantity, we would show nothing. In that case, show that truncated text as the user probably wanted to see it, thus the hover. If subfolders are non-zero we already show the count for the current folder (and also subfolders separately).

Comment 2

[:aceman]

Thanks.

Comment 3

[Jorg K (CEST = GMT+2)]

Comment on attachment 9053352 [details] [diff] [review]
1538789.patch

This looks like a beta candidate. Can you please consider whether your patches are needed on beta and request uplift.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/3f77eae3ca02
Don't show tooltip if not hovering a cell in the folder pane. Show full cell text when on a truncated cell. r=darktrojan DONTBUILD

Comment 5

[:aceman]

I'm not sure when this broke, if it was the conversion to CE, or conversion of trees in m-c. It is not highly critical, if you do not happen to hit that small spot where the is no cell, nothing happens, the other cells work fine. So it would only be a polishing patch landing on beta.

Comment 6

[Jorg K (CEST = GMT+2)]

TB 67 beta:
https://hg.mozilla.org/releases/comm-beta/rev/922f9c9d7024ad76354f18566d905c1cdbdeed75