Merge lp:~openerp-dev/openobject-addons/6.1-opw-577465-msh into lp:openobject-addons/6.1

Proposed by Mohammed Shekha(Open ERP)
Status: Approved
Approved by: Naresh(OpenERP)
Approved revision: 6926
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-577465-msh
Merge into: lp:openobject-addons/6.1
Diff against target: 14 lines (+3/-1)
1 file modified
account/account_invoice_view.xml (+3/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-577465-msh
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+117419@code.launchpad.net

Description of the change

Hello,

Fixed the issue of field amount_tax not shown at proper place when the compute tax is invisible in some state.

(The issue is faced in web-client and This is usability issue.)

There is col=4 and colspan=2 so and button has colspan=2, so when button is hidden so whole td is hidden and so amount_tax will get extra colspan and due to which field will positioned wrongly.

Thanks.

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

This bug was qualified as Not Relevant on Trunk (means that architecture on trunk has changed and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6926. By Mohammed Shekha(Open ERP)

[FIX]Fixed the issue of field amount tax not shown at proper place when compute tax button is hidden(in state other than draft).

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/account_invoice_view.xml'
2--- account/account_invoice_view.xml 2012-02-16 18:50:48 +0000
3+++ account/account_invoice_view.xml 2012-07-31 12:13:25 +0000
4@@ -198,7 +198,9 @@
5 </field>
6 </group>
7 <group col="4" colspan="2">
8- <button colspan="2" name="button_reset_taxes" states="draft" string="Compute Taxes" type="object" icon="terp-stock_format-scientific" help="This action will erase taxes"/>
9+ <group colspan="2" col="1">
10+ <button name="button_reset_taxes" states="draft" string="Compute Taxes" type="object" icon="terp-stock_format-scientific" help="This action will erase taxes"/>
11+ </group>
12 <field name="amount_untaxed"/>
13 <label string="" colspan="2"/>
14 <field name="amount_tax"/>