From d9a15eaee5c7ead67e1e4ba476d1cd9124b88a14 Mon Sep 17 00:00:00 2001
From: "Xavier Luyckx (xlu)" <xlu@odoo.com>
Date: Mon, 4 Aug 2025 09:54:13 +0200
Subject: [PATCH] [FIX] theme_*: adapt xpath for s_timeline(_images, _list)
 changes

*: theme_artists, theme_buzzy, theme_nano

Before, the timeline (*) dots were invisible when using preset 4
because they shared the same color as the background (`text-o-color-1`
on `o-color-1` background).

The fix in the relative PR implements a solution that ensures dots are
always visible by using button primary colors for all presets.

Thus, this commit adapts xpath to ensure compatibility with the changes.

task-4922302

closes odoo/design-themes#1126

Related: odoo/odoo#221505
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
---
 .../views/snippets/s_timeline_images.xml          | 12 ------------
 theme_buzzy/__manifest__.py                       |  1 -
 theme_buzzy/views/snippets/s_timeline_images.xml  | 15 ---------------
 theme_nano/views/snippets/s_timeline_images.xml   | 14 --------------
 4 files changed, 42 deletions(-)
 delete mode 100644 theme_buzzy/views/snippets/s_timeline_images.xml

diff --git a/theme_artists/views/snippets/s_timeline_images.xml b/theme_artists/views/snippets/s_timeline_images.xml
index 9f5ab6245..0819b719b 100644
--- a/theme_artists/views/snippets/s_timeline_images.xml
+++ b/theme_artists/views/snippets/s_timeline_images.xml
@@ -10,22 +10,10 @@
     <xpath expr="//img" position="attributes">
         <attribute name="class" remove="rounded" separator=" "/>
     </xpath>
-    <xpath expr="//span[hasclass('o_dot')]" position="attributes">
-        <attribute name="class" add="text-o-color-4" remove="text-o-color-1" separator=" "/>
-    </xpath>
-    <xpath expr="//div[hasclass('o_dot_line')]" position="attributes">
-        <attribute name="style">border-color: var(--o-color-1) !important;</attribute>
-    </xpath>
     <!-- Milestone #2 -->
     <xpath expr="(//img)[2]" position="attributes">
         <attribute name="class" remove="rounded" separator=" "/>
     </xpath>
-    <xpath expr="(//span[hasclass('o_dot')])[2]" position="attributes">
-        <attribute name="class" add="text-o-color-4" remove="text-o-color-1" separator=" "/>
-    </xpath>
-    <xpath expr="(//div[hasclass('o_dot_line')])[2]" position="attributes">
-        <attribute name="style">border-color: var(--o-color-1) !important;</attribute>
-    </xpath>
 </template>
 
 </odoo>
diff --git a/theme_buzzy/__manifest__.py b/theme_buzzy/__manifest__.py
index 21cf03b9a..9fe00751d 100644
--- a/theme_buzzy/__manifest__.py
+++ b/theme_buzzy/__manifest__.py
@@ -59,7 +59,6 @@
         'views/snippets/s_image_punchy.xml',
         'views/snippets/s_striped_center_top.xml',
         'views/snippets/s_company_team_card.xml',
-        'views/snippets/s_timeline_images.xml',
         'views/snippets/s_image_title.xml',
         'views/snippets/s_key_images.xml',
         'views/snippets/s_big_number.xml',
diff --git a/theme_buzzy/views/snippets/s_timeline_images.xml b/theme_buzzy/views/snippets/s_timeline_images.xml
deleted file mode 100644
index 3255d4c03..000000000
--- a/theme_buzzy/views/snippets/s_timeline_images.xml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<odoo>
-
-<template id="s_timeline_images" inherit_id="website.s_timeline_images">
-   <!-- Milestone #1-->
-   <xpath expr="//span[hasclass('o_dot')]" position="attributes">
-      <attribute name="class" add="text-o-color-2" remove="text-o-color-1" separator=" "/>
-   </xpath>
-   <!-- Milestone #2 -->
-   <xpath expr="(//span[hasclass('o_dot')])[2]" position="attributes">
-      <attribute name="class" add="text-o-color-2" remove="text-o-color-1" separator=" "/>
-   </xpath>
-</template>
-
-</odoo>
diff --git a/theme_nano/views/snippets/s_timeline_images.xml b/theme_nano/views/snippets/s_timeline_images.xml
index 02091d401..5eabb9d6c 100644
--- a/theme_nano/views/snippets/s_timeline_images.xml
+++ b/theme_nano/views/snippets/s_timeline_images.xml
@@ -6,20 +6,6 @@
     <xpath expr="//section" position="attributes">
         <attribute name="class" add="o_cc o_cc5" separator=" "/>
     </xpath>
-    <!-- Milestone #1-->
-    <xpath expr="//span[hasclass('o_dot')]" position="attributes">
-        <attribute name="class" add="text-o-color-4" remove="text-o-color-1" separator=" "/>
-    </xpath>
-    <xpath expr="//div[hasclass('o_dot_line')]" position="attributes">
-        <attribute name="style">border-color: var(--o-color-1) !important;</attribute>
-    </xpath>
-    <!-- Milestone #2 -->
-    <xpath expr="(//span[hasclass('o_dot')])[2]" position="attributes">
-        <attribute name="class" add="text-o-color-4" remove="text-o-color-1" separator=" "/>
-    </xpath>
-    <xpath expr="(//div[hasclass('o_dot_line')])[2]" position="attributes">
-        <attribute name="style">border-color: var(--o-color-1) !important;</attribute>
-    </xpath>
 </template>
 
 </odoo>