Code review comment for lp:~openerp-dev/openerp-web/trunk-bug-1119289-dhruvit

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Nimesh,

to be more precise about this: rev. 3701 is a refactoring of https://code.launchpad.net/~openerp-dev/openerp-web/7.0-opw-589823-msh. This fixes a bug in which the columns break out of the page layout, and it adds the scrollbar. Now, if you take a look at the screenshot that demonstrates the issue (in the bug's first comment), you will see that the scollbar is already there. Camp2camp must have gotten that as an OPW fix. The bug's description clearly complains about the unused space in combination with the scrollbar and the cramped one2many layout. Therefore, revision 3700 fixes the problem and revision 3701 reverts it again.

Dhruvit, please revert revision 3701 and we are done. Yes, this resizes the sheet which is a huge improvement on usability.

« Back to merge proposal