278484 - Tree does not redraw after modifying tree with setCellText

Status: VERIFIED INVALID

(SeaMonkey :: General, defect)

Description

[Michael (GoodWill)]

Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:1.8a6) Gecko/20050111

I am not sure if its a bug since there preceious few piece of code out there
that implement setCelText.

I attaching an xul file in where youc an run two scenarios.
On with teh problem and one where it works at the expense of reapllying the view
schema to a tree.

I do not know if this is by design.

Comment 1

[Michael (GoodWill)]

Attachment: xul file that demostrates the problem.

Comment 2

[Michael (GoodWill)]

note teh problem is on seamonkey release build 1.8a6
Will check if firefox has teh same problem.

Comment 3

[Justin Wood (:Callek)]

<goodwill> can anyone confirm my observations for bug 278484
|<--	goodwill has left irc.mozilla.org (Quit: ....)
<bz_gone> er...
<bz_gone> don't trees allow you to batch stuff?
<bz_gone> so you need to call invalidate() manually after changing stuff?
<bz_gone> could be wrong...
<timelyx> bz: they do

I assume this is INVALID then.  Reopen if bz's and timeless's thoughts were
wrong (which I imagine they are not)

Comment 4

[[e-mail]]

Correct, it is up to the view to tell the tree box object to repaint.
Normal content and builder views do this automatically when their data changes.

Note to reporter: it is unnecessary to write a dummy setter. The JavaScript
engine will throw an exception if you try to set a readonly property.

Comment 5

[Michael (GoodWill)]

This is mainly the problem with setCellText. the constructor/prototype code for
was taken AS IS from the pageInfo.js and pageInfo.xul from 1.8a6. 

then I played around with code and notices that the view is being re-set when
before setCellText again with the same constructor/prototype. and only reason I
foudn that it fixes the refresh problem.

P.S. I did not wite the dummy setter, its the same code as pageInfo.js.

Comment 6

[Michael (GoodWill)]

I guess the main querstion for me here is: 
teh tree.view constructor was set in teh begining. Why do I need to reapply it
before I do setCelltext?

Comment 7

[Michael (GoodWill)]

Filed a documentation bug with a sample application: bug 278536