openerp-community-reviewer team mailing list archive
-
openerp-community-reviewer team
-
Mailing list archive
-
Message #00531
Re: [Merge] lp:~therp-nl/web-addons/7.0-web_popup_large into lp:web-addons
Review: Needs Information code review, no test
I also see this as an usability improvement. Only one question: why the "if (!options.width)" check and not putting always options.width ='95%'?
Regards.
--
https://code.launchpad.net/~therp-nl/web-addons/7.0-web_popup_large/+merge/186496
Your team Web-Addons Core Editors is subscribed to branch lp:web-addons.