391574 - Position of text field not correct when editing individual tree cells (if horizontally scrolled)

Status: UNCONFIRMED

(Core :: XUL, defect)

Description

[westblue]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; sl; rv:1.8.1.6) Gecko/20070725 Firefox/2.0.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7) Gecko/2007080210 GranParadiso/3.0a7

Position of text field is not located correctly when editing individual tree cells if the tree is scrolled horizontally.

Reproducible: Always

Steps to Reproduce:
1.Create a tree with the horizontal scroller active.
2.Enable editing of individual cells.
3.Scroll horizontally and double-click on an editable cell.
Actual Results:  
Position of the text field is not aligned with a tree column.

Expected Results:  
Position of the text field should be aligned with a tree column.

Comment 1

[westblue]

Attachment: Tree with horizontal scroll and enabled editing of individual cells

Comment 2

[westblue]

getCoordsForCellItem method also doesn't work if horizontally scrolled. It returns 0 for x and y cords.

Comment 3

[Mime Čuvalo]

This bug is probably related to bug 392216 which deals with *vertical* scrolling.

I hope Jan can get around to looking into it b/f Firefox 3 though.

Comment 4

[Mime Čuvalo]

Bug 399961 is actually being worked on which fixes a similar issue with in nsTreeBodyFrame.cpp, the getCellAt function.

In this case it's the getCoordsForCellItem that's messing up.  I'll attach a temporary workaround to this for anybody who's interested but that's all it is: temporary.  The bug lies in fixing getCoordsForCellItem.

Comment 5

[Mime Čuvalo]

Attachment: Temporary workaround

Comment 6

[Jan Varga [:janv]]

The plan is to remove the XUL "tree" widget in future (bug 1446335), but this has a patch, so for now I'm not going to wontfix this.