From 59c12749e083b093632481690c50c48a32931ce1 Mon Sep 17 00:00:00 2001
From: Raphael Collet <rco@openerp.com>
Date: Mon, 4 Apr 2011 10:14:03 +0200
Subject: [PATCH] user interface cleanup

bzr revid: rco@openerp.com-20110404081403-puhk9066y2qdvlxb
---
 addons/portal/portal.py       | 2 +-
 addons/portal/portal_view.xml | 9 +++------
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/addons/portal/portal.py b/addons/portal/portal.py
index 5e18185c3e6..ef048f2ea75 100644
--- a/addons/portal/portal.py
+++ b/addons/portal/portal.py
@@ -145,7 +145,7 @@ class users(osv.osv):
     _inherit = 'res.users'
     
     def default_get(self, cr, uid, fields, context=None):
-        """ override default values of menu_id for portal users """
+        """ override default value of menu_id for portal users """
         defs = super(users, self).default_get(cr, uid, fields, context)
         
         # the value of 'menu_id' is passed in context by the portal form view
diff --git a/addons/portal/portal_view.xml b/addons/portal/portal_view.xml
index 5f9e8daf794..1850ed56ff8 100644
--- a/addons/portal/portal_view.xml
+++ b/addons/portal/portal_view.xml
@@ -33,12 +33,9 @@ portal's users.
             <field name="arch" type="xml">
                 <form string="Portal Form">
                     <field name="name"/>
-                    <notebook>
+                    <field name="group_id" on_change="onchange_group(group_id)"/>
+                    <notebook colspan="4">
                         <page string="Users">
-                            <group colspan="4">
-                                <field name="group_id"
-                                    on_change="onchange_group(group_id)"/>
-                            </group>
                             <field name="user_ids" nolabel="1" colspan="4"
                                 context="{'menu_id': menu_action_id}"/>
                         </page>
@@ -46,7 +43,7 @@ portal's users.
                             <field name="menu_action_id"/>
                             <newline/>
                             <field name="parent_menu_id"
-                                context="{'ir.ui.menu.full_list':True}"/>
+                                context="{'ir.ui.menu.full_list': True}"/>
                             <button name="create_parent_menu" type="object"
                                 string="Create Parent Menu"/>
                         </page>