From b79f7f8525f7d0e4976e3ee4bc2818a80455c3f1 Mon Sep 17 00:00:00 2001
From: Fabien Pinckaers <fp@openerp.com>
Date: Thu, 22 Feb 2024 10:19:25 +0530
Subject: [PATCH] [IMP] product_document: order by name to control on ecommerce
 appearance

Order name rather than ID so that the user has more control on documents
published in eCommerce and quote builder. Minor CSS cleanup.

closes odoo/odoo#154961

Signed-off-by: Fabien Pinckaers (fp) <fp@odoo.com>
---
 addons/product/models/product_document.py | 2 +-
 addons/website_sale/views/templates.xml   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/addons/product/models/product_document.py b/addons/product/models/product_document.py
index 58f9aab0336..6b5cdc503d0 100644
--- a/addons/product/models/product_document.py
+++ b/addons/product/models/product_document.py
@@ -10,7 +10,7 @@ class ProductDocument(models.Model):
     _inherits = {
         'ir.attachment': 'ir_attachment_id',
     }
-    _order = 'id desc'
+    _order = 'name'
 
     ir_attachment_id = fields.Many2one(
         'ir.attachment',
diff --git a/addons/website_sale/views/templates.xml b/addons/website_sale/views/templates.xml
index 4fa73c1951e..7eb8d0b362a 100644
--- a/addons/website_sale/views/templates.xml
+++ b/addons/website_sale/views/templates.xml
@@ -1136,13 +1136,13 @@
                                 </table>
                             </div>
                             <t t-set="product_documents" t-value="product.sudo().product_document_ids.filtered(lambda doc: doc.shown_on_product_page)"/>
-                            <div id="product_documents" t-if="product_documents" class="row">
-                                <u class="h5">Documents</u>
+                            <div id="product_documents" class="my-2" t-if="product_documents">
+                                <h5>Documents</h5>
                                 <t t-foreach="product_documents" t-as="document_sudo">
                                     <t t-set="attachment_sudo" t-value="document_sudo.ir_attachment_id"/>
                                     <t t-set="target" t-value="attachment_sudo.type == 'url' and '_blank' or '_self'"/>
                                     <t t-set="icon" t-value="attachment_sudo.type == 'url' and 'fa-link' or 'fa-download'"/>
-                                    <div class="d-flex mb-2 mt-2">
+                                    <div>
                                         <a t-att-href="'/shop/' + slug(product) + '/document/' + str(document_sudo.id)" t-att-target="target">
                                             <i t-att-class="'fa ' + icon"/>
                                             <t t-out="attachment_sudo.name"/>