From 92e163a7939c1eae65c285fd893739917affc959 Mon Sep 17 00:00:00 2001
From: Mario Voigt <mariovoigt89@t-online.de>
Date: Sat, 15 Aug 2020 16:31:38 +0200
Subject: [PATCH] Improved Bounding Boxes

---
 extensions/fablabchemnitz_boundingbox.inx |  1 +
 extensions/fablabchemnitz_boundingbox.py  | 54 +++++++++++++----------
 2 files changed, 31 insertions(+), 24 deletions(-)

diff --git a/extensions/fablabchemnitz_boundingbox.inx b/extensions/fablabchemnitz_boundingbox.inx
index 9d48b424..03848ef4 100644
--- a/extensions/fablabchemnitz_boundingbox.inx
+++ b/extensions/fablabchemnitz_boundingbox.inx
@@ -6,6 +6,7 @@
     <param min="-10000.0" max="10000.0" name="offset" type="float" gui-text="Offset from object (all directions)">0.0</param>
     <param name="box" type="bool" gui-text="Draw boxes">true</param>
     <param name="circle" type="bool" gui-text="Draw circles">false</param>
+    <param name="split" type="bool" gui-text="Handle selection as group">True</param>
     <effect>
         <object-type>all</object-type>
         <effects-menu>
diff --git a/extensions/fablabchemnitz_boundingbox.py b/extensions/fablabchemnitz_boundingbox.py
index 4833de41..7a9b5e17 100644
--- a/extensions/fablabchemnitz_boundingbox.py
+++ b/extensions/fablabchemnitz_boundingbox.py
@@ -10,30 +10,36 @@ class DrawBBoxes(inkex.Effect):
         self.arg_parser.add_argument('--offset', type=float, default=0.0, help='Offset from object (all directions)')
         self.arg_parser.add_argument('--box', type=inkex.Boolean, default=0.0, help='Draw boxes')
         self.arg_parser.add_argument('--circle', type=inkex.Boolean, default=0.0, help='Draw circles')
-		
+        self.arg_parser.add_argument('--split', type = inkex.Boolean, default = True, help = 'Handle selection as group')
+      
+    def drawBBox(self, node):
+        bbox = node.bounding_box()
+        if self.options.box:
+            attribs = {
+                'style' : str(inkex.Style({'stroke':'#ff0000','stroke-width'  : '1','fill':'none'})),
+                'x'     : str(bbox.left - self.options.offset),
+                'y'     : str(bbox.top - self.options.offset),
+                'width' : str(bbox.width + 2 * self.options.offset),
+                'height': str(bbox.height + 2 * self.options.offset),
+            }
+            etree.SubElement(self.svg.get_current_layer(), inkex.addNS('rect','svg'), attribs)
+		    	
+        if self.options.circle:			    	
+            attribs = {
+                'style': str(inkex.Style({'stroke':'#ff0000','stroke-width'  : '1','fill':'none'})),
+                'cx'   : str(bbox.center_x),
+                'cy'   : str(bbox.center_y),
+                #'r'   : str(bbox.width / 2 + self.options.offset),
+                'r'    : str(math.sqrt((bbox.width + 2 * self.options.offset)* (bbox.width + 2 * self.options.offset) + (bbox.height + 2 * self.options.offset) * (bbox.height + 2 * self.options.offset)) / 2),
+            }
+            etree.SubElement(self.svg.get_current_layer(), inkex.addNS('circle','svg'), attribs)
+      
     def effect(self):
         if len(self.svg.selected) > 0:
-            bboxes = [(id, node, node.bounding_box()) for id, node in self.svg.selected.items()]
-			
-            if self.options.box:
-                for id, node, bbox in bboxes:
-                    attribs = {
-                        'style' : str(inkex.Style({'stroke':'#ff0000','stroke-width'  : '1','fill':'none'})),
-                        'x'     : str(bbox.left - self.options.offset),
-                        'y'     : str(bbox.top - self.options.offset),
-                        'width' : str(bbox.width + 2 * self.options.offset),
-                        'height': str(bbox.height + 2 * self.options.offset),
-                    }
-                    etree.SubElement(self.svg.get_current_layer(), inkex.addNS('rect','svg'), attribs )
-			    	
-            if self.options.circle:			    	
-                for id, node, bbox in bboxes:
-                    attribs = {
-                        'style': str(inkex.Style({'stroke':'#ff0000','stroke-width'  : '1','fill':'none'})),
-                        'cx'   : str(bbox.center_x),
-                        'cy'   : str(bbox.center_y),
-                        #'r'   : str(bbox.width / 2 + self.options.offset),
-                        'r'    : str(math.sqrt((bbox.width + 2 * self.options.offset)* (bbox.width + 2 * self.options.offset) + (bbox.height + 2 * self.options.offset) * (bbox.height + 2 * self.options.offset)) / 2),
-                    }
-                    etree.SubElement(self.svg.get_current_layer(), inkex.addNS('circle','svg'), attribs )
+            if self.options.split is False:
+                for id, item in self.svg.selected.items():
+                    self.drawBBox(item)
+            else:
+                self.drawBBox(self.svg.selected)
+
 DrawBBoxes().run()
\ No newline at end of file