From 83ef586a092157c6e5981946315e4994cb89757a Mon Sep 17 00:00:00 2001
From: fredtempez <ftempez@gmail.com>
Date: Sat, 2 Feb 2019 12:04:01 +0100
Subject: [PATCH] =?UTF-8?q?nouvelle=20pr=C3=A9sentation=20de=20l'=C3=A9dit?=
 =?UTF-8?q?ion=20d'une=20page?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/page/page.php              |  6 ++-
 core/module/page/view/edit/edit.js.php | 47 ++++++++++++++++---
 core/module/page/view/edit/edit.php    | 64 +++++++++++++++-----------
 3 files changed, 84 insertions(+), 33 deletions(-)

diff --git a/core/module/page/page.php b/core/module/page/page.php
index afd66ba4..823febe6 100755
--- a/core/module/page/page.php
+++ b/core/module/page/page.php
@@ -47,7 +47,7 @@ class page extends common {
 		'3-9' => 'Barre latérale 1/4 - Page 3/4',
 		'9-3' => 'Page 3/4    - Barre latérale 1/4',
 		'3-6-3' => 'Barre latérale 1/4 - Page 1/2 - Barre latérale 1/4',
-		'bar' => 'Barre latérale'
+		'bar' => 'Cette page est une barre latérale'
 	];
 	
 	/**
@@ -208,9 +208,13 @@ class page extends common {
 				if ($this->getinput('pageEditBlock') !== 'bar') {
 					$barLeft = $this->getinput('pageEditBarLeft');
 					$barRight = $this->getinput('pageEditBarRight');
+
 				} else {
+					// Une barre ne peut pas avoir de barres 
 					$barLeft = "";
 					$barRight = "";
+					// Une barre est masquée
+					$position = 0;
 				}
 				// Modifie la page ou en crée une nouvelle si l'id a changé
 				$this->setData([
diff --git a/core/module/page/view/edit/edit.js.php b/core/module/page/view/edit/edit.js.php
index 6d65bbc4..64d292c0 100755
--- a/core/module/page/view/edit/edit.js.php
+++ b/core/module/page/view/edit/edit.js.php
@@ -53,12 +53,10 @@ pageEditModuleIdDOM.on("change", function() {
 });
 if($("#pageEditModuleId").val() === "form" ||
    $("#pageEditModuleId").val() === "gallery") {
-	   console.log('ok');
     	$("#configModulePositionWrapper").addClass("disabled");
 		$("#configModulePositionWrapper").slideDown();	
 	}
 	else {
-		console.log('pas ok');
 	    $("#configModulePositionWrapper").removeClass("disabled");
 		$("#configModulePositionWrapper").slideUp();				
 }
@@ -123,12 +121,12 @@ if($("#pageEditModuleId").val() === "redirection" ||
 var pageEditBlockDOM = $("#pageEditBlock");
 pageEditBlockDOM.on("change", function() {
 	switch ($(this).val()) {
-		case "bar":
+		case "bar":						
 		case "12":
 			$("#pageEditBarLeftWrapper").removeClass("disabled");
 			$("#pageEditBarLeftWrapper").slideUp();
 			$("#pageEditBarRightWrapper").removeClass("disabled");
-			$("#pageEditBarRightWrapper").slideUp();	
+			$("#pageEditBarRightWrapper").slideUp();				
 			break;
 		case "3-9":
 		case "4-8":
@@ -149,8 +147,27 @@ pageEditBlockDOM.on("change", function() {
 			$("#pageEditBarLeftWrapper").slideDown();
 			$("#pageEditBarRightWrapper").addClass("disabled");
 			$("#pageEditBarRightWrapper").slideDown();				
-			break;
+			break;	
 	}
+	if ($(this).val() === "bar") {
+			$("#pageEditPositionWrapper").removeClass("disabled");
+			$("#pageEditPositionWrapper").slideUp();
+			$("#pageEditTargetBlank").removeClass("disabled");
+			$("#pageEditTargetBlank").slideUp();
+			$("#pageDisableWrapper").removeClass("disabled");
+			$("#pageDisableWrapper").slideUp();	
+			$("#pageEditTargetBlankWrapper").removeClass("disabled");
+			$("#pageEditTargetBlankWrapper").slideUp();
+	} else {
+			$("#pageEditPositionWrapper").addClass("disabled");
+			$("#pageEditPositionWrapper").slideDown();
+			$("#pageEditTargetBlank").addClass("disabled");
+			$("#pageEditTargetBlank").slideDown();
+			$("#pageDisableWrapper").addClass("disabled");
+			$("#pageDisableWrapper").slideDown();	
+			$("#pageEditTargetBlankWrapper").addClass("disabled");
+			$("#pageEditTargetBlankWrapper").slideDown();
+	}	
 });
 switch ($("#pageEditBlock").val()) {
 		case "bar":
@@ -181,7 +198,25 @@ switch ($("#pageEditBlock").val()) {
 			$("#pageEditBarRightWrapper").slideDown();				
 			break;
 };
-
+if ($("#pageEditBlock").val() === "bar") {
+			$("#pageEditPositionWrapper").removeClass("disabled");
+			$("#pageEditPositionWrapper").slideUp();
+			$("#pageEditTargetBlank").removeClass("disabled");
+			$("#pageEditTargetBlank").slideUp();
+			$("#pageDisableWrapper").removeClass("disabled");
+			$("#pageDisableWrapper").slideUp();	
+			$("#pageEditTargetBlankWrapper").removeClass("disabled");
+			$("#pageEditTargetBlankWrapper").slideUp();
+	} else {
+			$("#pageEditPositionWrapper").addClass("disabled");
+			$("#pageEditPositionWrapper").slideDown();
+			$("#pageEditTargetBlank").addClass("disabled");
+			$("#pageEditTargetBlank").slideDown();
+			$("#pageDisableWrapper").addClass("disabled");
+			$("#pageDisableWrapper").slideDown();	
+			$("#pageEditTargetBlankWrapper").addClass("disabled");
+			$("#pageEditTargetBlankWrapper").slideDown();
+	}
 
 
 
diff --git a/core/module/page/view/edit/edit.php b/core/module/page/view/edit/edit.php
index 9d66358c..07924687 100755
--- a/core/module/page/view/edit/edit.php
+++ b/core/module/page/view/edit/edit.php
@@ -107,19 +107,27 @@
 				<?php echo template::select('pageEditPosition', [], [
 					'label' => 'Position'
 				]); ?>
-				<?php echo template::checkbox('pageEditTargetBlank', true, 'Ouvrir dans un nouvel onglet', [
-					'checked' => $this->getData(['page', $this->getUrl(2), 'targetBlank'])
-				]); ?>
-				<?php echo template::checkbox('pageDisable', true, 'Page inactive', [
-					'checked' => $this->getData(['page', $this->getUrl(2), 'disable'])					
-				]); ?>
+				<div class="row">
+					<div class="col6">
+					<?php echo template::checkbox('pageEditTargetBlank', true, 'Nouvel onglet', [
+						'checked' => $this->getData(['page', $this->getUrl(2), 'targetBlank'])
+					]); ?>
+					</div>
+					<div class="col6">
+					<?php echo template::checkbox('pageDisable', true, 'Page inactive', [
+						'checked' => $this->getData(['page', $this->getUrl(2), 'disable'])					
+					]); ?>
+					</div>
+				</div>
 			</div>
+
+		</div>
+		<div class="col6">
 			<div class="block">
 				<h4>Mise en page</h4>
 				<?php echo template::select('pageEditBlock', $module::$pageBlocks, [
 						'label' => 'Gabarit de mise en page :',
-						'help' => 'Une page définie comme barre latérale est utilisée pour la pagination d\'une page standard > Cette page sera définie comme barre latérale.
-						          <br>N\'oubliez pas de masquer cette barre latérale dans le menu.',
+						'help' => 'Une page définie comme barre latérale est utilisée pour la pagination d\'une page standard.',
 						'selected' => $this->getData(['page', $this->getUrl(2) , 'block'])
 				]); ?>				
 			<!-- Sélection des barres latérales	 -->
@@ -143,25 +151,29 @@
 						'selected' => $this->getData(['page', $this->getUrl(2), 'barRight'])
 					]); ?>
 				<?php endif; ?>				
-				</div>							
+			</div>							
 		</div>
-		<div class="col6">
-			<div class="block">
-				<h4>Options avancées</h4>
-				<?php echo template::select('pageEditGroup', self::$groupPublics, [
-					'label' => 'Groupe requis pour accéder à la page',
-					'selected' => $this->getData(['page', $this->getUrl(2), 'group'])
-				]); ?>
-				<?php echo template::text('pageEditMetaTitle', [
-					'label' => 'Méta-titre',
-					'value' => $this->getData(['page', $this->getUrl(2), 'metaTitle'])
-				]); ?>
-				<?php echo template::textarea('pageEditMetaDescription', [
-					'label' => 'Méta-description',
-					'maxlength' => '500',
-					'value' => $this->getData(['page', $this->getUrl(2), 'metaDescription'])
-				]); ?>
-			</div>
+	</div>
+	<div class='row'>
+		<div class="block">
+					<h4>Options avancées</h4>
+					<div class='col6'>
+						<?php echo template::select('pageEditGroup', self::$groupPublics, [
+							'label' => 'Groupe requis pour accéder à la page :',
+							'selected' => $this->getData(['page', $this->getUrl(2), 'group'])
+						]); ?>
+					</div>
+					<div class='col12'>
+						<?php echo template::text('pageEditMetaTitle', [
+							'label' => 'Méta-titre',
+							'value' => $this->getData(['page', $this->getUrl(2), 'metaTitle'])
+						]); ?>
+						<?php echo template::textarea('pageEditMetaDescription', [
+							'label' => 'Méta-description',
+							'maxlength' => '500',
+							'value' => $this->getData(['page', $this->getUrl(2), 'metaDescription'])
+						]); ?>
+					</div>						
 		</div>
 	</div>
 <?php echo template::formClose(); ?>