rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stanton Sievers" <siever...@gmail.com>
Subject Review Request 15460: Home view binds click events multiple times
Date Tue, 12 Nov 2013 19:15:23 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for rave.

Bugs: RAVE-1078

Repository: rave


>From the JIRA:

The HomeView that gets registered in rave_ui.js will bind click events every time the view's
render() method is called.  This causes problems, for instance when the widget is maximized
and then minimized, because the elements to which the events were bound are never removed
nor are the events ever unbound.  

Thus, in the case of maximizing and then minimizing a widget (i.e. navigating to the "canvas"
view and then back to the "home" view) the events are bound multiple times.

You can see this behavior if you set a breakpoint in the "maximize()" method defined in HomeView.prototype.render
in rave_ui.js.  If you repeat the process of maximizing and minimizing the widget, the "maximize()"
method will get called more than once.



Diff: https://reviews.apache.org/r/15460/diff/


Existing tests pass.


Stanton Sievers

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message