From e44c46467fba5add60a9084c9dbc7bbaf7ecac74 Mon Sep 17 00:00:00 2001
From: Guewen Baconnier <guewen.baconnier@camptocamp.com>
Date: Thu, 4 Dec 2014 10:15:38 +0100
Subject: [PATCH] Improve usability by splitting the view in 2 parts: rule,
 result

---
 .../account_statement_operation_rule_view.xml | 24 ++++++++++---------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/account_statement_operation_rule/view/account_statement_operation_rule_view.xml b/account_statement_operation_rule/view/account_statement_operation_rule_view.xml
index d2d12577..8025126d 100644
--- a/account_statement_operation_rule/view/account_statement_operation_rule_view.xml
+++ b/account_statement_operation_rule/view/account_statement_operation_rule_view.xml
@@ -17,18 +17,20 @@
             <group name="options">
               <field name="rule_type"/>
             </group>
-            <group name="amount" string="Balance" attrs="{'invisible': [('rule_type', 'not in', ('rounding', 'currency'))]}">
-              <label for="amount" string="When the balance is between"/>
-              <div>
-                <field name="amount_min" class="oe_inline" /> And
-                <field name="amount_max" class="oe_inline" />
-              </div>
-              <label for="currencies" string="And the currency is one of" attrs="{'invisible': [('rule_type', '!=', 'currency')]}"/>
-              <div attrs="{'invisible': [('rule_type', '!=', 'currency')]}">
-                <field name="currencies" class="oe_inline" widget="many2many_tags"/>
-              </div>
+            <group name="rule" string="Rule">
+              <group name="amount" attrs="{'invisible': [('rule_type', 'not in', ('rounding', 'currency'))]}">
+                <label for="amount" string="When the balance is between"/>
+                <div>
+                  <field name="amount_min" class="oe_inline" /> And
+                  <field name="amount_max" class="oe_inline" />
+                </div>
+                <label for="currencies" string="And the currency is one of" attrs="{'invisible': [('rule_type', '!=', 'currency')]}"/>
+                <div attrs="{'invisible': [('rule_type', '!=', 'currency')]}">
+                  <field name="currencies" class="oe_inline" widget="many2many_tags"/>
+                </div>
+              </group>
             </group>
-            <group name="operations">
+            <group name="operations" string="Result">
               <label for="operations" string="Then the following operations will be applied:" colspan="2"/>
               <field name="operations" nolabel="1"/>
             </group>