Ticket #2531287 (accepted enhancement)
ReporterJonathan Potter |
Opened: 10/10/11 Last modified: 09/19/12 Status: accepted Type: enhancement |
Owner Eric Ferraiuolo |
Target Release: BACKLOG Priority: P2 (high) |
|---|---|---|---|
| Summary: | Widget-StdMod: Elements added to the bodyContent during subclass's renderUI phase are added twice | ||
| Description: | Steps to reproduce: Expected behavior: "_addStdModContent" prints once. This only happens when bodyContent is set during the renderUI phase. I think the reason this is happening because of the following. _renderUIStdMod listens for the bodyContent change with:
But _syncUIStdMod also checks to see if the bodyContent has changed which it does by checking if _stdModParsed is set. So when bodyContent is set in the subclass's renderUI function, _renderUIStdMod |
||
| Type: | enhancement | Observed in Version: | development master |
| Component: | Overlay | Severity: | S3 (normal) |
| Assigned To: | Eric Ferraiuolo | Target Release: | BACKLOG |
| Location: | Library Code | Priority: | P2 (high) |
| Tags: | Relates To: | ||
| Browsers: | Safari 5.x - Mac,Chrome | ||
| URL: | |||
| Test Information: | |||
Change History
|
Posted: 10/10/11
|
|
Posted: 11/8/11
|
|
Posted: 11/8/11
|
|
Posted: 11/8/11
|
|
Posted: 12/14/11
|
|
Posted: 02/16/12
|
|
Posted: 04/9/12
|
|
Posted: 07/9/12
|
|
Posted: 09/19/12
Moving from 3.NEXT to BACKLOG. |
Punting on this for now. It doesn't add duplicate content, but it is doing unnecessary work which should be optimized for performance.
I created a jsFiddle which shows the content only being rendered once, but a lot of event firing:
http://jsfiddle.net/ericf/r5Rsp/