Merge lp:~openerp-commiter/openerp-web/trunk-bug-992571-psa into lp:openerp-web

[Paramjit Singh Sahota(OpenERP)]

Hello,

  It is fixed. I have modify code to Add and remove class on notebook/page to hide/show and active page on selection change.

Thanks,
PSA (Paramjit Singh).

[Nicolas Vanhoren (OpenERP) (niv-openerp)]

This is really not a correct way to do it. You apply css styles on visible/invisible elements, completely ignoring the fact those styles will be added to *all* elements, not just the notebook pages. A correct way to do it is to customize or re-implement invisibility handling for the precise case of the notebook.

review: Disapprove

Unmerged revisions

2457. By Paramjit Singh Sahota(OpenERP) on 2012-05-21

[FIX][Trunk/6.1] view update on selection change.

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-05-16 14:52:31 +0000
+++ addons/web/static/src/js/view_form.js	2012-05-21 12:54:03 +0000
@@ -1385,6 +1385,8 @@
                 this.$element.hide();
             } else {
                 this.$element.show();
+                this.$element.addClass('ui-state-active');
+                this.$element.removeClass('ui-tabs-hide');
             }
         };
         this.on("change:effective_invisible", this, check_visibility);