From 2bcbe5efabc21a7bd82e72c67f72851ccc089095 Mon Sep 17 00:00:00 2001
From: Thomas Fradet <t.fradet8@gmail.com>
Date: Mon, 27 May 2019 18:55:27 +0200
Subject: [PATCH] front improvements

---
 CHANGELOG.md              | 33 +++++++++++++++++++++++++++++++++
 block_career.php          | 19 +++++++++----------
 styles.css                |  6 ------
 view/view_career_list.php |  4 +++-
 4 files changed, 45 insertions(+), 17 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bd947ba..25c7132 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -66,9 +66,42 @@ Ajout d'un lien pour revenir au cours, visible par l'étudiant et l'enseignant.
 
 Ajout d'un bouton en haut de page pour éditer le parcours, visible uniquement par l'enseignant. 
 
+#### Description des sections
+
+[ ] Revoir l'apparence de la description des sections. 
+
 ### Gestion des droits
 
 Ajout de vérification du droit d'éditer le cours pour les interfaces d'édition de parcours. Modification de noms de paramètres. 
 
+### Liste des parcours et édition
+
+#### Amélioration de l'apparence de la liste des parcours
+
+Sous forme de liste cliquable. 
+
+#### Textes
+
+Amélioration des textes sur les pages de liste et d'édition des parcours. 
+
+#### Liste à cocher
+
+[ ] Transformer le multiselect en liste à cocher. 
+
+Attention : actuellement, on peut ajouter plusieurs fois le même élément ce qui créée des erreurs. 
+
+### Bloc en lui-même
+
+#### Amélioration de l'apparence de la liste
+
+Amélioration de l'apparence de la liste dans le bloc et fusion des boutons avec cette liste (en couleur). 
+
+### Place du block
+
+[ ] Faire apparaître le bloc sur toutes les pages du cours, ou au moins sur celle qui contient le parcours. Sinon, faire un menu déroullant pour changer de parcours depuis cette vue. 
+
+[ ] Vérifier que le block n'est pas instanciable sur un autre type de page que la page du cours, par exemple la page d'accueil du site ou le profile. 
+
+
 
 
diff --git a/block_career.php b/block_career.php
index 27a5b2d..40a9cbe 100644
--- a/block_career.php
+++ b/block_career.php
@@ -67,11 +67,11 @@ class block_career extends block_base
 		$active = "";
 		
 		$this->content = new stdClass;
-		$this->content->text = '<a href="' . $CFG->wwwroot . '/course/view.php?id=' . $COURSE->id . '"  class="btn btn-success btn-career-block mb-3">Accueil du cours</a>';
 		
-		// $image = "";
-		
-		$this->content->text .= '<div class="list-group">';
+		$this->content->text = '';
+		$this->content->text .= '<div class="list-group iena-list-group">';
+
+		$this->content->text .= '<a href="' . $CFG->wwwroot . '/course/view.php?id=' . $COURSE->id . '"  class="list-group-item list-group-item-success list-group-item-action" style="border-top-left-radius: .25rem; border-top-right-radius: .25rem;">Accueil du cours</a>';
 		
 		foreach ($request as $value) {
 			
@@ -85,21 +85,20 @@ class block_career extends block_base
 				$active = "";
 			}
 			
-			$this->content->text .= "<a href='" . $CFG->wwwroot . "/blocks/career/career_unit.php?career=" . $value->id . "' class='list-group-item $active'>$value->name</a>";
+			$this->content->text .= "<a href='" . $CFG->wwwroot . "/blocks/career/career_unit.php?career=" . $value->id . "' class='list-group-item list-group-item-action $active'>$value->name</a>";
 			// $this->content->text .= "<a href='" . $CFG->wwwroot . "/blocks/career/career_unit.php?career=" . $value->id . "' class='full list-group-item list-group-item-action $active'><div class=' left img_center'>$image</div>
 			// &nbsp&nbsp $value->name</a><br>";
 		}
 		
+		if (has_capability('moodle/course:update', $context = context_course::instance($COURSE->id), $USER->id)) {
+			$this->content->text .= '<a href="' . $CFG->wwwroot . '/blocks/career/career_list.php?courseid=' . $COURSE->id . '" " class="list-group-item list-group-item-primary list-group-item-action" style="border-bottom-left-radius: .25rem; border-bottom-right-radius: .25rem;">Gérer les parcours</a>';
+		}
+
 		$this->content->text .= '</div>';
 		
 		if (empty($request)) {
 			$this->content->text .= "<p>" . get_string('any_carrer', 'block_career') . "</p>";
 		}
-
-		if (has_capability('moodle/course:update', $context = context_course::instance($COURSE->id), $USER->id)) {
-			$this->content->text .= '<a href="' . $CFG->wwwroot . '/blocks/career/career_list.php?courseid=' . $COURSE->id . '" type="button " class="btn btn-primary btn-career-block mt-3">Gérer les parcours</a>';
-		}
-		
 		
 		// $this->content->text .= "<p></p>";
 		
diff --git a/styles.css b/styles.css
index 220e7cb..5515d94 100644
--- a/styles.css
+++ b/styles.css
@@ -1,9 +1,3 @@
-/* Block itself */
-
-.btn-career-block {
-	width: 100%;
-}
-
 /* Path view */
 .career-section-title {
 	background-color: #008196;
diff --git a/view/view_career_list.php b/view/view_career_list.php
index 6ddaae7..47549d0 100644
--- a/view/view_career_list.php
+++ b/view/view_career_list.php
@@ -22,6 +22,8 @@ public function get_content()
 
 	$request = $DB->get_records_sql('SELECT * FROM {block_career} WHERE course = ?', array($_GET["courseid"]));
 
+	$content .= "<a href='$CFG->wwwroot/blocks/career/career_setting.php?courseid=" . $COURSE->id . "' class='btn btn-primary m-b-1'>" . get_string('add_path', 'block_career') . "</a>";
+
 	$content .= "<div class='list-group'>";
 
 	foreach ($request as $value) {
@@ -41,7 +43,7 @@ public function get_content()
 		$content .= "<p>" . get_string('any_carrer', 'block_career') . "</p>";
 	}
 			// Button for adding course to the list
-	$content .= "<a href='$CFG->wwwroot/blocks/career/career_setting.php?courseid=" . $COURSE->id . "' class='btn btn-primary'>" . get_string('add_path', 'block_career') . "</a>";
+	$content .= "<a href='$CFG->wwwroot/blocks/career/career_setting.php?courseid=" . $COURSE->id . "' class='btn btn-primary m-t-1'>" . get_string('add_path', 'block_career') . "</a>";
 
 
 	return $content;
-- 
GitLab