Unlink clone recursive is now toggleable

pull/2834/head
CapellanCitizen 2024-04-11 13:25:22 -04:00
rodzic ce668501ab
commit a1997ac002
2 zmienionych plików z 12 dodań i 5 usunięć

Wyświetl plik

@ -3,7 +3,7 @@
# Copyright (c) 2010 Authors
# Licensed under the GNU GPL version 3.0 or later. See the file LICENSE for details.
import inkex
from inkex import Boolean, errormsg, BaseElement
from ..elements import Clone
from ..i18n import _
@ -13,20 +13,26 @@ from typing import List, Tuple
class UnlinkClone(InkstitchExtension):
def __init__(self, *args, **kwargs):
InkstitchExtension.__init__(self, *args, **kwargs)
self.arg_parser.add_argument("-r", "--recursive", dest="recursive", type=Boolean, default=True)
def effect(self):
recursive: bool = self.options.recursive
if not self.get_elements():
return
if not self.svg.selection:
inkex.errormsg(_("Please select one or more clones to unlink."))
errormsg(_("Please select one or more clones to unlink."))
return
# Two passes here: One to resolve all clones, and then another to replace those clones with their resolved versions.
# This way we don't accidentally remove a node that another clone refers to.
clones_resolved: List[Tuple[inkex.BaseElement, inkex.BaseElement]] = []
clones_resolved: List[Tuple[BaseElement, BaseElement]] = []
for element in self.elements:
if isinstance(element, Clone):
resolved = element.resolve_clone(recursive=False)
resolved = element.resolve_clone(recursive=recursive)
clones_resolved.append((element.node, resolved))
for (clone, resolved) in clones_resolved:

Wyświetl plik

@ -3,7 +3,8 @@
<name>Unlink Clone</name>
<id>org.{{ id_inkstitch }}.unlink_clone</id>
<param name="extension" type="string" gui-hidden="true">unlink_clone</param>
<effect>
<param name="recursive" type="boolean" gui-text="Recursive">true</param>
<effect needs-live-preview="false">
<object-type>all</object-type>
<effects-menu>
<submenu name="{{ menu_inkstitch }}" translatable="no">