From 28b0168463e6f992d888a04be9743e9ce612f307 Mon Sep 17 00:00:00 2001
From: Elisabeth Dickinson <edi@odoo.com>
Date: Fri, 11 Aug 2023 07:28:23 +0000
Subject: [PATCH] [IMP] web: truncate date in list view

When the width of the column is too small to fit the date, the date was
cut by the following column instead of having its content
truncated. This could be seen on the "Creation date" in the list view of
Sales Quotations.

task-3463329
part of task-3326263

closes odoo/odoo#144039

X-original-commit: b9675c38b49d99e00e401dbdb90445354aab43c3
Related: odoo/enterprise#51684
Signed-off-by: Pierre Paridans (app) <app@odoo.com>
---
 .../static/src/views/fields/datetime/datetime_field.xml   | 4 ++--
 .../static/tests/views/fields/daterange_field_tests.js    | 8 ++++----
 addons/web/static/tests/views/list_view_tests.js          | 8 ++++----
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/addons/web/static/src/views/fields/datetime/datetime_field.xml b/addons/web/static/src/views/fields/datetime/datetime_field.xml
index 9fb5f61f58b..3dacd40d27f 100644
--- a/addons/web/static/src/views/fields/datetime/datetime_field.xml
+++ b/addons/web/static/src/views/fields/datetime/datetime_field.xml
@@ -5,7 +5,7 @@
         <div class="d-flex gap-2 align-items-center" t-ref="root">
             <!-- Start date -->
             <t t-if="props.readonly">
-                <t t-esc="getFormattedValue(0)" />
+                <span class="text-truncate" t-esc="getFormattedValue(0)" />
             </t>
             <t t-elif="!props.required and isEmpty(startDateField) and !isEmpty(endDateField)">
                 <button
@@ -42,7 +42,7 @@
             <!-- End date -->
             <t t-if="endDateField">
                 <t t-if="props.readonly">
-                    <t t-esc="getFormattedValue(1)" />
+                    <span class="text-truncate" t-esc="getFormattedValue(1)" />
                 </t>
                 <t t-elif="!props.required and !isEmpty(startDateField) and isEmpty(endDateField)">
                     <button
diff --git a/addons/web/static/tests/views/fields/daterange_field_tests.js b/addons/web/static/tests/views/fields/daterange_field_tests.js
index 57c52ac00b9..e10a5c4a268 100644
--- a/addons/web/static/tests/views/fields/daterange_field_tests.js
+++ b/addons/web/static/tests/views/fields/daterange_field_tests.js
@@ -1126,10 +1126,10 @@ QUnit.module("Fields", (hooks) => {
             .map((node) => {
                 if (node === arrowIcon) {
                     return "->";
-                } else if (node.nodeType === 3) {
+                } else if (node.nodeType === Node.TEXT_NODE) {
                     return node.nodeValue.trim();
                 } else {
-                    return false;
+                    return node.innerText?.trim();
                 }
             })
             .filter(Boolean);
@@ -1157,10 +1157,10 @@ QUnit.module("Fields", (hooks) => {
             .map((node) => {
                 if (node === arrowIcon) {
                     return "->";
-                } else if (node.nodeType === 3) {
+                } else if (node.nodeType === Node.TEXT_NODE) {
                     return node.nodeValue.trim();
                 } else {
-                    return false;
+                    return node.innerText?.trim();
                 }
             })
             .filter(Boolean);
diff --git a/addons/web/static/tests/views/list_view_tests.js b/addons/web/static/tests/views/list_view_tests.js
index 630b210921c..15b0dfec0af 100644
--- a/addons/web/static/tests/views/list_view_tests.js
+++ b/addons/web/static/tests/views/list_view_tests.js
@@ -11572,10 +11572,10 @@ QUnit.module("Views", (hooks) => {
             .map((node) => {
                 if (node === arrowIcon) {
                     return "->";
-                } else if (node.nodeType === 3) {
+                } else if (node.nodeType === Node.TEXT_NODE) {
                     return node.nodeValue.trim();
                 } else {
-                    return false;
+                    return node.innerText?.trim();
                 }
             })
             .filter(Boolean);
@@ -11603,10 +11603,10 @@ QUnit.module("Views", (hooks) => {
             .map((node) => {
                 if (node === arrowIcon) {
                     return "->";
-                } else if (node.nodeType === 3) {
+                } else if (node.nodeType === Node.TEXT_NODE) {
                     return node.nodeValue.trim();
                 } else {
-                    return false;
+                    return node.innerText?.trim();
                 }
             })
             .filter(Boolean);