From 3ae39a1f2a52df8bc876ed434de83399cb5de02e Mon Sep 17 00:00:00 2001
From: "stefanorigano (SRI)" <sri@odoo.com>
Date: Wed, 13 Mar 2024 10:55:15 +0100
Subject: [PATCH] [FIX] website_event: `modal_ticket_registration` adjustments

This commit addresses various minor responsiveness and design issues in
the "event registration modal." Additionally, it modifies the condition
for rendering prices range and to display it if more than two
tickets exist only.

task-3718417
part of task-3698364

Part-of: odoo/odoo#152434
---
 .../views/event_templates_page_registration.xml        | 10 +++++-----
 .../views/website_event_templates.xml                  |  6 +++---
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/addons/website_event/views/event_templates_page_registration.xml b/addons/website_event/views/event_templates_page_registration.xml
index 8007ca7d708..609c63869c5 100644
--- a/addons/website_event/views/event_templates_page_registration.xml
+++ b/addons/website_event/views/event_templates_page_registration.xml
@@ -351,7 +351,7 @@
             <div class="modal-content">
             <div class="modal-header">
                 <div class="o_wevent_registration_title modal-title fs-5" id="staticBackdropLabel">Tickets</div>
-                <div t-if="len(event.event_ticket_ids) &gt; 1" class="o_wevent_price_range ms-2"/>
+                <div t-if="len(event.event_ticket_ids) &gt; 2" class="o_wevent_price_range ms-2"/>
                 <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
             </div>
 
@@ -374,10 +374,10 @@
                         </span>
                         <div id="o_wevent_tickets_collapse" class="modal-body collapse show">
                             <div t-foreach="tickets" t-as="ticket"
-                                class="d-flex justify-content-between o_wevent_ticket_selector"
+                                t-attf-class="d-flex justify-content-between o_wevent_ticket_selector mb-2 pb-2 {{not ticket_last and 'border-bottom' or ''}}"
                                 t-att-name="ticket.name">
                                 <div itemscope="itemscope" itemtype="http://schema.org/Offer">
-                                    <h5 itemprop="name" t-field="ticket.name" class="my-0"/>
+                                    <h5 itemprop="name" t-field="ticket.name" class="h6 my-0"/>
                                     <t t-if="ticket.description">
                                         <small t-field="ticket.description" class="text-muted py-2"/>
                                         <br/>
@@ -395,8 +395,8 @@
                                         (<span t-out="ticket.event_id.date_tz"/>)
                                     </small>
                                 </div>
-                                <div class="d-flex align-items-center justify-content-between p-0">
-                                    <div class="o_wevent_registration_multi_select flex-md-grow-1 me-2 text-end"/>
+                                <div class="d-flex flex-column flex-md-row align-items-center justify-content-between gap-2">
+                                    <div class="o_wevent_registration_multi_select flex-md-grow-1 text-end"/>
                                     <div class="ms-auto">
                                         <select t-if="not ticket.is_expired and ticket.sale_available"
                                             t-attf-name="nb_register-#{ticket.id}"
diff --git a/addons/website_event_sale/views/website_event_templates.xml b/addons/website_event_sale/views/website_event_templates.xml
index 465b34cbb64..dab3ebb4ffc 100644
--- a/addons/website_event_sale/views/website_event_templates.xml
+++ b/addons/website_event_sale/views/website_event_templates.xml
@@ -26,17 +26,17 @@
                      t-options="{'widget': 'monetary', 'from_currency': event.company_id.sudo().currency_id, 'display_currency': website.currency_id}"/>
             </t>
             <span t-if="website.show_line_subtotals_tax_selection == 'tax_excluded'"
-                  class="badge text-bg-secondary fs-6"
+                  class="fs-6"
                   t-field="ticket.price_reduce"
                   t-options="{'widget': 'monetary', 'from_currency': event.company_id.sudo().currency_id, 'display_currency': website.currency_id}"/>
             <span t-else=""
                   t-field="ticket.price_reduce_taxinc"
-                  class="badge text-bg-secondary fs-6"
+                  class="fs-6"
                   t-options="{'widget': 'monetary', 'from_currency': event.company_id.sudo().currency_id, 'display_currency': website.currency_id}"/>
             <span itemprop="price" class="d-none" t-out="ticket.price"/>
             <span itemprop="priceCurrency" class="d-none" t-out="website.currency_id.name"/>
         </t>
-        <span t-else="" class="badge text-bg-secondary text-uppercase fs-6">Free</span>
+        <span t-else="" class="badge text-bg-success rounded-pill p-2 px-3">Free</span>
     </xpath>
     <xpath expr="//div[hasclass('o_wevent_price_range')]" position="inside">
         <t t-if="website.show_line_subtotals_tax_selection == 'tax_excluded'" t-set="all_prices" t-value="event.event_ticket_ids.mapped('price_reduce')"/>