From 459e0b3b166f1e9490fec7f1d6525b1b54f76ac9 Mon Sep 17 00:00:00 2001
From: yflory <yann.flory@xwiki.com>
Date: Wed, 26 Jul 2017 13:30:39 +0200
Subject: [PATCH] CSS and bug fixes for todo

---
 customize.dist/messages.js | 4 +++-
 www/todo/inner.html        | 2 +-
 www/todo/main.js           | 6 +++---
 www/todo/todo.less         | 6 ++++++
 4 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/customize.dist/messages.js b/customize.dist/messages.js
index 216dd4bbe..2f31a516e 100644
--- a/customize.dist/messages.js
+++ b/customize.dist/messages.js
@@ -151,10 +151,12 @@ define(req, function($, Default, Language) {
     messages._applyTranslation = function () {
         $('[data-localization]').each(translateText);
         $('[data-localization-append]').each(translateAppend);
-        $('#pad-iframe').contents().find('[data-localization]').each(translateText);
         $('[data-localization-title]').each(translateTitle);
         $('[data-localization-placeholder]').each(translatePlaceholder);
+        $('#pad-iframe').contents().find('[data-localization]').each(translateText);
+        $('#pad-iframe').contents().find('[data-localization-append]').each(translateAppend);
         $('#pad-iframe').contents().find('[data-localization-title]').each(translateTitle);
+        $('#pad-iframe').contents().find('[data-localization-placeholder]').each(translatePlaceholder);
     };
 
     messages.driveReadme = '["BODY",{"class":"cke_editable cke_editable_themed cke_contents_ltr cke_show_borders","contenteditable":"true","spellcheck":"false","style":"color: rgb(51, 51, 51);"},' +
diff --git a/www/todo/inner.html b/www/todo/inner.html
index 6fd175357..7675ed8d3 100644
--- a/www/todo/inner.html
+++ b/www/todo/inner.html
@@ -11,7 +11,7 @@
     <div id="container">
         <div class="cp-create-form">
             <input type="text" id="newTodoName" data-localization-placeholder="todo_newTodoNamePlaceholder" />
-            <button class="btn btn-success" data-localization-title="todo_newTodoNameTitle">Add the task</button>
+            <button class="btn btn-success fa fa-plus" data-localization-title="todo_newTodoNameTitle"></button>
         </div>
         <div id="tasksList"></div>
     </div>
diff --git a/www/todo/main.js b/www/todo/main.js
index 749fdfe4d..9fc7ba946 100644
--- a/www/todo/main.js
+++ b/www/todo/main.js
@@ -58,7 +58,7 @@ define([
 
             return $('<span>', {
                 'class': 'cp-task-checkbox fa ' + checked,
-                //title: title,
+                title: title,
             }).on('click', function () {
                 entry.state = (entry.state + 1) % 2;
                 if (typeof(cb) === 'function') {
@@ -73,8 +73,8 @@ define([
             }).appendTo($list);
             $taskDiv.data('id', el);
 
-            var $box = makeCheckbox(el, function (state) {
-                display();
+            makeCheckbox(el, function (/*state*/) {
+                APP.display();
             })
             .appendTo($taskDiv);
 
diff --git a/www/todo/todo.less b/www/todo/todo.less
index 9a16733e8..188c58e6c 100644
--- a/www/todo/todo.less
+++ b/www/todo/todo.less
@@ -62,6 +62,7 @@ body {
     }
 
     button {
+        cursor: pointer;
         border-radius: 0;
         background-color: darken(@toolbar-todo-bg, 20%);
         border:0;
@@ -110,4 +111,9 @@ body {
     .cp-task-checkbox-unchecked {
 
     }
+
+    button {
+        border-radius: 0;
+        border:0;
+    }
 }