From 37bf69dd9dabf2b4be342a49ddf02e9422a8a919 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C3=ABl=20Mattiello?= <mcm@odoo.com>
Date: Tue, 3 Sep 2019 10:04:18 +0000
Subject: [PATCH] [FIX] website_slides: fixes some display issues in eLearning
 frontend

Fix some display issues in eLearning

  * fix the share button in documentation. It was not working as model was
    not correctly placed;
  * fix top3 users banners broken in mobile;
  * fix left menu of fullscreen being behind content on mobile (image, video);
  * removed upload button in training channel aside as there are already
    buttons available for that purpose;

Task 2049640 (website slides back2basics and fiximp)
---
 .../src/scss/slides_slide_fullscreen.scss     |  1 +
 .../views/website_slides_templates_course.xml | 72 ++++++++-----------
 .../website_slides_templates_homepage.xml     |  4 +-
 3 files changed, 34 insertions(+), 43 deletions(-)

diff --git a/addons/website_slides/static/src/scss/slides_slide_fullscreen.scss b/addons/website_slides/static/src/scss/slides_slide_fullscreen.scss
index 1b7ff14ae61..a6c406e5242 100644
--- a/addons/website_slides/static/src/scss/slides_slide_fullscreen.scss
+++ b/addons/website_slides/static/src/scss/slides_slide_fullscreen.scss
@@ -35,6 +35,7 @@
     .o_wslides_fs_sidebar {
         background-image: linear-gradient(160deg, $o-wslides-color-dark1, $o-wslides-color-dark2);
         position: relative;
+        z-index: $zindex-fixed;
 
         .o_wslides_fs_sidebar_content {
             min-width: $o-wslides-fs-side-width;
diff --git a/addons/website_slides/views/website_slides_templates_course.xml b/addons/website_slides/views/website_slides_templates_course.xml
index e4b1bb8b7f2..494b4c78b37 100644
--- a/addons/website_slides/views/website_slides_templates_course.xml
+++ b/addons/website_slides/views/website_slides_templates_course.xml
@@ -237,6 +237,37 @@
                 </t>
             </div>
         </div>
+
+        <div class="modal fade" id="slideChannelShareModal" tabindex="-1" role="dialog" aria-labelledby="slideChannelShareModalLabel" aria-hidden="true">
+            <div class="modal-dialog" role="document">
+                <div class="modal-content">
+                    <div class="modal-header">
+                        <h5 class="modal-title" id="slideChannelShareModalLabel">
+                            Share
+                        </h5>
+                        <button type="button" class="close" data-dismiss="modal" aria-label="Close">
+                            <span>&amp;times;</span>
+                        </button>
+                    </div>
+                    <div class="modal-body">
+                        <h5>Share on Social Networks</h5>
+                        <t t-call="website_slides.slides_share">
+                            <t t-set="record" t-value="channel"/>
+                        </t>
+                        <h5 class="mt16">Share Link</h5>
+                        <div class="input-group">
+                            <input type="text" class="form-control o_wslides_js_share_link" readonly="readonly" onclick="this.select();"
+                                t-att-value="channel.website_url"/>
+                            <div class="input-group-append">
+                                <button class="btn btn-sm btn-primary o_clipboard_button" >
+                                    <span class="fa fa-clipboard"> Copy Text</span>
+                                </button>
+                            </div>
+                        </div>
+                    </div>
+                </div>
+            </div>
+        </div>
     </t>
 </template>
 
@@ -299,47 +330,6 @@
                     data-toggle="modal" data-target="#slideChannelShareModal">
                     <i class="fa fa-share-square fa-fw"/> Share
                 </button>
-                <div class="modal fade" id="slideChannelShareModal" tabindex="-1" role="dialog" aria-labelledby="slideChannelShareModalLabel" aria-hidden="true">
-                    <div class="modal-dialog" role="document">
-                        <div class="modal-content">
-                            <div class="modal-header">
-                                <h5 class="modal-title" id="slideChannelShareModalLabel">
-                                    Share
-                                </h5>
-                                <button type="button" class="close" data-dismiss="modal" aria-label="Close">
-                                    <span>&amp;times;</span>
-                                </button>
-                            </div>
-                            <div class="modal-body">
-                                <h5>Share on Social Networks</h5>
-                                <t t-call="website_slides.slides_share">
-                                    <t t-set="record" t-value="channel"/>
-                                </t>
-                                <h5 class="mt16">Share Link</h5>
-                                <div class="input-group">
-                                    <input type="text" class="form-control o_wslides_js_share_link" readonly="readonly" onclick="this.select();"
-                                        t-att-value="channel.website_url"/>
-                                    <div class="input-group-append">
-                                        <button class="btn btn-sm btn-primary o_clipboard_button" >
-                                            <span class="fa fa-clipboard"> Copy Text</span>
-                                        </button>
-                                    </div>
-                                </div>
-                            </div>
-                        </div>
-                    </div>
-                </div>
-            </div>
-
-            <div t-if="channel.can_upload" class="justify-content-center mt-1">
-                <a role="button" class="btn btn-link btn-block o_wslides_js_slide_upload" title="Upload Presentation"
-                    aria-label="Upload Presentation" href="#"
-                    t-att-data-modules-to-install="modules_to_install"
-                    t-att-data-channel-id="channel.id"
-                    t-att-data-can-upload="channel.can_upload"
-                    t-att-data-can-publish="channel.can_publish">
-                    <i class="fa fa-cloud-upload fa-fw"/> Upload
-                </a>
             </div>
         </div>
     </div>
diff --git a/addons/website_slides/views/website_slides_templates_homepage.xml b/addons/website_slides/views/website_slides_templates_homepage.xml
index 33f84388453..ee959eccca3 100644
--- a/addons/website_slides/views/website_slides_templates_homepage.xml
+++ b/addons/website_slides/views/website_slides_templates_homepage.xml
@@ -11,10 +11,10 @@
                     <a href="/profile/users" class="btn btn-primary btn-lg">Go meet them</a>
                 </div>
                 <div class="col-12 col-md-5" t-if="top3_users">
-                    <div class="ml-4">
+                    <div class="ml-4 ml-md-2 mt-3">
                         <t t-foreach="top3_users" t-as="user">
                             <img class="rounded-circle img-fluid shadow ml-n4"
-                                style="border: 6px solid white"
+                                style="border: 6px solid white; max-width: 40%"
                                 t-attf-src="/web/image/res.users/#{user['id']}/image_128"/>
                         </t>
                     </div>