From 4b360985b698715824bd12dde2722c487272fc53 Mon Sep 17 00:00:00 2001
From: qsm-odoo <qsm@odoo.com>
Date: Tue, 23 Apr 2024 15:43:44 +0200
Subject: [PATCH] [IMP] *: simplify s_hr snippet xml and css

See community commit for details.

closes odoo/design-themes#799

Related: odoo/odoo#162828
Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
---
 theme_bewise/views/customizations.xml         |  8 +++---
 theme_graphene/views/customizations.xml       | 28 +++++++++----------
 theme_kiddo/views/snippets/s_image_text.xml   |  2 +-
 theme_kiddo/views/snippets/s_text_image.xml   |  2 +-
 .../views/snippets/s_three_columns.xml        |  6 ++--
 theme_paptic/views/customizations.xml         |  4 +--
 6 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/theme_bewise/views/customizations.xml b/theme_bewise/views/customizations.xml
index dc0156458..ba40cdbe4 100644
--- a/theme_bewise/views/customizations.xml
+++ b/theme_bewise/views/customizations.xml
@@ -49,8 +49,8 @@
     </xpath>
     <xpath expr="//p[2]" position="replace">
         <p>Journalists, directors, developers, etc. working effectively in their fields of excellence.</p>
-        <div class="s_hr text-start p0 pb32" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr p0 pb32" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//a[hasclass('btn')]" position="attributes">
@@ -75,8 +75,8 @@
     </xpath>
     <xpath expr="//p[2]" position="replace">
         <p>All students benefit from individualised support.</p>
-        <div class="s_hr text-start p0 pb32" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr p0 pb32" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//a[hasclass('btn')]" position="attributes">
diff --git a/theme_graphene/views/customizations.xml b/theme_graphene/views/customizations.xml
index 200b6e15d..1d8773181 100644
--- a/theme_graphene/views/customizations.xml
+++ b/theme_graphene/views/customizations.xml
@@ -42,8 +42,8 @@
     </xpath>
     <xpath expr="//h2" position="replace">
         <h3>Unique experiences to drive engagement</h3>
-        <div class="s_hr text-start pt16 pb0" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: rgba(0, 0, 0, 0) !important;"/>
+        <div class="s_hr pt16 pb0" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: rgba(0, 0, 0, 0) !important;"/>
         </div>
     </xpath>
     <xpath expr="//p[1]" position="replace" mode="inner">
@@ -51,8 +51,8 @@
     </xpath>
     <xpath expr="//p[2]" position="replace">
         <p>We have one goal in mind, the user satisfaction.</p>
-        <div class="s_hr text-start p0 pb32" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr p0 pb32" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//div[hasclass('col-lg-6')][1]" position="attributes">
@@ -84,13 +84,13 @@
         <attribute name="class" add="col-lg-4 offset-lg-1" remove="col-lg-6" separator=" "/>
     </xpath>
     <xpath expr="//h2" position="after">
-        <div class="s_hr text-start pt16 pb0" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: rgba(0, 0, 0, 0) !important;"/>
+        <div class="s_hr pt16 pb0" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: rgba(0, 0, 0, 0) !important;"/>
         </div>
     </xpath>
     <xpath expr="//p[2]" position="after">
-        <div class="s_hr text-start p0 pb32" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr p0 pb32" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//a[hasclass('btn')]" position="attributes">
@@ -164,8 +164,8 @@
 <template id="s_three_columns" inherit_id="website.s_three_columns" name="Graphene s_three_columns">
     <xpath expr="//div[hasclass('row')]/div//h3[1]" position="replace">
         <h5>Powerful API</h5>
-        <div class="s_hr text-start pt8 pb16" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr pt8 pb16" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//div[hasclass('row')]/div//p[1]" position="replace" mode="inner">
@@ -173,8 +173,8 @@
     </xpath>
     <xpath expr="//div[hasclass('row')]/div[2]//h3[1]" position="replace">
         <h5>Mobile Experience</h5>
-        <div class="s_hr text-start pt8 pb16" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr pt8 pb16" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//div[hasclass('row')]/div[2]//p[1]" position="replace" mode="inner">
@@ -182,8 +182,8 @@
     </xpath>
     <xpath expr="//div[hasclass('row')]/div[3]//h3[1]" position="replace">
         <h5>Support Team</h5>
-        <div class="s_hr text-start pt8 pb16" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-width: 1px; border-top-style: solid; border-top-color: var(--o-color-3) !important;"/>
+        <div class="s_hr pt8 pb16" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: var(--o-color-3) !important;"/>
         </div>
     </xpath>
     <xpath expr="//div[hasclass('row')]/div[3]//p[1]" position="replace" mode="inner">
diff --git a/theme_kiddo/views/snippets/s_image_text.xml b/theme_kiddo/views/snippets/s_image_text.xml
index 0b1b94b7a..ef3de3805 100644
--- a/theme_kiddo/views/snippets/s_image_text.xml
+++ b/theme_kiddo/views/snippets/s_image_text.xml
@@ -23,7 +23,7 @@
         Activities
     </xpath>
     <xpath expr="//h2" position="after">
-        <div class="s_hr text-start pb32 pt8" data-snippet="s_hr" data-name="Separator">
+        <div class="s_hr pb32 pt8" data-snippet="s_hr" data-name="Separator">
             <hr class="w-50 me-auto" style="border-top: 4px dotted var(--o-color-2) !important;"/>
         </div>
     </xpath>
diff --git a/theme_kiddo/views/snippets/s_text_image.xml b/theme_kiddo/views/snippets/s_text_image.xml
index efb1b6bac..e350740fb 100644
--- a/theme_kiddo/views/snippets/s_text_image.xml
+++ b/theme_kiddo/views/snippets/s_text_image.xml
@@ -20,7 +20,7 @@
 
     <!-- Separator -->
     <xpath expr="//h2" position="after">
-        <div class="s_hr text-start pb32 pt8" data-snippet="s_hr" data-name="Separator">
+        <div class="s_hr pb32 pt8" data-snippet="s_hr" data-name="Separator">
             <hr class="w-50 me-auto" style="border-top: 4px dotted var(--o-color-1) !important;"/>
         </div>
     </xpath>
diff --git a/theme_kiddo/views/snippets/s_three_columns.xml b/theme_kiddo/views/snippets/s_three_columns.xml
index 2cf36043f..b26894b24 100644
--- a/theme_kiddo/views/snippets/s_three_columns.xml
+++ b/theme_kiddo/views/snippets/s_three_columns.xml
@@ -27,7 +27,7 @@
     </xpath>
     <!-- Column n°1 - Add separator -->
     <xpath expr="//h3" position="after">
-        <div class="s_hr text-start pb32 pt8" data-snippet="s_hr" data-name="Separator">
+        <div class="s_hr pb32 pt8" data-snippet="s_hr" data-name="Separator">
             <hr class="w-25 mx-auto" style="border-top: 5px dotted var(--o-color-2) !important;"/>
         </div>
     </xpath>
@@ -59,7 +59,7 @@
     </xpath>
     <!-- Column n°2 - Add separator -->
     <xpath expr="(//h3)[2]" position="after">
-        <div class="s_hr text-start pb32 pt8" data-snippet="s_hr" data-name="Separator">
+        <div class="s_hr pb32 pt8" data-snippet="s_hr" data-name="Separator">
             <hr class="w-25 mx-auto" style="border-top: 5px dotted var(--o-color-1) !important;"/>
         </div>
     </xpath>
@@ -91,7 +91,7 @@
     </xpath>
     <!-- Column n°3 - Add separator -->
     <xpath expr="(//h3)[3]" position="after">
-        <div class="s_hr text-start pb32 pt8" data-snippet="s_hr" data-name="Separator">
+        <div class="s_hr pb32 pt8" data-snippet="s_hr" data-name="Separator">
             <hr class="w-25 mx-auto" style="border-top: 5px dotted var(--o-color-2) !important;"/>
         </div>
     </xpath>
diff --git a/theme_paptic/views/customizations.xml b/theme_paptic/views/customizations.xml
index e7066e1dd..bb4d6e3e4 100644
--- a/theme_paptic/views/customizations.xml
+++ b/theme_paptic/views/customizations.xml
@@ -99,8 +99,8 @@
         <p>&#160;</p>
         <h5>Scalable and Modular</h5>
         <p>Whether you start small or are already a large company, our solution adapts to your needs and grows with you.</p>
-        <div class="s_hr text-start pb16 pt8" data-snippet="s_hr" data-name="Separator">
-            <hr class="w-100 mx-auto" style="border-top-style: solid; border-top-color: rgb(233, 236, 239) !important; border-top-width: 3px !important;"/>
+        <div class="s_hr pb16 pt8" data-snippet="s_hr" data-name="Separator">
+            <hr class="w-100 mx-auto" style="border-top-color: rgb(233, 236, 239) !important; border-top-width: 3px !important;"/>
         </div>
     </xpath>