diff options
| author | Kaalleen <36401965+kaalleen@users.noreply.github.com> | 2025-07-11 22:14:52 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-07-11 22:14:52 +0200 |
| commit | 2235ec6571601e12be7eb3c74907668fb68bebd4 (patch) | |
| tree | 61d85cb615a1b2327283e89f09592e1eb85ad91c /lib/tartan/palette.py | |
| parent | 7bd72274780ccaeae8bbae6e761341079df21ecb (diff) | |
Fix issue with bad color names (#3816)
* fix issue with bad color names and define element colors at one place and reuse
* fix bad tartan color
* verify color in gradient block
* add thread color tests
* use default color behavior for elements linked to non-existing definitions (gradients)
* Added mypy change for tests (authored by: CapellanCitizen)
Diffstat (limited to 'lib/tartan/palette.py')
| -rw-r--r-- | lib/tartan/palette.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/lib/tartan/palette.py b/lib/tartan/palette.py index 25bd2100..b8c36331 100644 --- a/lib/tartan/palette.py +++ b/lib/tartan/palette.py @@ -8,9 +8,10 @@ import re from typing import TYPE_CHECKING, List, cast import wx -from inkex import Color +from inkex import Color, ColorError from .colors import string_to_color + if TYPE_CHECKING: from ..gui.tartan.stripe_panel import StripePanel @@ -175,7 +176,11 @@ class Palette: color = wx.Colour(color).GetAsString(wx.C2S_HTML_SYNTAX) except wx.PyNoAppError: # however when we render an embroidery element we do not want to open wx.App - color = str(Color(color).to_named()) + try: + color = str(Color(color).to_named()) + except ColorError: + color = None + if not color: color = '#000000' render = 0 |
