Merge lp:~openerp-dev/openerp-web/6.1-opw-581018-cpa into lp:openerp-web/6.1

Proposed by Chirag Patel (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/6.1-opw-581018-cpa
Merge into: lp:openerp-web/6.1
Diff against target: 21 lines (+7/-0)
1 file modified
addons/web/static/src/js/view_form.js (+7/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/6.1-opw-581018-cpa
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+132850@code.launchpad.net

Description of the change

Hello,

Notebook all page hidden but last page content display.

If we have attrs that set invisible on some condition the pages of notebook and all pages are hidden.

When open form view, it will display the content of last hidden page.
All page should be hidden.

Thanks.

To post a comment you must log in.

Unmerged revisions

2474. By Chirag Patel (OpenERP)

[FIX] Fixed minor changes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/view_form.js'
2--- addons/web/static/src/js/view_form.js 2012-10-23 07:30:10 +0000
3+++ addons/web/static/src/js/view_form.js 2012-11-05 09:12:22 +0000
4@@ -1233,10 +1233,17 @@
5 this._super.apply(this, arguments);
6 this.$element_tab = this.notebook.$element.find(
7 '> ul > li:eq(' + this.index + ')');
8+ this.$element_page_div = this.notebook.$element.find(
9+ '> div:eq(' + this.index + ')');
10 },
11 update_dom: function() {
12 if (this.invisible && this.index === this.notebook.$element.tabs('option', 'selected')) {
13 this.notebook.do_select_first_visible_tab();
14+ this.$element_page_div.addClass('ui-tabs-hide');
15+ } else {
16+ if (this.$element_tab.hasClass("ui-state-active")){
17+ this.$element_page_div.removeClass('ui-tabs-hide');
18+ }
19 }
20 this.$element_tab.toggle(!this.invisible);
21 this.$element.toggle(!this.invisible);