From 16b911d4892954bef39c28b6dfcd80b0fe366f60 Mon Sep 17 00:00:00 2001
From: fredtempez <ftempez@gmail.com>
Date: Sun, 10 Nov 2019 20:55:30 +0100
Subject: [PATCH] =?UTF-8?q?[9.2.10]=20Afficher=20les=20infos=20sur=20l'ima?=
 =?UTF-8?q?ge=20et=20hauteur=20de=20la=20banni=C3=A8re=20image?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/theme/view/header/header.js.php | 40 +++++++++++----------
 core/module/theme/view/header/header.php    |  8 +++--
 2 files changed, 27 insertions(+), 21 deletions(-)

diff --git a/core/module/theme/view/header/header.js.php b/core/module/theme/view/header/header.js.php
index 36a20a6f..13311523 100755
--- a/core/module/theme/view/header/header.js.php
+++ b/core/module/theme/view/header/header.js.php
@@ -12,16 +12,18 @@
  * @link http://zwiicms.com/
  */
 
+
+
 /**
  * Aperçu en direct
  */
-$("input, select, .lity-iframe").on("change", function() {
+$("input, select").on("change", function() {
 
-		// Récupérer la taille de l'image
-
-		var tmpImg = new Image();
-		tmpImg.src= "<?php echo helper::baseUrl(false); ?>" + "site/file/source/" + $("#themeHeaderImage").val();
+	// Récupérer la taille de l'image
 
+	var tmpImg = new Image();
+	
+	tmpImg.onload = function() {
 		// Informations affichées
 		$("#themeHeaderImageHeight").html(tmpImg.height + "px");
 		$("#themeHeaderImageWidth").html(tmpImg.width + "px");
@@ -34,8 +36,21 @@ $("input, select, .lity-iframe").on("change", function() {
 		}
 
 		//Modifier la dropdown liste
-		$("#themeHeaderHeight option:eq(0)").val(tmpImgHeight + "px");
-		$("#themeHeaderHeight option:first-child").html("Hauteur de l\'image sélectionnée (" + tmpImgHeight + "px)");
+		if ($("#themeHeaderImage").val() !== "") {
+			// Une image est ajoutée ou changée
+			if ($("#themeHeaderHeight option").length === 4) {
+				$("#themeHeaderHeight ").prepend('<option selected="selected" value="0"> Hauteur de l\'image sélectionnée </option>');
+			}
+			// Modifier la valeur
+			$("#themeHeaderHeight option:eq(0)").val(tmpImgHeight + "px");
+			// Modifier l'option
+			$("#themeHeaderHeight option:first-child").html("Hauteur de l\'image sélectionnée (" + tmpImgHeight + "px)");
+		} else {
+			$("#themeHeaderHeight option:first-child").html("Hauteur de l\'image sélectionnée");
+		}
+	};
+
+	tmpImg.src= "<?php echo helper::baseUrl(false); ?>" + "site/file/source/" + $("#themeHeaderImage").val();
 
 	// Import des polices de caractères
 	var headerFont = $("#themeHeaderFont").val();
@@ -119,17 +134,6 @@ $("input, select, .lity-iframe").on("change", function() {
 		.appendTo("head");
 }).trigger("change");
 
-/**
-
-$("#themeHeaderHeight").on("change", function() {
-	if($(this).val() === 'none') {
-		$("#themeHeaderTextHide").prop("disabled", true);
-		$("#themeHeaderTextHide").prop("checked", true).trigger("change");	
-	} else {
-		$("#themeHeaderTextHide").prop("disabled", false);
-	}
-}).trigger("change");
-*/
 
 
 // Affiche / Cache les options de l'image du fond
diff --git a/core/module/theme/view/header/header.php b/core/module/theme/view/header/header.php
index be416313..a0ad4192 100755
--- a/core/module/theme/view/header/header.php
+++ b/core/module/theme/view/header/header.php
@@ -51,7 +51,6 @@
                             'type' => 1,
                             'value' => $imageFile
                     ]); ?>
-                    <em>Largeur <span id="themeHeaderImageWidth"></span> - Hauteur <span id="themeHeaderImageHeight"></span></em>
                 </div>
             </div>
             <div id="themeHeaderImageOptions" class="displayNone">
@@ -85,7 +84,10 @@
                         <?php echo template::checkbox('themeHeaderlinkHomePage', true, 'Bannière cliquable', [
                                 'checked' => $this->getData(['theme', 'header', 'linkHomePage'])
                             ]); ?>  
-                    </div>    
+                    </div>   
+                    <div class="col4">
+                        <em>(largeur:<span id="themeHeaderImageWidth"></span> ; hauteur:<span id="themeHeaderImageHeight"></span>)</em>
+                    </div> 
                 </div>
             </div>
         </div>
@@ -106,7 +108,7 @@
                     <?php echo template::select('themeHeaderHeight', $module::$headerHeights, [
 							'label' => 'Hauteur',
                             'selected' => $this->getData(['theme', 'header', 'height']),
-                            'help' => "La hauteur maximale est de 600 pixels lorsque l'option \'Hauteur de l\'image\' est sélectionnée."
+                            'help' => "Quelque soit la taille de la bannière, la hauteur maximale autorisée est de 600 pixels."
 						]); ?>
                 </div>
                 <div class="col4">