Code review comment for lp:~agilebg/account-invoice-report/7.0-fix_invoice_webkit_description_bugs

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :

I disagree with renaming back list_main_table to list_invoice_table

This was intended to be able to use same css through invoice, sale_order and so on.

Plus it seems to me it reintroduce
https://bugs.launchpad.net/account-invoice-report/+bug/1231329

Does this fix still ensure that a formated_note line is on the same page as its invoice line?
I see you removed that nobreak.

review: Needs Information

« Back to merge proposal