From 3c21c5d97510807c34a12077e9a8bc236dab1db1 Mon Sep 17 00:00:00 2001
From: chrqls <charles.quelos@gmail.com>
Date: Thu, 19 Jun 2014 18:09:21 +0200
Subject: [PATCH] [#39] Removing useless code and better UI when deleting a
 question

---
 lib/smile/game.js         | 38 --------------------------------------
 static/js/smileteacher.js |  9 +--------
 static/smile-teacher.html | 12 ++++++++----
 3 files changed, 9 insertions(+), 50 deletions(-)

diff --git a/lib/smile/game.js b/lib/smile/game.js
index 79816f6..27c6169 100644
--- a/lib/smile/game.js
+++ b/lib/smile/game.js
@@ -41,44 +41,6 @@ Game.prototype.setRatingMetadata = function setRatingMetadata(metadata) {
     }
 };
 
-/*  OLD METHOD TO RETAKE. 
-    NOT CALLED ANYMORE: TO DELETE SOON *//*
-Game.prototype.retake_dirty = function retake_dirty() {
-
-    this.questionRatings = {};
-    this.questionCorrectCountMap = {};
-    for (var key in this.students.currentStudents) {
-        this.students.currentStudents[key].retake();
-    }
-
-    //var oldGame = this;
-    //this = new Game();
-    //this.students = oldGame.students;
-    //this.studentsWrapper = new StudentsWrapper(oldGame.students);  
-
-    // Removing questions of past messages
-    var sizeQuestions = this.questions.numberOfQuestions;
-
-    for (var i=0; i<sizeQuestions;i++) {
-
-        // Handle deletion
-        var status = this.questions.deleteQuestion(0);
-
-         // Verify we get back an SessionID of the deleted question
-        if (status !== 0) {
-            var msgs = this.messages.past;
-
-            for (var j=0; j<msgs.length; j++) {
-
-                if ((msgs[j].SessionID !== undefined) && (msgs[j].SessionID === status.SessionID) ){
-                    this.messages.past.splice(j, 1); // Delete the question from the ghost of messages past
-                }
-            }
-        }
-    }
-    this.questions = new Questions();
-};*/
-
 /*
     When retaking, there is 2 things to manage: 
         - the past messages to clean
diff --git a/static/js/smileteacher.js b/static/js/smileteacher.js
index ee0bc28..12e5e5f 100644
--- a/static/js/smileteacher.js
+++ b/static/js/smileteacher.js
@@ -351,8 +351,7 @@ GlobalViewModel.startMakingQuestionsWithIQSet = function() {
                                       .querySelector('[class=selected]')
                                       .getAttribute('position_iqset');
         
-        /* We get the position to get the id of the iqset
-        var iqset = JSON.parse(smile_iqset(GlobalViewModel.iqsets()[GlobalViewModel.position()].id));*/
+        /* We get the position to get the id of the iqset */
         var iqset = JSON.parse(smile_iqset(GlobalViewModel.iqsets()[positionOfIQSet].id));
         var iqdata = iqset.iqdata;
 
@@ -456,8 +455,6 @@ function detailStudent() {
             }
         }
 
-        // console.log('studentToDetail='+JSON.stringify(studentToDetail));
-
         GlobalViewModel.student_detail.push(studentToDetail);
 
         // We highlight the right answers
@@ -958,10 +955,6 @@ function calculateAndShowResults(dataAll) {
         for(var i=0; i<percents_success.length; i++) {
 
             if($(this).find('input[type=hidden]').attr('name') === percents_success_sessionIDs[i]) {
-                
-
-                console.log('TEST');
-                var html = '<span class="">';
 
                 var html;
 
diff --git a/static/smile-teacher.html b/static/smile-teacher.html
index 80576bd..b638147 100644
--- a/static/smile-teacher.html
+++ b/static/smile-teacher.html
@@ -501,7 +501,10 @@ <h4><span data-bind="text: sessionName"></span></h4>
 										    	<div style="margin:5px 0"><span style="margin:0" data-bind="text: $data"></span></div>
 										    </div>
 										    <input type="hidden" data-bind="attr: {value: $data.answer}" />
-										    <div><img data-bind="attr:{src: urlImage}"></div>
+										    <div>
+										    	<span data-bind="text: urlImage"></span><br>
+										    	<img data-bind="attr:{src: urlImage}">
+										    </div>
 									    </div>    
 									</div>
 								</div>
@@ -529,10 +532,11 @@ <h4><span data-bind="text: sessionName"></span></h4>
 										<img data-bind="attr:{src: urlImage}">
 									</p>
 								</div>
-								<a href="#" onclick="confirmDeletion(this.name)" data-bind="attr: {name: session_id}" class="alert expand button delete">Delete</a>    
+								<ul class="button-group radius even-2">
+									<li><a href="#" data-bind='click: $root.redirectView' class="secondary expand button">Back</a></li>
+									<li><a href="#" onclick="confirmDeletion(this.name)" data-bind="attr: {name: session_id}" class="alert expand button delete">Delete</a></li>
+								</ul>
 							</div>
-							
-							<a href="#" data-bind='click: redirectView' class="secondary expand button">Back</a>
 						</section>
 
 						<section smile="confirm-deletion" class="hidden">