Added ability to delete tasks

app/TaskListBox.vala

@@ -24,6 +24,7 @@ namespace Agenda {
     public class TaskListBox : Gtk.ListBox {
 
         public signal void task_changed (int index, Task task);
+        public signal void task_deleted (int index, Task task);
 
         public TaskListBox (TaskRepositoryFile tasks) {
             this.bind_model (tasks, list_box_create_widget);
@@ -38,6 +39,10 @@ namespace Agenda {
                 task_changed (index, task);
             });
 
+            row.task_deleted.connect ((index, task) => {
+                    task_deleted (index, task);
+            });
+
             return row;
         }
     }

app/TaskListBoxRow.vala

@@ -17,12 +17,12 @@
     You should have received a copy of the GNU General Public License
     along with Agenda.  If not, see <http://www.gnu.org/licenses/>.
 
-***/
-namespace Agenda {
+***/ namespace Agenda {
 
     public class TaskListBoxRow : Gtk.ListBoxRow {
 
         public signal void task_changed (int index, Task task);
+        public signal void task_deleted (int index, Task task);
 
         public TaskListBoxRow (Task task) {
 
@@ -33,6 +33,7 @@ namespace Agenda {
             label.set_attributes (attr_list);
 
             var check_button = new Gtk.CheckButton ();
+
             check_button.set_active (task.complete);
             check_button.toggled.connect (() => {
                 task.complete = check_button.active;
@@ -45,6 +46,15 @@ namespace Agenda {
                 label.set_attributes (attr_list);
             });
 
+            var delete_button = new Gtk.Button.from_icon_name ("edit-delete");
+            delete_button.halign = Gtk.Align.END;
+            delete_button.hexpand = true;
+
+            delete_button.clicked.connect (() => {
+                    var index = this.get_index ();
+                    task_deleted (index, task);
+            });
+
             var grid = new Gtk.Grid ();
             grid.set_margin_start (12);
             grid.set_margin_end (12);
@@ -54,9 +64,9 @@ namespace Agenda {
 
             grid.add (check_button);
             grid.add (label);
+            grid.add (delete_button);
 
             grid.show_all ();
-
             this.add (grid);
         }
     }

app/Window.vala

@@ -59,6 +59,9 @@ namespace Agenda {
             this.get_style_context ().add_class ("rounded");
             this.set_size_request (MIN_WIDTH, MIN_HEIGHT);
 
+            // Recive button events
+            set_events (Gdk.EventMask.BUTTON_PRESS_MASK);
+
             var header = new Gtk.HeaderBar ();
             header.show_close_button = true;
             header.get_style_context ().add_class ("titlebar");
@@ -102,6 +105,7 @@ namespace Agenda {
 
             task_list_box = new Agenda.TaskListBox (Application.tasks);
             task_list_box.task_changed.connect (update_task);
+            task_list_box.task_deleted.connect (delete_task);
 
             scrolled_window = new Gtk.ScrolledWindow (null, null);
             scrolled_window.expand = true;
@@ -154,7 +158,7 @@ namespace Agenda {
 
         public void append_task () {
             Task task = new Task.with_attributes (
-                "",
+                new DateTime.now_local ().hash ().to_string (),
                 false,
                 task_entry.text);
 
@@ -167,6 +171,10 @@ namespace Agenda {
             Application.tasks.update (index, task);
         }
 
+        public void delete_task (int index, Task task) {
+            Application.tasks.remove (index, task);
+        }
+
         public void restore_window_position () {
             var size = Application.settings.get_value ("window-size");
             var position = Application.settings.get_value ("window-position");

lib/ITaskRepository.vala

@@ -25,6 +25,6 @@ namespace Agenda {
         public abstract Task? get_by_id (int id);
         public abstract void add (Task task);
         public abstract void update (int index, Task task);
-        public abstract bool remove (Task task);
+        public abstract bool remove (int index, Task task);
     }
 }

lib/TaskRepositoryFile.vala

@@ -44,6 +44,7 @@ namespace Agenda {
         }
 
         public Task? get_by_id (int index) {
+
             if (index > this.task_list.size) {
                 return null;
             }
@@ -67,10 +68,10 @@ namespace Agenda {
             return (uint) this.task_list.size;
         }
 
-        public bool remove (Task task) {
+        public bool remove (int index, Task task) {
             var removed = this.task_list.remove (task);
             if (removed) {
-                this.items_changed (0, 1, 0);
+                this.items_changed (index, 1, 0);
                 this.save ();
             }
 

src/Models/TaskList.vala

@@ -88,6 +88,30 @@ namespace Agenda {
 
             list_changed ();
         }
+        /*
+         *  Sort the tasks so finished tasks are at bottom
+         */
+        public void sort_tasks () {
+            Gtk.TreeIter iter;
+            bool valid = get_iter_first (out iter);
+            Task[] tasks = {};
+
+            while (valid) {
+                Task task = get_task (iter);
+                if (task.complete) {
+                    tasks += task;
+                    remove (ref iter);
+                    continue;
+                }
+                valid = iter_next (ref iter);
+            }
+
+            int i;
+            for (i = 0; i < tasks.length; i++) {
+                append_task (tasks[i]);
+            }
+            list_changed ();
+        }
 
         public void clear_undo () {
             undo_list = new TaskListHistory ();

src/Widgets/TaskView.vala

@@ -21,6 +21,7 @@
 
 namespace Agenda {
 
+
     public class TaskView : Gtk.TreeView {
 
         private TaskList task_list;
@@ -58,6 +59,8 @@ namespace Agenda {
             text.ypad = 6;
             text.editable = true;
             text.max_width_chars = 10;
+            text.wrap_width = 50;
+            text.wrap_mode = Pango.WrapMode.WORD_CHAR;
             text.ellipsize_set = true;
             text.ellipsize = Pango.EllipsizeMode.END;
 
@@ -82,6 +85,7 @@ namespace Agenda {
             set_tooltip_column (TaskList.Columns.TEXT);
 
             text.editing_started.connect ( (editable, path) => {
+                debug ("Editing started");
                 is_editing = true;
             });
 
@@ -144,10 +148,17 @@ namespace Agenda {
 
         private void text_edited (string path, string edited_text) {
             /* If the user accidentally blanks a task, abort the edit */
+
+            debug ("String: %s length: %d", edited_text, edited_text.length);
             if (task_is_empty (edited_text)) {
                 return;
             }
-            task_list.set_task_text (path, edited_text);
+            if (edited_text.length > EDITED_TEXT_MAX_LEN) {
+                string new_cut_text = edited_text.slice(0, EDITED_TEXT_MAX_LEN);
+                task_list.set_task_text (path, new_cut_text);
+            } else {
+                task_list.set_task_text (path, edited_text);
+            }
             is_editing = false;
         }
     }

src/Window.vala

@@ -23,6 +23,8 @@ namespace Agenda {
 
     const int MIN_WIDTH = 500;
     const int MIN_HEIGHT = 600;
+    // Limit for any edited text
+    const int EDITED_TEXT_MAX_LEN = 64;
 
     const string HINT_STRING = _("Add a new task…");
 
@@ -69,6 +71,16 @@ namespace Agenda {
             header.get_style_context ().add_class (Gtk.STYLE_CLASS_FLAT);
             this.set_titlebar (header);
 
+            var button = new Gtk.Button.from_icon_name ("user-trash-symbolic", Gtk.IconSize.BUTTON);
+            button.clicked.connect (() => {
+                // Remove completed tasks
+                if (task_list != null) {
+                     task_list.remove_completed_tasks();
+                }
+                stdout.printf ("You deleted completed tasks\n");
+            });
+            header.pack_start(button);
+
             // Set up geometry
             Gdk.Geometry geo = Gdk.Geometry ();
             geo.min_width = MIN_WIDTH;
@@ -134,7 +146,7 @@ namespace Agenda {
             task_entry.name = "TaskEntry";
             task_entry.get_style_context ().add_class ("task-entry");
             task_entry.placeholder_text = HINT_STRING;
-            task_entry.max_length = 64;
+            task_entry.max_length = EDITED_TEXT_MAX_LEN;
             task_entry.hexpand = true;
             task_entry.valign = Gtk.Align.START;
             task_entry.set_icon_tooltip_text (
@@ -229,6 +241,8 @@ namespace Agenda {
 
             task_list.append_task (task);
             history_list.add_item (task.text);
+            // When adding a new task rearrange the tasks
+            task_list.sort_tasks ();
             task_entry.text = "";
         }