yellow team mailing list archive
-
yellow team
-
Mailing list archive
-
Message #01147
Re: Preserve zoom settings in the environment view. (issue 6720048)
LGTM with the minor noted.
Thanks
https://codereview.appspot.com/6720048/diff/4001/app/views/environment.js
File app/views/environment.js (right):
https://codereview.appspot.com/6720048/diff/4001/app/views/environment.js#newcode243
app/views/environment.js:243: if (sourceEvent.type === 'dblclick' &&
.on("dblclick.zoom", null) during the bind will remove the dblclick
behavior.
https://codereview.appspot.com/6720048/
--
https://code.launchpad.net/~frankban/juju-gui/preserve-zoom/+merge/130087
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~frankban/juju-gui/preserve-zoom into lp:juju-gui.
References