group symbol+connector and label with command description

2 files changed, 51 insertions, 6 deletions

diff --git a/lib/commands.py b/lib/commands.py
index cadfa080..5a471c2e 100644
--- a/lib/commands.py
+++ b/lib/commands.py
@@ -3,13 +3,42 @@ import cubicsuperpath
 
 from .svg import apply_transforms
 from .svg.tags import SVG_USE_TAG, SVG_SYMBOL_TAG, CONNECTION_START, CONNECTION_END, XLINK_HREF
+from .utils import cache
+from .i18n import _
 
+COMMANDS = {
+    # l10n: command attached to an object
+    "fill_start": _("Fill stitch starting position"),
+
+    # l10n: command attached to an object
+    "fill_end": _("Fill stitch ending position"),
+
+    # l10n: command attached to an object
+    "stop": _("Stop (pause machine) after sewing this object"),
+
+    # l10n: command attached to an object
+    "trim": _("Trim thread after sewing this object"),
+
+    # l10n: command attached to an object
+    "ignore_object": _("Ignore this object (do not stitch)"),
+
+    # l10n: command that affects entire layer
+    "ignore_layer": _("Ignore layer (do not stitch any objects in this layer)")
+}
+
+OBJECT_COMMANDS = [ "fill_start", "fill_end", "stop", "trim", "ignore_object" ]
+LAYER_COMMANDS = [ "ignore_layer" ]
 
 class CommandParseError(Exception):
     pass
 
 
 class BaseCommand(object):
+    @property
+    @cache
+    def description(self):
+        return get_command_description(self.command)
+
     def parse_symbol(self):
         if self.symbol.tag != SVG_SYMBOL_TAG:
             raise CommandParseError("use points to non-symbol")
@@ -87,6 +116,9 @@ class StandaloneCommand(BaseCommand):
 
         self.parse_symbol()
 
+def get_command_description(command):
+    return COMMANDS[command]
+
 
 def find_commands(node):
     """Find the symbols this node is connected to and return them as Commands"""
diff --git a/lib/extensions/object_commands.py b/lib/extensions/object_commands.py
index 27a07969..e9ee6063 100644
--- a/lib/extensions/object_commands.py
+++ b/lib/extensions/object_commands.py
@@ -7,14 +7,15 @@ from random import random
 from shapely import geometry as shgeo
 
 from .commands import CommandsExtension
+from ..commands import OBJECT_COMMANDS, get_command_description
 from ..i18n import _
 from ..elements import SatinColumn
-from ..svg.tags import SVG_GROUP_TAG, SVG_USE_TAG, SVG_PATH_TAG, INKSCAPE_GROUPMODE, XLINK_HREF, CONNECTION_START, CONNECTION_END, CONNECTOR_TYPE
+from ..svg.tags import *
 from ..svg import get_correction_transform
 
 
 class ObjectCommands(CommandsExtension):
-    COMMANDS = ["fill_start", "fill_end", "stop", "trim", "ignore_object"]
+    COMMANDS = OBJECT_COMMANDS
 
     def add_connector(self, symbol, element):
         # I'd like it if I could position the connector endpoint nicely but inkscape just
@@ -27,14 +28,16 @@ class ObjectCommands(CommandsExtension):
                 "id": self.uniqueId("connector"),
                 "d": "M %s,%s %s,%s" % (start_pos[0], start_pos[1], end_pos.x, end_pos.y),
                 "style": "stroke:#000000;stroke-width:1px;stroke-opacity:0.5;fill:none;",
-                "transform": get_correction_transform(symbol),
                 CONNECTION_START: "#%s" % symbol.get('id'),
                 CONNECTION_END: "#%s" % element.node.get('id'),
                 CONNECTOR_TYPE: "polyline",
+
+                # l10n: the name of the line that connects a command to the object it applies to
+                INKSCAPE_LABEL: _("connector")
             }
         )
 
-        symbol.getparent().insert(symbol.getparent().index(symbol), path)
+        symbol.getparent().insert(0, path)
 
     def get_command_pos(self, element, index, total):
         # Put command symbols 30 pixels out from the shape, spaced evenly around it.
@@ -71,7 +74,15 @@ class ObjectCommands(CommandsExtension):
 
             pos = self.get_command_pos(element, i, len(commands))
 
-            symbol = inkex.etree.SubElement(element.node.getparent(), SVG_USE_TAG,
+            group = inkex.etree.SubElement(element.node.getparent(), SVG_GROUP_TAG,
+                {
+                    "id": self.uniqueId("group"),
+                    INKSCAPE_LABEL: _("Ink/Stitch Command") + ": %s" % get_command_description(command),
+                    "transform": get_correction_transform(element.node)
+                }
+            )
+
+            symbol = inkex.etree.SubElement(group, SVG_USE_TAG,
                 {
                     "id": self.uniqueId("use"),
                     XLINK_HREF: "#inkstitch_%s" % command,
@@ -79,7 +90,9 @@ class ObjectCommands(CommandsExtension):
                     "width": "100%",
                     "x": str(pos.x),
                     "y": str(pos.y),
-                    "transform": get_correction_transform(element.node)
+
+                    # l10n: the name of a command symbol (example: scissors icon for trim command)
+                    INKSCAPE_LABEL: _("command marker"),
                 }
             )