From f66eaebc70d88b9eebf5c63f5b69eb622a45a1c0 Mon Sep 17 00:00:00 2001
From: "Julien Carion (juca)" <juca@odoo.com>
Date: Tue, 19 Mar 2024 15:06:36 +0100
Subject: [PATCH] [IMP] web: pivot: keep horizontal scrollbar inside viewport

This commit simply removes the table-responsive class from the pivot
view in desktop mode so that its eventual horizontal scrollbar will
remain inside the viewport instead of being positioned at the very
bottom of the page. Also hides the scrollbar in sample data mode so that
the user cannot scroll horizontally in this case which introduces weird
display.

closes odoo/odoo#158083

Signed-off-by: Romeo Fragomeli (rfr) <rfr@odoo.com>
---
 addons/web/static/src/views/pivot/pivot_renderer.xml | 2 +-
 addons/web/static/src/views/pivot/pivot_view.scss    | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/addons/web/static/src/views/pivot/pivot_renderer.xml b/addons/web/static/src/views/pivot/pivot_renderer.xml
index d46f787369c..3bcc1de5d35 100644
--- a/addons/web/static/src/views/pivot/pivot_renderer.xml
+++ b/addons/web/static/src/views/pivot/pivot_renderer.xml
@@ -3,7 +3,7 @@
 
     <t t-name="web.PivotRenderer">
         <t t-call="{{ props.buttonTemplate }}"/>
-        <div t-if="model.hasData() and model.metaData.activeMeasures.length" class="o_pivot table-responsive mx-3">
+        <div t-if="model.hasData() and model.metaData.activeMeasures.length" t-att-class="env.isSmall ? 'table-responsive' : ''" class="o_pivot mx-3">
             <table
                 class="table-hover table table-sm table-bordered table-borderless"
                 t-att-class="{ o_enable_linking: !model.metaData.disableLinking }"
diff --git a/addons/web/static/src/views/pivot/pivot_view.scss b/addons/web/static/src/views/pivot/pivot_view.scss
index 8d066fa4dd4..f2e88afcec3 100644
--- a/addons/web/static/src/views/pivot/pivot_view.scss
+++ b/addons/web/static/src/views/pivot/pivot_view.scss
@@ -39,7 +39,8 @@
 
 // ------- Sample mode -------
 .o_pivot_view .o_view_sample_data {
-    table {
+    overflow: hidden !important;
+    .o_pivot {
         @include o-sample-data-disabled;
     }
 }