openerp-community-reviewer team mailing list archive

[Nicolas Bessi - Camptocamp <nicolas.bessi@xxxxxxxxxxxxxx>]

Review: Needs Information

 <th class="list_main_headers" style="width: 100%">
277	+ <table style="width:100%">
278	+ <tr>
279	+ <th class="main_col1">${_("Description")}</th>

I'm not sure about this one. Why do you include this table into header?

Else LGTM
-- 
https://code.launchpad.net/~camptocamp/sale-reports/7.0-sync-webkit-reports-20130926/+merge/187702
Your team OpenERP Community Reviewer is subscribed to branch lp:sale-reports.