search

sgodden / echo3ext20

Automatically exported from code.google.com/p/echo3ext20
Other
0 stars 1 forks source link

Conditional re-rendering method uses brute force and uses static fields #9

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The conditional re-rendering method invoked when the server update is
complete uses brute force, in that it re-renders the root ext panel every
time, whether children were added or not.

Additionally, it forces all open windows to re-render, instead of the one
currently focused.

The proper solution is to allow containers to be observers on their
children, and to bubble up add and remove events that occur, so that the
root containers can determine whether they really need to re-render.

Original issue reported on code.google.com by sgod...@gmail.com on 14 Apr 2008 at 11:04

GoogleCodeExporter commented 9 years ago 
Additionally, the list of open windows is held statically, which could cause
problems.  That need would be removed by the suggested resolution above anyway.

Original comment by sgod...@gmail.com on 14 Apr 2008 at 11:05

GoogleCodeExporter commented 9 years ago 
Fixed

Original comment by sgod...@gmail.com on 14 Apr 2008 at 12:19