From a5b4acdbdf3a1fe57cea02ad3dee8d181f42b4b8 Mon Sep 17 00:00:00 2001
From: "stefanorigano (SRI)" <sri@odoo.com>
Date: Wed, 24 Jul 2024 13:55:31 +0200
Subject: [PATCH] [IMP] theme_*: `s_comparison` redesign adaptations

*: theme_anelusia, theme_artists, theme_beauty, theme_bewise,
   theme_bookstore, theme_buzzy, theme_clean, theme_graphene,
   theme_loftspace, theme_odoo_experts, theme_paptic, theme_treehouse

task-3665304
part of task-3619705

closes odoo/design-themes#842

Related: odoo/odoo#174299
Signed-off-by: Benoit Socias (bso) <bso@odoo.com>
---
 .../views/snippets/s_comparisons.xml          | 21 --------------
 theme_artists/views/new_page_template.xml     |  4 +--
 .../views/snippets/s_comparisons.xml          | 28 -------------------
 theme_beauty/views/snippets/s_comparisons.xml |  7 -----
 theme_bewise/views/customizations.xml         |  4 +--
 .../views/snippets/s_comparisons.xml          |  4 +--
 theme_buzzy/views/snippets/s_comparisons.xml  | 10 +++----
 theme_clean/views/snippets/s_comparisons.xml  |  8 ------
 theme_graphene/views/customizations.xml       | 20 +------------
 .../views/snippets/s_comparisons.xml          | 12 ++------
 .../views/snippets/s_comparisons.xml          | 11 +++-----
 theme_paptic/views/customizations.xml         |  5 ----
 .../views/snippets/s_comparisons.xml          |  2 +-
 13 files changed, 19 insertions(+), 117 deletions(-)

diff --git a/theme_anelusia/views/snippets/s_comparisons.xml b/theme_anelusia/views/snippets/s_comparisons.xml
index b9bd565be..23bbd9b79 100644
--- a/theme_anelusia/views/snippets/s_comparisons.xml
+++ b/theme_anelusia/views/snippets/s_comparisons.xml
@@ -6,27 +6,6 @@
     <xpath expr="//section" position="attributes">
         <attribute name="class" add="o_cc o_cc4" separator=" "/>
     </xpath>
-    <!-- Card #1 -->
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-black-15" remove="bg-200" separator=" "/>
-    </xpath>
-    <xpath expr="//a[hasclass('btn')]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
-    <!-- Card #2 -->
-    <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-black-15" remove="bg-o-color-5" separator=" "/>
-    </xpath>
-    <xpath expr="(//a[hasclass('btn')])[2]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
-    <!-- Card #3 -->
-    <xpath expr="(//div[hasclass('card')])[3]" position="attributes">
-        <attribute name="class" add="bg-black-15" remove="bg-o-color-2" separator=" "/>
-    </xpath>
-    <xpath expr="(//a[hasclass('btn')])[3]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
 </template>
 
 </odoo>
diff --git a/theme_artists/views/new_page_template.xml b/theme_artists/views/new_page_template.xml
index 512cea92a..e2b7ba7b6 100644
--- a/theme_artists/views/new_page_template.xml
+++ b/theme_artists/views/new_page_template.xml
@@ -43,8 +43,8 @@
 </template>
 
 <template id="new_page_template_s_comparisons" inherit_id="website.new_page_template_s_comparisons">
-    <xpath expr="(//div[hasclass('card')])[3]" position="attributes">
-        <attribute name="class" add="bg-o-color-2" remove="bg-o-color-1" separator=" "/>
+    <xpath expr="(//a[hasclass('btn')])[3]" position="attributes">
+        <attribute name="class" add="btn-success" remove="btn-outline-primary" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_artists/views/snippets/s_comparisons.xml b/theme_artists/views/snippets/s_comparisons.xml
index 80524fb7d..be0a7532b 100644
--- a/theme_artists/views/snippets/s_comparisons.xml
+++ b/theme_artists/views/snippets/s_comparisons.xml
@@ -6,34 +6,6 @@
     <xpath expr="//section" position="attributes">
         <attribute name="class" add="o_cc o_cc5 o_colored_level" separator=" "/>
     </xpath>
-    <!-- Card #1 -->
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-200" separator=" "/>
-    </xpath>
-    <!-- Card #1 - First list item -->
-    <xpath expr="//li[hasclass('list-group-item')]" position="replace" mode="inner">
-        Basic sales &amp; marketing
-    </xpath>
-    <!-- Card #1 - Button -->
-    <xpath expr="//a[hasclass('btn')]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
-    <!-- Card #2 -->
-    <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-o-color-5" separator=" "/>
-    </xpath>
-    <!-- Card #2 - Button -->
-    <xpath expr="(//a[hasclass('btn')])[2]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
-    <!-- Card #3 -->
-    <xpath expr="(//div[hasclass('card')])[3]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-o-color-2" separator=" "/>
-    </xpath>
-    <!-- Card #3 - Button -->
-    <xpath expr="(//a[hasclass('btn')])[3]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
-    </xpath>
 </template>
 
 </odoo>
diff --git a/theme_beauty/views/snippets/s_comparisons.xml b/theme_beauty/views/snippets/s_comparisons.xml
index 871585a87..9f1321658 100644
--- a/theme_beauty/views/snippets/s_comparisons.xml
+++ b/theme_beauty/views/snippets/s_comparisons.xml
@@ -2,14 +2,7 @@
 <odoo>
 
 <template id="s_comparisons" inherit_id="website.s_comparisons">
-    <!-- Card #1 -->
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-3" remove="bg-200" separator=" "/>
-    </xpath>
     <!-- Card #2 -->
-    <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-o-color-5" separator=" "/>
-    </xpath>
     <xpath expr="(//div[hasclass('card')])[2]//a[hasclass('btn')]" position="attributes">
         <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
     </xpath>
diff --git a/theme_bewise/views/customizations.xml b/theme_bewise/views/customizations.xml
index 825bf1f1f..b7a5bdd40 100644
--- a/theme_bewise/views/customizations.xml
+++ b/theme_bewise/views/customizations.xml
@@ -374,8 +374,8 @@
 
 <!-- ======== COMPARISONS ======== -->
 <template id="s_comparisons" inherit_id="website.s_comparisons" name="Be Wise s_comparisons">
-    <xpath expr="//div[hasclass('card')]//li[hasclass('list-group-item')]" position="replace" mode="inner">
-        Basic sales &amp; marketing
+    <xpath expr="(//a[hasclass('btn')])[3]" position="attributes">
+        <attribute name="class" add="btn-success" remove="btn-outline-primary" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_bookstore/views/snippets/s_comparisons.xml b/theme_bookstore/views/snippets/s_comparisons.xml
index 45ed07624..6b57deba8 100644
--- a/theme_bookstore/views/snippets/s_comparisons.xml
+++ b/theme_bookstore/views/snippets/s_comparisons.xml
@@ -4,11 +4,11 @@
 <template id="s_comparisons" inherit_id="website.s_comparisons">
     <!-- Left card -->
     <xpath expr="//div[hasclass('row')]//div" position="attributes">
-        <attribute name="class" add="pt64" remove="pt32" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
     <!-- Right card -->
     <xpath expr="//div[hasclass('row')]//div[3]" position="attributes">
-        <attribute name="class" add="pt64" remove="pt32" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_buzzy/views/snippets/s_comparisons.xml b/theme_buzzy/views/snippets/s_comparisons.xml
index 9a22c9250..9abeed51f 100644
--- a/theme_buzzy/views/snippets/s_comparisons.xml
+++ b/theme_buzzy/views/snippets/s_comparisons.xml
@@ -8,21 +8,21 @@
     </xpath>
     <!-- Box #1 -->
     <xpath expr="//div[hasclass('row')]//div" position="attributes">
-        <attribute name="class" add="pt88" remove="pt32" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
     <xpath expr="//div[hasclass('row')]//div/div" position="attributes">
-        <attribute name="class" add="shadow bg-o-color-5" remove="bg-200" separator=" "/>
+        <attribute name="class" add="shadow" separator=" "/>
     </xpath>
     <!-- Box #2 -->
     <xpath expr="//div[hasclass('row')]//div[2]/div" position="attributes">
-        <attribute name="class" add="shadow bg-o-color-1" remove="bg-o-color-5" separator=" "/>
+        <attribute name="class" add="shadow" separator=" "/>
     </xpath>
     <!-- Box #3 -->
     <xpath expr="//div[hasclass('row')]//div[3]" position="attributes">
-        <attribute name="class" add="pt88" remove="pt32" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
     <xpath expr="//div[hasclass('row')]//div[3]/div" position="attributes">
-        <attribute name="class" add="shadow bg-o-color-5" remove="bg-o-color-2" separator=" "/>
+        <attribute name="class" add="shadow" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_clean/views/snippets/s_comparisons.xml b/theme_clean/views/snippets/s_comparisons.xml
index 809ededcd..cb8604c4a 100644
--- a/theme_clean/views/snippets/s_comparisons.xml
+++ b/theme_clean/views/snippets/s_comparisons.xml
@@ -6,14 +6,6 @@
     <xpath expr="//section" position="attributes">
         <attribute name="class" add="o_cc o_cc2 o_colored_level" separator=" "/>
     </xpath>
-    <!-- Left card -->
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-2" remove="bg-200" separator=" "/>
-    </xpath>
-    <!-- Middle card -->
-    <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-o-color-2" remove="bg-o-color-5" separator=" "/>
-    </xpath>
 </template>
 
 </odoo>
diff --git a/theme_graphene/views/customizations.xml b/theme_graphene/views/customizations.xml
index 83a4fd363..2c463d165 100644
--- a/theme_graphene/views/customizations.xml
+++ b/theme_graphene/views/customizations.xml
@@ -210,25 +210,7 @@
 <!-- ======== COMPARISON ======== -->
 <template id="s_comparisons" inherit_id="website.s_comparisons" name="Graphene s_comparisons">
     <xpath expr="//section" position="attributes">
-        <attribute name="class" add="pb48 pt24 o_cc o_cc4" remove="pt32 pb32" separator=" "/>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div//a" position="replace">
-        <a href="/contactus" class="btn btn-primary">Start Now</a>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div[2]//a" position="replace">
-        <a href="/contactus" class="btn btn-primary">Start Now</a>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div[2]//li[2]" position="replace">
-        <li class="list-group-item">Access to all modules</li>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div[3]//a" position="replace">
-        <a href="/contactus" class="btn btn-primary">Start Now</a>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div[3]//li[2]" position="replace">
-        <li class="list-group-item">Access to all modules and features</li>
-    </xpath>
-    <xpath expr="//div[hasclass('row')]/div[3]//li[4]" position="replace">
-        <li class="list-group-item">24/7 support</li>
+        <attribute name="class" add="pt24 o_cc o_cc4" remove="pt48" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_loftspace/views/snippets/s_comparisons.xml b/theme_loftspace/views/snippets/s_comparisons.xml
index a9d65f9f1..4137c1a13 100644
--- a/theme_loftspace/views/snippets/s_comparisons.xml
+++ b/theme_loftspace/views/snippets/s_comparisons.xml
@@ -4,19 +4,11 @@
 <template id="s_comparisons" inherit_id="website.s_comparisons">
     <!-- Card #1 -->
     <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-3" remove="bg-200" separator=" "/>
-    </xpath>
-    <!-- Card #2 -->
-    <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-o-color-5" separator=" "/>
-    </xpath>
-    <!-- Card #2 - button -->
-    <xpath expr="(//div[hasclass('card')])[2]//a[hasclass('btn')]" position="attributes">
-        <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
+        <attribute name="class" add="bg-o-color-3" separator=" "/>
     </xpath>
     <!-- Card #3 -->
     <xpath expr="(//div[hasclass('card')])[3]" position="attributes">
-        <attribute name="class" add="bg-o-color-3" remove="bg-o-color-2" separator=" "/>
+        <attribute name="class" add="bg-o-color-3" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_odoo_experts/views/snippets/s_comparisons.xml b/theme_odoo_experts/views/snippets/s_comparisons.xml
index d3f0ffafe..40ca05b67 100644
--- a/theme_odoo_experts/views/snippets/s_comparisons.xml
+++ b/theme_odoo_experts/views/snippets/s_comparisons.xml
@@ -4,24 +4,21 @@
 <template id="s_comparisons" inherit_id="website.s_comparisons">
     <!-- Box #01 -->
     <xpath expr="//div[hasclass('row')]//div" position="attributes">
-        <attribute name="class" add="pt80" remove="pt32" separator=" "/>
-    </xpath>
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-3" remove="bg-200" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
     <!-- Box #02 -->
     <xpath expr="(//div[hasclass('card')])[2]" position="attributes">
-        <attribute name="class" add="bg-o-color-1" remove="bg-o-color-5" separator=" "/>
+        <attribute name="class" add="bg-o-color-1" separator=" "/>
     </xpath>
     <xpath expr="(//div[hasclass('card')])[2]//a[hasclass('btn')]" position="attributes">
         <attribute name="class" add="btn-secondary" remove="btn-primary" separator=" "/>
     </xpath>
     <!-- Box #03 -->
     <xpath expr="//div[hasclass('row')]//div[3]" position="attributes">
-        <attribute name="class" add="pt80" remove="pt32" separator=" "/>
+        <attribute name="class" add="pt16" separator=" "/>
     </xpath>
     <xpath expr="(//div[hasclass('card')])[3]" position="attributes">
-        <attribute name="class" add="bg-o-color-3" remove="bg-o-color-2" separator=" "/>
+        <attribute name="class" add="bg-o-color-3" separator=" "/>
     </xpath>
 </template>
 
diff --git a/theme_paptic/views/customizations.xml b/theme_paptic/views/customizations.xml
index 0b26568fd..4292d0e23 100644
--- a/theme_paptic/views/customizations.xml
+++ b/theme_paptic/views/customizations.xml
@@ -283,17 +283,12 @@
 <!-- ==== Comparisons ===== -->
 <template id="s_comparisons" inherit_id="website.s_comparisons" name="Paptic s_comparisons">
     <xpath expr="//section" position="attributes">
-        <attribute name="class" add="pt0 pb80" remove="pt32 pb32" separator=" "/>
         <attribute name="data-oe-shape-data">{"shape":"web_editor/Floats/03","flip":[]}</attribute>
     </xpath>
     <!-- Shape -->
     <xpath expr="//div[hasclass('container')]" position="before">
         <div class="o_we_shape o_web_editor_Floats_03"/>
     </xpath>
-    <!-- Left card -->
-    <xpath expr="//div[hasclass('card')]" position="attributes">
-        <attribute name="class" add="bg-o-color-2" remove="bg-200" separator=" "/>
-    </xpath>
 </template>
 
 <!-- ==== Call to action ===== -->
diff --git a/theme_treehouse/views/snippets/s_comparisons.xml b/theme_treehouse/views/snippets/s_comparisons.xml
index e4a2a1c53..e662a2bcd 100644
--- a/theme_treehouse/views/snippets/s_comparisons.xml
+++ b/theme_treehouse/views/snippets/s_comparisons.xml
@@ -4,7 +4,7 @@
 <template id="s_comparisons" inherit_id="website.s_comparisons">
     <!-- Section -->
     <xpath expr="//section" position="attributes">
-        <attribute name="class" add="o_cc o_cc2 pt64 pb64" remove="pt32 pb32" separator=" "/>
+        <attribute name="class" add="o_cc o_cc2" separator=" "/>
     </xpath>
 </template>