Item visibility & fix pdf export of a deleted canvas

Author: Philip-Scott

src/Services/FileManager.vala

@@ -52,8 +52,6 @@ public class Spice.Services.FileManager {
     }
 
     public static File? open_image () {
-
-
         List<Gtk.FileFilter> filters = new List<Gtk.FileFilter> ();
         Gtk.FileFilter filter = new Gtk.FileFilter ();
         filter.set_filter_name ("Images");
@@ -180,6 +178,8 @@ public class Spice.Services.FileManager {
         Timeout.add (600 * pages_to_draw, () => {
             bool first = true;
             foreach (var slide in manager.slides) {
+                if (!slide.visible) continue;
+
                 if (!first) {
                     pdf.copy_page ();
                 } else {

src/Widgets/CanvasItems/CanvasItem.vala

@@ -46,6 +46,15 @@ public abstract class  Spice.CanvasItem : Gtk.EventBox {
         }
     }
 
+    public bool item_visible {
+        get {
+            return this.visible;
+        } set {
+            this.visible = value;
+            this.no_show_all = !value;
+        }
+    }
+
     protected double start_x = 0;
     protected double start_y = 0;
     protected int start_w = 0;

src/Widgets/SlideList.vala

@@ -129,7 +129,6 @@ public class Spice.SlideList : Gtk.ScrolledWindow {
                 this.visible = val;
                 this.no_show_all = !val;
 
-                stderr.printf (@"Row.Visible changed $val\n");
 
                 this.show_all ();
             });

src/Widgets/Toolbars/CommonBar.vala

@@ -36,10 +36,10 @@ public class Spice.Widgets.CommonToolbar : Spice.Widgets.Toolbar {
 
         delete_button.clicked.connect (() => {
             if (this.item != null) {
-                var action = new Spice.Services.HistoryManager.HistoryAction<CanvasItem,bool>.item_changed (this.item, "visible");
+                var action = new Spice.Services.HistoryManager.HistoryAction<CanvasItem,bool>.item_changed (this.item, "item-visible");
                 Spice.Services.HistoryManager.get_instance ().add_undoable_action (action, true);
 
-                this.item.visible = false;
+                this.item.item_visible = false;
             } else {
                 var action = new Spice.Services.HistoryManager.HistoryAction<Slide,bool>.slide_changed (this.manager.current_slide, "visible");
                 Spice.Services.HistoryManager.get_instance ().add_undoable_action (action, true);