Merge pull request #92 from PlotPyStack/feature/91-svg-icons

Replace PNG icons with SVG icons

Author: PierreRaybaut

doc/intro/licenses.rst

@@ -42,12 +42,16 @@ Additional asset licenses:
 -------------------------------------------------------------------------------
 
 
-Icons under plotpy/images
-=========================
+Icons under plotpy/data/icons
+=============================
 
 
 The following lists the source, license and accompanying details for
-image and icon under the plotpy/images directory.
+image and icon under the plotpy/data/icons directory.
+
+The icons are SVG reconstructions (2026) of the historical PNG icons: the notices
+and licenses of the original artwork remain. They are also provided in the
+plotpy/data/icons/LICENSES.txt file.
 
 
 -------------------------------------------------------------------------------
@@ -66,7 +70,7 @@ Crystal Project Icons
 | Download (mirror): https://github.com/thecodingmachine/crystal-project
 | License: GNU Lesser General Public License 2.1 or later https://www.gnu.org/licenses/old-licenses/lgpl-2.1.html
 
-No modifications made.
+Modifications: SVG reconstruction of the historical PNG icons, 2026.
 
 
 All Icons are free software; you can redistribute it and/or modify it under the
@@ -85,31 +89,31 @@ https://www.gnu.org/licenses/lgpl.html
 
 Files covered:
 
-* plotpy/images/arrow_down.png
-* plotpy/images/arrow_up.png
-* plotpy/images/editors/copywop.png
-* plotpy/images/editors/edit.png
-* plotpy/images/editors/editclear.png
-* plotpy/images/editors/editcopy.png
-* plotpy/images/editors/editcut.png
-* plotpy/images/editors/editdelete.png
-* plotpy/images/editors/editpaste.png
-* plotpy/images/editors/edit_add.png
-* plotpy/images/eraser.png
-* plotpy/images/exit.png
-* plotpy/images/editors/imshow.png
-* plotpy/images/editors/insert.png
-* plotpy/images/magnifier.png
-* plotpy/images/editors/plot.png
-* plotpy/images/editors/rename.png
-* plotpy/images/fileclose.png
-* plotpy/images/fileimport.png
-* plotpy/images/filenew.png
-* plotpy/images/fileopen.png
-* plotpy/images/filesave.png
-* plotpy/images/filesaveas.png
-* plotpy/images/print.png
-* plotpy/images/save_all.png
+* plotpy/data/icons/arrow_down.svg
+* plotpy/data/icons/arrow_up.svg
+* plotpy/data/icons/cell_edit.svg
+* plotpy/data/icons/edit.svg
+* plotpy/data/icons/editors/edit.svg
+* plotpy/data/icons/editors/edit_add.svg
+* plotpy/data/icons/editors/editcopy.svg
+* plotpy/data/icons/editors/editdelete.svg
+* plotpy/data/icons/editors/editpaste.svg
+* plotpy/data/icons/editors/filesave.svg
+* plotpy/data/icons/editors/imshow.svg
+* plotpy/data/icons/editors/insert.svg
+* plotpy/data/icons/editors/plot.svg
+* plotpy/data/icons/editors/rename.svg
+* plotpy/data/icons/eraser.svg
+* plotpy/data/icons/exit.svg
+* plotpy/data/icons/fileclose.svg
+* plotpy/data/icons/fileimport.svg
+* plotpy/data/icons/filenew.svg
+* plotpy/data/icons/fileopen.svg
+* plotpy/data/icons/filesave.svg
+* plotpy/data/icons/filesaveas.svg
+* plotpy/data/icons/magnifier.svg
+* plotpy/data/icons/print.svg
+* plotpy/data/icons/save_all.svg
 
 -------------------------------------------------------------------------------
 
@@ -125,7 +129,7 @@ Copyright (c) 2008 Python Software Foundation
 | License: Python Software Foundation License Version 2 https://docs.python.org/3/license.html
 | Trademark: Python Software Foundation Trademark Usage Policy https://www.python.org/psf/trademarks/
 
-No modifications made.
+Modifications: SVG reconstruction of the historical PNG, 2026.
 
 
 The Python logo is a trademark of the Python Software Foundation,
@@ -153,7 +157,7 @@ https://docs.python.org/3/license.html
 
 Files covered:
 
-* plotpy/images/python.png
+* plotpy/data/icons/python.svg
 
 
 -------------------------------------------------------------------------------

doc/release_notes/release_2.12.md

@@ -6,6 +6,8 @@
 
 * **Images on high-DPI screens** — With a display scaling factor such as 125 %, 150 % or 200 %, images were rendered at the resolution of the plot canvas and then upscaled to the screen resolution: with a 200 % scaling, images were shown with half the resolution of the screen, and with a 125 % scaling, one screen pixel row and column out of five were duplicated, which made fine image details (e.g. noise, thin lines) look irregular. Images are now rendered at the resolution of the screen; printing and exporting are not affected. Rendering images at the screen resolution takes more time: for a 2000 × 2000 image shown in a 700 × 700 plot, about 8 ms instead of 7 ms with a 125 % scaling and 19 ms instead of 7 ms with a 200 % scaling (104 ms instead of 28 ms with the anti-aliasing interpolation). The new `image/device_resolution` option of the `plot` configuration section restores the previous behavior
 * **Raw image items** — `RawImageItem` objects were not drawn: each redraw failed with `ValueError: Invalid destination rectangle (expected tuple of 4 integers)`
+* **Blurry icons with display scaling** — PlotPy's icons (toolbar tools, item list, curve, marker and line styles, brush patterns, ...) were 16×16 pixel PNG images, which looked blurry with a display scaling greater than 100 % (e.g. 125 %, 150 %, 200 %). They are now SVG images, which remain sharp at any display scaling. Applications requesting a PlotPy icon by its former PNG name (e.g. `get_icon("csection.png")`) still get it with guidata 3.17.1 or later, whose `get_image_file_path` falls back to the SVG image with the same name. The notices of the original artwork are provided in `plotpy/data/icons/LICENSES.txt` (fixes [Issue #91](https://github.com/PlotPyStack/PlotPy/issues/91))
+* **Image lock tool icon** — The "Lock" tool of transformable images showed the "not found" icon until an image was selected: it now shows the lock icon
 
 ⚡ Performance:
 

plotpy/builder/annotation.py

@@ -53,7 +53,7 @@ def __get_annotationparam(
         readonly: bool | None = None,
         private: bool | None = None,
     ) -> AnnotationParam:
-        param = AnnotationParam(_("Annotation"), icon="annotation.png")
+        param = AnnotationParam(_("Annotation"), icon="annotation.svg")
         if title is not None:
             param.title = title
         if subtitle is not None:

plotpy/builder/curvemarker.py

@@ -260,7 +260,7 @@ def mcurve(self, *args, **kwargs) -> CurveItem | list[CurveItem]:
         basename = _("Curve")
         curves = []
         for yi, stylei in zip(y, style):
-            param = CurveParam(title=basename, icon="curve.png")
+            param = CurveParam(title=basename, icon="curve.svg")
             if "label" in kwargs:
                 param.label = kwargs.pop("label")
             else:
@@ -406,7 +406,7 @@ def curve(
             )
 
         basename = _("Curve")
-        param = CurveParam(title=basename, icon="curve.png")
+        param = CurveParam(title=basename, icon="curve.svg")
         if not title:
             global CURVE_COUNT
             CURVE_COUNT += 1
@@ -447,8 +447,8 @@ def merror(self, *args, **kwargs) -> ErrorBarCurveItem:
         """  # noqa: E501
         x, y, dx, dy, style = self.__get_arg_triple_errorbar(args)
         basename = _("Curve")
-        curveparam = CurveParam(title=basename, icon="curve.png")
-        errorbarparam = ErrorBarParam(title=_("Error bars"), icon="errorbar.png")
+        curveparam = CurveParam(title=basename, icon="curve.svg")
+        errorbarparam = ErrorBarParam(title=_("Error bars"), icon="errorbar.svg")
         if "label" in kwargs:
             curveparam.label = kwargs["label"]
         else:
@@ -569,8 +569,8 @@ def error(
             error(x, y, None, dy, marker='o', markerfacecolor='w')
         """
         basename = _("Curve")
-        curveparam = CurveParam(title=basename, icon="curve.png")
-        errorbarparam = ErrorBarParam(title=_("Error bars"), icon="errorbar.png")
+        curveparam = CurveParam(title=basename, icon="curve.svg")
+        errorbarparam = ErrorBarParam(title=_("Error bars"), icon="errorbar.svg")
         if not title:
             global CURVE_COUNT
             CURVE_COUNT += 1
@@ -627,8 +627,8 @@ def histogram(
             :py:class:`.HistogramItem` object
         """
         basename = _("Histogram")
-        histparam = HistogramParam(title=basename, icon="histogram.png")
-        curveparam = CurveParam(_("Curve"), icon="curve.png")
+        histparam = HistogramParam(title=basename, icon="histogram.svg")
+        curveparam = CurveParam(_("Curve"), icon="curve.svg")
         curveparam.read_config(CONF, "histogram", "curve")
         if not title:
             global HISTOGRAM_COUNT
@@ -980,7 +980,7 @@ def marker(
         Returns:
             :py:class:`.Marker` object
         """
-        param = MarkerParam(_("Marker"), icon="marker.png")
+        param = MarkerParam(_("Marker"), icon="marker.svg")
         param.read_config(CONF, "plot", "marker/cursor")
         if (
             color

plotpy/builder/image.py

@@ -218,7 +218,7 @@ def image(
 
         assert isinstance(xdata, (tuple, list)) and len(xdata) == 2
         assert isinstance(ydata, (tuple, list)) and len(ydata) == 2
-        param = ImageParam(title=_("Image"), icon="image.png")
+        param = ImageParam(title=_("Image"), icon="image.svg")
         data, filename, title = self._get_image_data(
             data, filename, title, to_grayscale=True
         )
@@ -370,7 +370,7 @@ def maskedimage(
 
         assert isinstance(xdata, (tuple, list)) and len(xdata) == 2
         assert isinstance(ydata, (tuple, list)) and len(ydata) == 2
-        param = MaskedImageParam(title=_("Image"), icon="image.png")
+        param = MaskedImageParam(title=_("Image"), icon="image.svg")
         data, filename, title = self._get_image_data(
             data, filename, title, to_grayscale=True
         )
@@ -468,7 +468,7 @@ def maskedxyimage(
         if isinstance(y, (list, tuple)):
             y = np.array(y)
 
-        param = MaskedXYImageParam(title=_("Image"), icon="image.png")
+        param = MaskedXYImageParam(title=_("Image"), icon="image.svg")
         assert data.ndim == 2, "Data must have 2 dimensions"
 
         self.__set_image_param(
@@ -533,7 +533,7 @@ def rgbimage(
         """
         assert isinstance(xdata, (tuple, list)) and len(xdata) == 2
         assert isinstance(ydata, (tuple, list)) and len(ydata) == 2
-        param = RGBImageParam(title=_("Image"), icon="image.png")
+        param = RGBImageParam(title=_("Image"), icon="image.svg")
         data, filename, title = self._get_image_data(
             data, filename, title, to_grayscale=False
         )
@@ -593,7 +593,7 @@ def quadgrid(
         Returns:
             :py:class:`.QuadGridItem` object
         """
-        param = QuadGridParam(title=_("Image"), icon="image.png")
+        param = QuadGridParam(title=_("Image"), icon="image.svg")
         self.__set_image_param(
             param,
             title,
@@ -682,7 +682,7 @@ def trimage(
         Returns:
             :py:class:`.TrImageItem` object
         """
-        param = TrImageParam(title=_("Image"), icon="image.png")
+        param = TrImageParam(title=_("Image"), icon="image.svg")
         data, filename, title = self._get_image_data(
             data, filename, title, to_grayscale=True
         )
@@ -758,7 +758,7 @@ def xyimage(
         Returns:
             :py:class:`.XYImageItem` object
         """
-        param = XYImageParam(title=_("Image"), icon="image.png")
+        param = XYImageParam(title=_("Image"), icon="image.svg")
         self.__set_image_param(
             param,
             title,
@@ -811,7 +811,7 @@ def imagefilter(
         Returns:
             :py:class:`.ImageFilterItem` object
         """
-        param = ImageFilterParam(_("Filter"), icon="funct.png")
+        param = ImageFilterParam(_("Filter"), icon="funct.svg")
         param.xmin, param.xmax, param.ymin, param.ymax = xmin, xmax, ymin, ymax
         if title is not None:
             param.label = title
@@ -900,7 +900,7 @@ def histogram2D(
             :py:class:`.Histogram2DItem` object
         """
         basename = _("2D Histogram")
-        param = Histogram2DParam(title=basename, icon="histogram2d.png")
+        param = Histogram2DParam(title=basename, icon="histogram2d.svg")
         if NX is not None:
             param.nx_bins = NX
         if NY is not None:

plotpy/builder/label.py

@@ -84,7 +84,7 @@ def label(
             make.label("Absolute position", "R", (0,0), "R")
         """
         basename = _("Label")
-        param = LabelParamWithContents(basename, icon="label.png")
+        param = LabelParamWithContents(basename, icon="label.svg")
         param.read_config(CONF, "plot", "label")
         if title:
             param.label = title
@@ -124,7 +124,7 @@ def legend(
             :py:class:`.LegendBoxItem` or
             :py:class:`.SelectedLegendBoxItem` object
         """
-        param = LegendParam(_("Legend"), icon="legend.png")
+        param = LegendParam(_("Legend"), icon="legend.svg")
         param.read_config(CONF, "plot", "legend")
         param.abspos = True
         param.absg = anchor
@@ -155,7 +155,7 @@ def info_label(
             :py:class:`.DataInfoLabel` object
         """
         basename = _("Computation")
-        param = LabelParam(basename, icon="label.png")
+        param = LabelParam(basename, icon="label.svg")
         param.read_config(CONF, "plot", "info_label")
         if title is not None:
             param.label = title

plotpy/builder/plot.py

@@ -365,7 +365,7 @@ def gridparam(
         Returns:
             :py:class:`.GridParam`: grid parameters
         """
-        gridparam = GridParam(title=_("Grid"), icon="lin_lin.png")
+        gridparam = GridParam(title=_("Grid"), icon="lin_lin.svg")
         gridparam.read_config(CONF, "plot", "grid")
         if background is not None:
             gridparam.background = background

plotpy/constants.py

@@ -59,9 +59,9 @@ class PlotType(enum.Enum):
 # ===============================================================================
 
 PARAMETERS_TITLE_ICON = {
-    "grid": (_("Grid..."), "grid.png"),
-    "axes": (_("Axes style..."), "axes.png"),
-    "item": (_("Parameters..."), "settings.png"),
+    "grid": (_("Grid..."), "grid.svg"),
+    "axes": (_("Axes style..."), "axes.svg"),
+    "item": (_("Parameters..."), "settings.svg"),
 }