- fix bug where correct checkboxes were not shown as correct in the review

- debounce authorization checkboxes

- fix bug where correct checkboxes were not shown as correct in the review
- debounce authorization checkboxes
Miguel Barão
1 parent 70459fa7
Showing 5 changed files with 7 additions and 8 deletions Show diff stats
BUGS.md
perguntations/static/js/admin.js
perguntations/templates/admin.html
perguntations/templates/grade.html
perguntations/templates/review-question-checkbox.html
  
 # BUGS
  
-- botao de autorizar desliga-se, fazer debounce.
-e.g. retornar None quando nao ha alteracoes relativamente à última vez.
+- retornar None quando nao ha alteracoes relativamente à última vez.
 ou usar push (websockets?)
 - quando scale_max não é 20, as cores das barras continuam a reflectir a escala 0,20
 - CRITICAL se answer for `i<n` a revisão de provas mostra apenas i (interpreta `<` como tag?)
@@ -62,6 +61,7 @@ ou usar push (websockets?)
  
 # FIXED
  
+- botao de autorizar desliga-se, fazer debounce.
 - link na pagina com a nota para voltar ao principio.
 - default logger config mostrar horas com segundos
 - test: mostrar duração do teste com progressbar no navbar.
@@ -36,7 +36,7 @@ $(document).ready(function() {
             });
         });
         // authorization checkboxes in the students_table:
-        $("tbody", "#students_table").on("change", "input", autorizeStudent);
+        $("tbody", "#students_table").on("change", "input", _.debounce(autorizeStudent));
     }
  
     // ----------------------------------------------------------------------
@@ -25,7 +25,7 @@
     <script defer src="/static/fontawesome-free/js/all.min.js"></script>
     <script defer src="/static/bootstrap/js/bootstrap.min.js"></script>
     <script defer src="/static/datatables/js/jquery.dataTables.min.js"></script>
-
+    <script defer src="/static/underscore/underscore-min.js"></script>
     <script defer src="/static/js/admin.js"></script>
     <script defer src="/static/js/clock.js"></script>
 </head>
@@ -45,7 +45,7 @@
         <strong>{{ f'{round(t["grade"], 1)}' }}</strong>
         valores na escala de {{t['scale'][0]}} a {{t['scale'][1]}}.
       </h1>
-      <p>O seu teste foi entregue e está registado.</p>
+      <p>O seu teste foi correctamente entregue e a nota registada.</p>
       <p><a href="/" class="btn btn-primary btn-lg active" role="button">Clique aqui para sair do teste</a></p>
       {% if t['grade'] - t['scale'][0] >= 0.75*(t['scale'][1] - t['scale'][0]) %}
         <i class="fas fa-thumbs-up fa-5x text-success" aria-hidden="true"></i>
@@ -90,8 +90,7 @@
                   aria-valuenow="{{ 100*(g[1] - t['scale'][0])/(t['scale'][1] - t['scale'][0]) }}"
                   aria-valuemin="0"
                   aria-valuemax="100"
-                                    style="min-width: 2em; width: {{ 100*(g[1]-t['scale'][0])/(t['scale'][1]-t['scale'][0]) }}%;"
-                  >
+                  style="min-width: 2em; width: {{ 100*(g[1]-t['scale'][0])/(t['scale'][1]-t['scale'][0]) }}%;">
  
                     {{ str(round(g[1], 1)) }}
  
@@ -38,7 +38,7 @@
                         <div class="text-right text-danger">
                             <i class="fas fa-times" aria-hidden="true"></i>
                         </div>
-                    {% elif q['correct'][n] < 0 %}
+                    {% else %}
                         <div class="text-right text-success">
                             <i class="fas fa-check" aria-hidden="true"></i>
                         </div>
1	1
2	2	# BUGS
3	3
4		-- botao de autorizar desliga-se, fazer debounce.
5		-e.g. retornar None quando nao ha alteracoes relativamente à última vez.
	4	+- retornar None quando nao ha alteracoes relativamente à última vez.
6	5	ou usar push (websockets?)
7	6	- quando scale_max não é 20, as cores das barras continuam a reflectir a escala 0,20
8	7	- CRITICAL se answer for `i<n` a revisão de provas mostra apenas i (interpreta `<` como tag?)
...	...	@@ -62,6 +61,7 @@ ou usar push (websockets?)
62	61
63	62	# FIXED
64	63
	64	+- botao de autorizar desliga-se, fazer debounce.
65	65	- link na pagina com a nota para voltar ao principio.
66	66	- default logger config mostrar horas com segundos
67	67	- test: mostrar duração do teste com progressbar no navbar.
...	...
...	...	@@ -36,7 +36,7 @@ $(document).ready(function() {
36	36	});
37	37	});
38	38	// authorization checkboxes in the students_table:
39		- $("tbody", "#students_table").on("change", "input", autorizeStudent);
	39	+ $("tbody", "#students_table").on("change", "input", _.debounce(autorizeStudent));
40	40	}
41	41
42	42	// ----------------------------------------------------------------------
...	...
...	...	@@ -25,7 +25,7 @@
25	25	<script defer src="/static/fontawesome-free/js/all.min.js"></script>
26	26	<script defer src="/static/bootstrap/js/bootstrap.min.js"></script>
27	27	<script defer src="/static/datatables/js/jquery.dataTables.min.js"></script>
28		-
	28	+ <script defer src="/static/underscore/underscore-min.js"></script>
29	29	<script defer src="/static/js/admin.js"></script>
30	30	<script defer src="/static/js/clock.js"></script>
31	31	</head>
...	...
...	...	@@ -45,7 +45,7 @@
45	45	<strong>{{ f'{round(t["grade"], 1)}' }}</strong>
46	46	valores na escala de {{t['scale'][0]}} a {{t['scale'][1]}}.
47	47	</h1>
48		- <p>O seu teste foi entregue e está registado.</p>
	48	+ <p>O seu teste foi correctamente entregue e a nota registada.</p>
49	49	<p><a href="/" class="btn btn-primary btn-lg active" role="button">Clique aqui para sair do teste</a></p>
50	50	{% if t['grade'] - t['scale'][0] >= 0.75*(t['scale'][1] - t['scale'][0]) %}
51	51	<i class="fas fa-thumbs-up fa-5x text-success" aria-hidden="true"></i>
...	...	@@ -90,8 +90,7 @@
90	90	aria-valuenow="{{ 100*(g[1] - t['scale'][0])/(t['scale'][1] - t['scale'][0]) }}"
91	91	aria-valuemin="0"
92	92	aria-valuemax="100"
93		- style="min-width: 2em; width: {{ 100*(g[1]-t['scale'][0])/(t['scale'][1]-t['scale'][0]) }}%;"
94		- >
	93	+ style="min-width: 2em; width: {{ 100*(g[1]-t['scale'][0])/(t['scale'][1]-t['scale'][0]) }}%;">
95	94
96	95	{{ str(round(g[1], 1)) }}
97	96
...	...
...	...	@@ -38,7 +38,7 @@
38	38	<div class="text-right text-danger">
39	39	<i class="fas fa-times" aria-hidden="true"></i>
40	40	</div>
41		- {% elif q['correct'][n] < 0 %}
	41	+ {% else %}
42	42	<div class="text-right text-success">
43	43	<i class="fas fa-check" aria-hidden="true"></i>
44	44	</div>
...	...