← Back to team overview

yellow team mailing list archive

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