From fbc90faa8798c5ae53c9628282918014ac329f8a Mon Sep 17 00:00:00 2001
From: Kaalleen <36401965+kaalleen@users.noreply.github.com>
Date: Wed, 21 Jul 2021 17:15:46 +0200
Subject: Current layer issue (#1279)

---
 lib/extensions/base.py | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'lib/extensions/base.py')

diff --git a/lib/extensions/base.py b/lib/extensions/base.py
index 70ca4701..8d5cccad 100644
--- a/lib/extensions/base.py
+++ b/lib/extensions/base.py
@@ -117,15 +117,17 @@ class InkstitchExtension(inkex.Effect):
             if g.get(INKSCAPE_GROUPMODE) == "layer":
                 g.set("style", "display:none")
 
-    def ensure_current_layer(self):
+    def get_current_layer(self):
         # if no layer is selected, inkex defaults to the root, which isn't
         # particularly useful
-        if self.svg.get_current_layer() is self.document.getroot():
+        current_layer = self.svg.get_current_layer()
+        if current_layer is self.document.getroot():
             try:
-                self.current_layer = self.document.xpath(".//svg:g[@inkscape:groupmode='layer']", namespaces=inkex.NSS)[0]
+                current_layer = self.document.xpath(".//svg:g[@inkscape:groupmode='layer']", namespaces=inkex.NSS)[0]
             except IndexError:
                 # No layers at all??  Fine, we'll stick with the default.
                 pass
+        return current_layer
 
     def no_elements_error(self):
         if self.svg.selected:
@@ -134,7 +136,7 @@ class InkstitchExtension(inkex.Effect):
         else:
             inkex.errormsg(_("There are no objects in the entire document that Ink/Stitch knows how to work with.") + "\n")
 
-        inkex.errormsg(_("Tip: Select some objects and use Path -> Object to Path to convert them to paths.") + "\n")
+        inkex.errormsg(_("Tip: Run Extensions > Ink/Stitch > Troubleshoot > Troubleshoot Objects") + "\n")
 
     def descendants(self, node, selected=False, troubleshoot=False):  # noqa: C901
         nodes = []
-- 
cgit v1.2.3


From 2f35a4a192eb6aa3b68b715da6c1ba984084e0e5 Mon Sep 17 00:00:00 2001
From: Kaalleen <36401965+kaalleen@users.noreply.github.com>
Date: Sun, 25 Jul 2021 07:24:34 +0200
Subject: Fix Style Issues (#1154)

Co-authored-by: Lex Neva <github.com@lexneva.name>
---
 lib/extensions/base.py | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

(limited to 'lib/extensions/base.py')

diff --git a/lib/extensions/base.py b/lib/extensions/base.py
index 8d5cccad..dc71d922 100644
--- a/lib/extensions/base.py
+++ b/lib/extensions/base.py
@@ -49,19 +49,7 @@ class InkStitchMetadata(MutableMapping):
 
     def __init__(self, document):
         self.document = document
-        self.metadata = self._get_or_create_metadata()
-
-    def _get_or_create_metadata(self):
-        metadata = self.document.find(SVG_METADATA_TAG)
-
-        if metadata is None:
-            metadata = etree.SubElement(self.document.getroot(), SVG_METADATA_TAG)
-
-            # move it so that it goes right after the first element, sodipodi:namedview
-            self.document.getroot().remove(metadata)
-            self.document.getroot().insert(1, metadata)
-
-        return metadata
+        self.metadata = document.metadata
 
     # Because this class inherints from MutableMapping, all we have to do is
     # implement these five methods and we get a full dict-like interface.
@@ -149,7 +137,8 @@ class InkstitchExtension(inkex.Effect):
             if len(list(layer_commands(node, "ignore_layer"))):
                 return []
 
-        if element.has_style('display') and element.get_style('display') is None:
+        if (node.tag in [EMBROIDERABLE_TAGS, SVG_GROUP_TAG] and
+                element.get_style('display', 'inline') is None):
             return []
 
         if node.tag == SVG_DEFS_TAG:
@@ -211,7 +200,7 @@ class InkstitchExtension(inkex.Effect):
         return patches
 
     def get_inkstitch_metadata(self):
-        return InkStitchMetadata(self.document)
+        return InkStitchMetadata(self.svg)
 
     def get_base_file_name(self):
         svg_filename = self.document.getroot().get(inkex.addNS('docname', 'sodipodi'), "embroidery.svg")
-- 
cgit v1.2.3


From d6e20fae8a03ac162ae0c863fff06b5bd8b77902 Mon Sep 17 00:00:00 2001
From: Kaalleen <36401965+kaalleen@users.noreply.github.com>
Date: Thu, 5 Aug 2021 21:36:44 +0200
Subject: fix element selection (#1305)

---
 lib/extensions/base.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'lib/extensions/base.py')

diff --git a/lib/extensions/base.py b/lib/extensions/base.py
index dc71d922..057a0e63 100644
--- a/lib/extensions/base.py
+++ b/lib/extensions/base.py
@@ -137,8 +137,7 @@ class InkstitchExtension(inkex.Effect):
             if len(list(layer_commands(node, "ignore_layer"))):
                 return []
 
-        if (node.tag in [EMBROIDERABLE_TAGS, SVG_GROUP_TAG] and
-                element.get_style('display', 'inline') is None):
+        if (node.tag in EMBROIDERABLE_TAGS or node.tag == SVG_GROUP_TAG) and element.get_style('display', 'inline') is None:
             return []
 
         if node.tag == SVG_DEFS_TAG:
-- 
cgit v1.2.3