From 43db60dffa9cc2d99ef59ad384c2f21006ba593e Mon Sep 17 00:00:00 2001 From: Pierre Paridans Date: Thu, 23 Jan 2020 08:47:28 +0000 Subject: [PATCH] [FIX] web: notebook tab headers container This commit adds a wrapper element around the form view's notebook tabs headers to allow the use of `overflow` and make it scrollable (e.g. on smaller screens) without breaking the tabs' styling (e.g. the bottom border). This commit fixes an issue introduced by odoo/enterprise@e07aafbe6fee9e568274c0e157645925dc1f6878. closes odoo/odoo#43843 X-original-commit: 967777793cf2a7591ef3d9325497b105eb86577c Related: odoo/enterprise#7906 Signed-off-by: Pierre Paridans --- addons/web/static/src/js/views/form/form_renderer.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/addons/web/static/src/js/views/form/form_renderer.js b/addons/web/static/src/js/views/form/form_renderer.js index 4954b3ed3c7..e393f22dba4 100644 --- a/addons/web/static/src/js/views/form/form_renderer.js +++ b/addons/web/static/src/js/views/form/form_renderer.js @@ -249,7 +249,7 @@ var FormRenderer = BasicRenderer.extend({ var $notebook = $(this); var name = $notebook.data('name'); if (name in state) { - var $page = $notebook.find('> ul > li').eq(state[name]); + var $page = $notebook.find('> .o_notebook_headers > .nav-tabs > .nav-item').eq(state[name]); if (!$page.hasClass('o_invisible_modifier')) { $page.find('a[data-toggle="tab"]').click(); } @@ -338,7 +338,6 @@ var FormRenderer = BasicRenderer.extend({ } }, /** - excludedElements: ".o_notebook .nav.nav-tabs", * @private * @param {string} name * @returns {string} @@ -901,9 +900,10 @@ var FormRenderer = BasicRenderer.extend({ }, }); }); + var $notebookHeaders = $('
').append($headers); var $notebook = $('
') .data('name', node.attrs.name || '_default_') - .append($headers, $pages); + .append($notebookHeaders, $pages); this._registerModifiers(node, this.state, $notebook); this._handleAttributes($notebook, node); return $notebook;