From 72244ca7553113655fe3df57ae889d9809f5e649 Mon Sep 17 00:00:00 2001
From: Paul Morelle <pmo@odoo.com>
Date: Wed, 4 Mar 2020 17:40:36 +0000
Subject: [PATCH] [FIX] website_*: force width of partners logos to 128px
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* = website_crm_partner_assign, website_customer

Follow up of [1] which still had a misalignment for logos higher than large
(portrait), or logos smaller than 128px.

The layout of the related pages relies on the logos being exactly 128px for the
text following the images to be aligned.

There are two issues to fix:

- When images are bigger than 128px, due to incomplete migration at this time:
  `o_image_128_max` is added to ensure they are not displayed bigger than 128px.

- When images are smaller than 128px, due to the original image being too small:
  the parent element of the image is forced to the expected width of 128px.

  The 128px width is actually not forced on the image itself but on the parent
  of the image in order to:

  - avoid a rectangular image (larger than tall) from using 128px height and
    creating empty space when its height is actually smaller,

  - and avoid images smaller than 128px to be scaled up.

Additional fix:

In "Our References" template, remove the `row` class on the container of
media elements, so that all media are always on their own line.

[1] 8748416b4d8ae4840d7738a0edf226f3240d97cb

closes odoo/odoo#47093

X-original-commit: 5e3242f904ddcee9cf660f13b446b88bead35777
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
---
 .../views/website_crm_partner_assign_templates.xml     |  5 +++--
 .../views/website_customer_templates.xml               | 10 ++++++----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/addons/website_crm_partner_assign/views/website_crm_partner_assign_templates.xml b/addons/website_crm_partner_assign/views/website_crm_partner_assign_templates.xml
index ea4d407f600..92df88e2b82 100644
--- a/addons/website_crm_partner_assign/views/website_crm_partner_assign_templates.xml
+++ b/addons/website_crm_partner_assign/views/website_crm_partner_assign_templates.xml
@@ -83,7 +83,8 @@
                         <div class="media mt-3">
                             <a t-attf-href="/partners/#{slug(partner)}?#{current_grade and 'grade_id=%s&amp;' % current_grade.id}#{current_country and 'country_id=%s' % current_country.id}"
                                t-field="partner.image_128"
-                               t-options='{"widget": "image", "class": "mr-3 o_image_128_max"}'
+                               class="mr-3 text-center" style="width: 128px;"
+                               t-options='{"widget": "image", "qweb_img_responsive": False, "class": "o_image_128_max"}'
                             ></a>
                             <div class="media-body o_partner_body" style="min-height: 64px;">
                                 <a t-attf-href="/partners/#{slug(partner)}?#{current_grade and 'grade_id=%s&amp;' % current_grade.id}#{current_country and 'country_id=%s' % current_country.id}">
@@ -164,7 +165,7 @@
     <t t-if="any([p.website_published for p in partner.implemented_partner_ids])">
         <h3 id="references">References</h3>
         <div t-foreach="partner.implemented_partner_ids" t-if="reference.website_published" t-as="reference" class="media mt-3">
-            <span t-field="reference.image_128" t-options='{"widget": "image", "class": "d-block mr-3"}'/>
+            <span t-field="reference.image_128" class="d-block mr-3 text-center" style="width: 128px;" t-options='{"widget": "image", "qweb_img_responsive": False, "class": "o_image_128_max"}'/>
             <div class="media-body" style="min-height: 64px;">
                 <span t-field="reference.self"/>
                 <div t-field='reference.website_short_description'/>
diff --git a/addons/website_customer/views/website_customer_templates.xml b/addons/website_customer/views/website_customer_templates.xml
index 5114b84db8f..32dfb894226 100644
--- a/addons/website_customer/views/website_customer_templates.xml
+++ b/addons/website_customer/views/website_customer_templates.xml
@@ -30,14 +30,15 @@
                             </form>
                         </div>
 
-                        <div class="row">
+                        <div>
 
                     <p t-if="not partners">No result found</p>
                     <t t-foreach="partners" t-as="partner">
                         <div class="media mt-3">
                             <a t-attf-href="/customers/#{slug(partner)}"
                                t-field="partner.image_128"
-                               t-options='{"widget": "image", "class": "mr-3 o_image_128_max"}'
+                               class="d-block mr-3 text-center" style="width: 128px;"
+                               t-options='{"widget": "image", "qweb_img_responsive": False, "class": "o_image_128_max"}'
                             ></a>
                             <div class="media-body" style="min-height: 64px;">
                                 <a t-attf-href="/customers/#{slug(partner)}">
@@ -198,7 +199,8 @@
         </h4>
         <div><a t-attf-href="/partners/#{slug(partner.assigned_partner_id)}"
                 t-field="partner.assigned_partner_id.image_128"
-                t-options='{"widget": "image", "class": "d-block mx-auto"}'
+                class="d-block"
+                t-options='{"widget": "image", "qweb_img_responsive": False, "class": "o_image_128_max"}'
              />
         </div>
         <address class="text-left">
@@ -218,7 +220,7 @@
             <div t-foreach="partner.implemented_partner_ids" t-as="reference" class="media mt-3">
               <t t-if="reference.website_published">
                 <a t-attf-href="/customers/#{slug(reference)}">
-                    <span t-field="reference.image_128" t-options='{"widget": "image", "class": "d-block mr-3 o_image_128_max"}'/>
+                    <span t-field="reference.image_128" class="d-block mr-3 text-center" style="width: 128px;" t-options='{"widget": "image", "qweb_img_responsive": False, "class": "o_image_128_max"}'/>
                 </a>
                 <div class="media-body" style="min-height: 64px;">
                     <a t-attf-href="/customers/#{slug(reference)}">