Opened 8 years ago

Last modified 5 years ago

#15240 closed enhancement

transform gui icons to svg — at Version 72

Reported by: Klumbumbus Owned by: team
Priority: major Milestone: 20.08
Component: Core Version:
Keywords: svg png icon ex-longterm Cc:

Description (last modified by Klumbumbus)

Since we have an icon scaling on hidpi screens now (ticket:9995#comment:110) it would be nice to transform all icons to svg. Preset icons are already all svg (#13357)

Here are the current numbers: 60 svg icons and 270 png icons. This statistic excludes the folders images/presets, images/icons (only potlatch icons), our logo (which exists 6 times as png versions additionally to svg) and two special cases (lambert maps).

svg:

png:

I think we could ask the osm community for help on this subject.

There are two possibilities:

  • transform the 270 png to svg
  • create a whole new icon set with an consistent look

For the second, there was already such an (not fully complete) dataset, see JOSM themes:






progress since creation of this ticket:

https://docs.google.com/spreadsheets/d/e/2PACX-1vTerX1bI5UhTzyxaK3NyyvnaQp6YvaqjFcn6AbvhMUFfjrEUnEjyYetQMAYtQ0SQ5_GipVbMucQ0tVn/pubchart

Change History (117)

by Klumbumbus, 8 years ago

Attachment: png_icons.png added

by Klumbumbus, 8 years ago

Attachment: svg_icons.png added

comment:1 by Don-vip, 8 years ago

Keywords: svg png icon added
Priority: normalmajor

comment:2 by Klumbumbus, 8 years ago

Description: modified (diff)

in reply to:  description comment:3 by Don-vip, 8 years ago

Replying to Klumbumbus:

For the second, there was already such an (not fully complete) dataset, see JOSM themes. Sadly the links and screenshoots in the forum are no longer available. Maybe we could recover them.

I remember very well. Felis was one of the finalists for the LogoContest and he almost won. It was really a hard thing to choose between the two final designs. I love what he did, too bad we can't have two logos :D

I just sent him an e-mail to let it know about this ticket.

comment:4 by Felis Pimeja, 8 years ago

FYI, as long as Dropbox has changed sharing mechanics, theme files are now available at:

https://drive.google.com/file/d/0BzTaubB85qV2cEFPcnA2UTBZbG8/view?usp=sharing

And I'm searching for the source files now.

comment:5 by bastiK, 8 years ago

Our current icons are partly based on the Tango icon theme, which has an svg set. See also GNOME set (based on Tango) with icons for download, delete and others.

in reply to:  4 comment:6 by bastiK, 8 years ago

Replying to Felis Pimeja:

FYI, as long as Dropbox has changed sharing mechanics, theme files are now available at:

https://drive.google.com/file/d/0BzTaubB85qV2cEFPcnA2UTBZbG8/view?usp=sharing

And I'm searching for the source files now.

Wow, it's pretty good! I like the black-red color scheme.

by Don-vip, 8 years ago

Attachment: black-red.jpg added

by Don-vip, 8 years ago

Attachment: blue-red.jpg added

by Don-vip, 8 years ago

Attachment: monochrome.jpg added

by Don-vip, 8 years ago

Attachment: standart.jpg added

comment:7 by Don-vip, 8 years ago

Description: modified (diff)

comment:8 by Don-vip, 8 years ago

See also new default theme for Gnome 3: https://git.gnome.org/browse/adwaita-icon-theme/

comment:9 by stoecker, 8 years ago

The colorschemes like black-red do look good, but in my eyes they aren't good as the standard design. This should still be colorful (when possible with a bit more unified style :-).

comment:10 by bastiK, 8 years ago

How to move forward?

I noticed that some of the vector icons don't scale well down to very small sizes:

Current download icon (24x24px):
https://josm.openstreetmap.de/export/7668/josm/trunk/images/download.png
Scalable image, rendered at 48x48 px:
https://upload.wikimedia.org/wikipedia/commons/thumb/9/92/Gnome-document-save.svg/48px-Gnome-document-save.svg.png
Scalable image, rendered at 24x24 px
https://upload.wikimedia.org/wikipedia/commons/thumb/9/92/Gnome-document-save.svg/24px-Gnome-document-save.svg.png

Just replacing the current download.png by download.svg would be nice for HiDPI users, but clearly is a loss in image quality for everyone else. My conclusion so far is that some scalable images are designed with a certain minimum size in mind and will just look bad when scaled below that limit. This affects some icons more than others, the undo/redo image scales fine for example.

Possible solutions:

  • Find svg icons that scale well to 24x24 px (and decently well to 16x16 px for the menus).
  • Ship both the 24x24px "pixel art" and the .svg and use whatever image is better suited.

in reply to:  10 ; comment:11 by Don-vip, 8 years ago

Replying to bastiK:

Possible solutions:

  • Find svg icons that scale well to 24x24 px (and decently well to 16x16 px for the menus).
  • Ship both the 24x24px "pixel art" and the .svg and use whatever image is better suited.

I don't think we will succeed to find good SVG icons scalable to small sizes. It's already difficult enough to find a SVG version. This leaves only the second option.

in reply to:  11 ; comment:12 by stoecker, 8 years ago

Replying to Don-vip:

Replying to bastiK:

Possible solutions:

  • Find svg icons that scale well to 24x24 px (and decently well to 16x16 px for the menus).
  • Ship both the 24x24px "pixel art" and the .svg and use whatever image is better suited.

I don't think we will succeed to find good SVG icons scalable to small sizes. It's already difficult enough to find a SVG version. This leaves only the second option.

  • Change the icons, so that the vanishing elements (usually some sort of boundaries) are wider, so they don't vanish.

E.g. see point 7 here: ​https://www.creativebloq.com/how-to/10-golden-rules-for-responsive-svgs

in reply to:  12 ; comment:13 by bastiK, 8 years ago

Replying to stoecker:

Replying to Don-vip:

Replying to bastiK:

Possible solutions:

  • Find svg icons that scale well to 24x24 px (and decently well to 16x16 px for the menus).
  • Ship both the 24x24px "pixel art" and the .svg and use whatever image is better suited.

I don't think we will succeed to find good SVG icons scalable to small sizes. It's already difficult enough to find a SVG version. This leaves only the second option.

  • Change the icons, so that the vanishing elements (usually some sort of boundaries) are wider, so they don't vanish.

E.g. see point 7 here: ​https://www.creativebloq.com/how-to/10-golden-rules-for-responsive-svgs

This is more the job of a graphics designer that that of a programmer. Take the download icon posted above as an example: How can you modify the SVG without loss of quality in either the 24 px (top) or the 48 px rendering?

Anyway, if we implement option 2, it is still possible to improve the SVGs one by one, as suggested.

in reply to:  13 comment:14 by stoecker, 8 years ago

This is more the job of a graphics designer that that of a programmer. Take the download icon posted above as an example: How can you modify the SVG without loss of quality in either the 24 px (top) or the 48 px rendering?

Anyway, if we implement option 2, it is still possible to improve the SVGs one by one, as suggested.

Sure.

But the option 2 means a workaround which again will live MANY years. I personally would rather accept slightly worse low-zoom icons than another workaround.

comment:15 by stoecker, 8 years ago

P.S. We already had 24x24 menu icons. Maybe default simply should be 30x30 nowadays even for normal screens?

by floscher, 8 years ago

Attachment: statusline-svg.patch added

Proposal for SVG icons for the status line

by floscher, 8 years ago

Attachment: statusline.gif added

Animated comparison between the statusline icons (current and proposed)

comment:16 by floscher, 8 years ago

I propose to replace the icons in the statusline with SVG versions.
Below you see a comparison between the current icons (PNG, 18×18 pixels, scaled to 180×180), then how my proposed SVG icons look when converted to a 18×18 raster image scaled up to 180×180 pixels. And finally how the SVG icons look as 180×180 pixel image.
Animated comparison between the statusline icons (current and proposed)
(https://josm.openstreetmap.de/attachment/ticket/15240/statusline-svg.patch is the corresponding patch)

by floscher, 8 years ago

Attachment: statusline-before.png added

Current appearance of the statusline icons

by floscher, 8 years ago

Attachment: statusline-after.png added

Proposed appearance of the statusline icons

comment:17 by floscher, 8 years ago

Before:
Current appearance of the statusline icons
After:
Proposed appearance of the statusline icons

comment:18 by stoecker, 8 years ago

In 13363/josm:

see #15240 replace statusline png icons with svg variants by floscher, set mime-types for many images

comment:19 by stoecker, 8 years ago

No need to do such a impressive presentation. :) The new images and a before/after is enough. We plan to replace all PNG images anyway.

comment:20 by Don-vip, 8 years ago

Nice! thank you :)

comment:21 by Hb---, 8 years ago

References to the old PNG-files are still used in the wiki, please see https://josm.openstreetmap.de/wiki/Help/StatusBar
Wiki has been updated.

Last edited 8 years ago by Hb--- (previous) (diff)

comment:22 by bastiK, 8 years ago

In 13375/josm:

see #15240 - replace image for geoimage layer with svg version (from Tango icon project)

comment:23 by stoecker, 8 years ago

In 13376/josm:

see #15240 - set proper mime-type

comment:24 by Klumbumbus, 7 years ago

some icons replaced in #16159

by floscher, 7 years ago

Attachment: anglesnap.svg added

by floscher, 7 years ago

Attachment: alignline.svg added

by floscher, 7 years ago

Attachment: black_x.svg added

Replacement for misc/black_x.png

by floscher, 7 years ago

Attachment: green_check.svg added

Replacement for misc/green_check.png

by floscher, 7 years ago

Attachment: green_check.2.svg added

Replacement for misc/green_check.png

comment:25 by floscher, 7 years ago

Here are four more:
https://josm.openstreetmap.de/export/13612/josm/trunk/images/anglesnap.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/alignline.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/misc/black_x.png Replacement for misc/black_x.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/misc/green_check.png Replacement for misc/green_check.png

By the way: Can the icons dialogs/valid https://josm.openstreetmap.de/export/13612/josm/trunk/images/dialogs/valid.png and misc/check_large https://josm.openstreetmap.de/export/13612/josm/trunk/images/misc/check_large.png also be replaced by the checkmark above or is there a need for three different green checkmarks?

comment:26 by stoecker, 7 years ago

In 13613/josm:

see #15240 - new SVG icons

comment:27 by stoecker, 7 years ago

I didn't replace misc/check_large. It is used in the upload dialog and I'd say it looks better there than the boxy green_check.

comment:28 by Klumbumbus, 7 years ago

interim result: 246 png icons left :)

by Klumbumbus, 7 years ago

Attachment: 246png.png added

comment:29 by stoecker, 7 years ago

Hmm, are sure that's right? The green check is svg and the large is missing and some icons are in two sizes and 3 blue checkmarks?

comment:30 by stoecker, 7 years ago

BTW The WMS/TMS related icons are all ugly.

Better ones would be fine.

comment:31 by Klumbumbus, 7 years ago

It seems you accidently moved the large green check to nodist instead of source:trunk/images/misc/green_check.png.

comment:32 by Klumbumbus, 7 years ago

in the screenshot I scaled all icons to 24px. multiple icons appear when there are different sizes available, e.g. source:trunk/images/imagery_menu.pngsource:trunk/images/imagery_small.pngsource:trunk/images/preferences/imagery.png source:trunk/images/dialogs/validator.pngsource:trunk/images/layer/validator_small.pngsource:trunk/images/preferences/validator.png

should be unified when replayed by svg

comment:33 by Klumbumbus, 7 years ago

Code of the screenshot. Can be used here in wiki with comment preview feature. (But takes a bit to load all images which is why I made the screenshot)

edit: code outdated and removed

Last edited 5 years ago by Klumbumbus (previous) (diff)

in reply to:  31 comment:34 by stoecker, 7 years ago

Replying to Klumbumbus:

It seems you accidently moved the large green check to nodist instead of source:trunk/images/misc/green_check.png.

Ooops. Can someone please fix this before next build?

comment:35 by Klumbumbus, 7 years ago

could be a meme: JOSM user level expert: knows the difference between the upload icon and the upload selection icon
source:trunk/images/upload.pngsource:trunk/images/uploadselection.png

Last edited 7 years ago by Klumbumbus (previous) (diff)

comment:36 by Klumbumbus, 7 years ago

In 13615/josm:

see #15240 - fix moved icons

in reply to:  35 comment:37 by Don-vip, 7 years ago

Replying to Klumbumbus:

could be a meme: JOSM user level expert: knows the difference between the upload icon and the upload selection icon
source:trunk/images/upload.pngsource:trunk/images/uploadselection.png

hahaha :D

by floscher, 7 years ago

Attachment: separator.svg added

Replacement for preferences/separator.png

by anonymous, 7 years ago

Attachment: closedway.svg added

Replacement for data/closedway.png

by floscher, 7 years ago

Attachment: multipolygon.svg added

Replacement for data/multipolygon.png

by floscher, 7 years ago

Attachment: node.svg added

Replacement for data/node.png

by floscher, 7 years ago

Attachment: sequence.svg added

Replacement for data/sequence.png

by floscher, 7 years ago

Attachment: way.svg added

Replacement for data/way.png

by floscher, 7 years ago

Attachment: separator.2.svg added

Replacement for data/sequence.png

by floscher, 7 years ago

Attachment: aligncircle.svg added

Replacement for aligncircle.png

by floscher, 7 years ago

Attachment: taginfo.svg added

Replacement for dialogs/taginfo.png

comment:38 by floscher, 7 years ago

And some more:
https://josm.openstreetmap.de/export/13612/josm/trunk/images/preferences/separator.pngReplacement for data/sequence.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/data/closedway.pngReplacement for data/closedway.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/data/multipolygon.pngReplacement for data/multipolygon.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/data/node.pngReplacement for data/node.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/data/sequence.pngReplacement for data/sequence.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/data/way.pngReplacement for data/way.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/dialogs/taginfo.pngReplacement for dialogs/taginfo.png
https://josm.openstreetmap.de/export/13612/josm/trunk/images/aligncircle.pngReplacement for aligncircle.png

The last one I changed, because the line around really should be the circle (I think), not the ways connecting the nodes. So I changed the color (if you want some other color, we could change it). And I made the points more irregular to emphasize that the nodes are not evenly distributed around a circle, but only moved to the closest point on a circle.

comment:39 by stoecker, 7 years ago

Hmm. taginfo does still not look nice. It's the website favicon, but looks ugly. Adding the world logo from the website behind the ti?

in reply to:  39 comment:40 by anonymous, 7 years ago

Replying to stoecker:

Hmm. taginfo does still not look nice. It's the website favicon, but looks ugly. Adding the world logo from the website behind the ti?

Don't know if that would work at this resolution (16x16). Probably has to be something simpler.

comment:41 by Don-vip, 7 years ago

Maybe just frame it with something like https://png.icons8.com/metro/50/000000/note.png

comment:42 by stoecker, 7 years ago

Or let the t look a bit like a "tag"?

comment:43 by stoecker, 7 years ago

https://www.google.de/search?q=icon+tag
They all have a hole inside.
Or simply such a tag with an i or ti inside. When we have something better we'd need to ask Jochen if he likes it.

comment:44 by Klumbumbus, 7 years ago

Or we pixel align the current taginfo icon a bit to make it look a bit sharper.

Regarding https://josm.openstreetmap.de/export/13612/josm/trunk/images/aligncircle.pngReplacement for aligncircle.png I agree that there should be a circle in the icon instead of a pentagon to better illustrate the align in circle action. However the irregular pattern destroys this illustration again a bit in my opinion. And if you select a way and hit O then the nodes are indeed evenly distributed. The thin light green is pretty hard to recognize on a light background too.

Currently Help/Action/AlignInCircle and Help/Action/CreateCircle use the same icon. While the align action works on single nodes too, the create action always creates a way, so probably the black way lines shouldn't be removed from the create action icon.

My suggestion (but I didn't test how it looks): a circle and even distributed nodes for both icons; for the create action the circle is black and for the align action grey or another color (but a bit darker than the light green).

by floscher, 7 years ago

Attachment: aligncircle.2.svg added

New version for aligncircle.png

comment:45 by floscher, 7 years ago

Here's my draft for an alternative TagInfo logo:
Updated TagInfo logo with more contrast

And another version of aligncircle: New version for aligncircle.png.

@Klumbumbus: Actually it depends on what you select if the nodes distribute equally around the circle: When selecting a way, they do. When selecting only the nodes, they do not distribute evenly around the circle. I thought they were always only jumping to their nearest position on the circle, but it seems to be more complicated.
Anyway, I now changed the icon to show evenly distributed nodes, also for aesthetic reasons. Also the color is now grey. I think a second version in black would be too much, especially because they are too similar… Not that in the end your joke about the download icons really becomes a meme 😉

in reply to:  45 ; comment:46 by stoecker, 7 years ago

Replying to floscher:

Here's my draft for an alternative TagInfo logo:

I contacted Jochen and asked for his opinion.

comment:47 by stoecker, 7 years ago

In 13630/josm:

see #15240 - new SVG icons made by floscher

by floscher, 7 years ago

Attachment: apply.svg added

Replacement for apply.png

comment:48 by Klumbumbus, 7 years ago

(replacement for gray_check.png source:trunk/images/misc/gray_check.png can be derived from the green and blue one too)

by floscher, 7 years ago

Attachment: gray_check.svg added

Replacement for misc/gray_check.png

comment:49 by floscher, 7 years ago

You might consider renaming gray_check.svg to grey_check.svg. Next to it there is the file grey_x.png, which uses the other spelling.

comment:50 by Hb---, 7 years ago

On Plugins the images /browser/trunk/images/misc/black_x.png and ../green_check.png are missing now.

They are placed by a script in the last column to indicate if a plugin needs a restart. The alt parameters for the images are localized. This script needs to be modified.

Replacing the former PNG images with their SVG counterparts will make the page display in the Help Browser worse, because that Browser can't properly resize images currently. Simple characters like ✓ and ✕ may give the same information.

in reply to:  50 ; comment:51 by Klumbumbus, 7 years ago

Did someone fix this already? The icons are displayed fine for me. (I remember that i fixed wiki:JavaBugs and wiki:OldJavaBugs 5 weeks ago and I think that the Plugins page was already working fine a that point.)

Replying to Hb---:

Replacing the former PNG images with their SVG counterparts will make the page display in the Help Browser worse, because that Browser can't properly resize images currently.

That shouldn't be a problem anyway in this case as these two icons have a small base size and shouldn't be resized.

Last edited 7 years ago by Klumbumbus (previous) (diff)

in reply to:  51 comment:52 by stoecker, 7 years ago

Replying to Klumbumbus:

Did someone fix this already?

Yes.

comment:53 by Don-vip, 7 years ago

In 13832/josm:

fix #5603, see #5256, see #15240 - add first/last buttons for geoimage dialog. Fixed shortcuts. Switch to SVG icons taken from https://commons.wikimedia.org/wiki/GNOME_Desktop_icons#Navigation

comment:54 by Don-vip, 7 years ago

In 13837/josm:

see #15240 - convert more GNOME icons to SVG versions. Taken from https://commons.wikimedia.org/wiki/Category:GNOME_Desktop_icons

comment:55 by Klumbumbus, 7 years ago

duplicate icons:
source:trunk/images/help/previous.svgsource:trunk/images/dialogs/previous.svg
source:trunk/images/help/next.svgsource:trunk/images/dialogs/next.svg

Last edited 7 years ago by Klumbumbus (previous) (diff)

comment:56 by Don-vip, 7 years ago

In 13844/josm:

see #15240 - remove duplicated icons

in reply to:  46 comment:57 by Don-vip, 7 years ago

Replying to stoecker:

I contacted Jochen and asked for his opinion.

any news?

comment:58 by Don-vip, 7 years ago

Another thing: the icon warnings scripts says bookmark.svg is not used, but it is:

        static ImageIcon getDefaultIcon() {
            return ImageProvider.get("dialogs", "bookmark", ImageSizes.SMALLICON);
        }

by stoecker, 7 years ago

Attachment: ti_jochen.png added

TagInfo image from Jochen Topf

comment:59 by stoecker, 7 years ago

TagInfo image from Jochen Topf

Jochen sent me this image which he also uses (which is to large for our purpose), but didn't answer whether he likes or does not like the small icon.

I'd say we simply use floscher's icon in JOSM.

comment:60 by stoecker, 7 years ago

In 13857/josm:

see #15240 - detect another type of ImageProvider calls for in image check script

by Don-vip, 7 years ago

Attachment: taginfo_preview.png added

comment:61 by Don-vip, 7 years ago

It's nice but the contrast should be increased, it's hard to view the tag on grey background:


by floscher, 7 years ago

Attachment: taginfo.2.svg added

Updated TagInfo logo with more contrast

by floscher, 7 years ago

Attachment: taginfo_preview2.png added

Preview of the context menu with TagInfo logo

comment:62 by floscher, 7 years ago

Should now be better:
Updated TagInfo logo with more contrast
Preview of the context menu with TagInfo logo

comment:63 by Don-vip, 7 years ago

In 13860/josm:

see #15240 - use new taginfo icon (by floscher)

comment:64 by Klumbumbus, 7 years ago

In 13862/josm:

see #15240 - new SVG icons made by floscher

comment:65 by Klumbumbus, 7 years ago

In 13863/josm:

see #15240 - add svg mime type

by floscher, 7 years ago

Attachment: rectangle.svg added

by floscher, 7 years ago

Attachment: normal.svg added

comment:66 by floscher, 7 years ago

Replacements for some icons in the directory images/cursor/modifier:
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/rectangle.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_add.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/selection.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_add.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_remove.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_remove.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_way.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_way_add.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_way_remove.png

As well as these two in images/cursor:
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/crosshair.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/normal.png

Last edited 7 years ago by Don-vip (previous) (diff)

by floscher, 7 years ago

Attachment: crosshair.2.svg added

by floscher, 7 years ago

Attachment: crosshair.svg added

comment:67 by Don-vip, 7 years ago

In 13871/josm:

see #15240 - switch 12 cursor icons to SVG (by floscher)

comment:68 by Klumbumbus, 7 years ago

Did you accidently not pixel align the red node symbols at these three icons?

https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_add.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_remove.png

by floscher, 7 years ago

Attachment: select_add.svg added

by floscher, 7 years ago

Attachment: selection.svg added

by floscher, 7 years ago

Attachment: select_node.svg added

by floscher, 7 years ago

Attachment: select_node_add.svg added

by floscher, 7 years ago

Attachment: select_node_remove.svg added

by floscher, 7 years ago

Attachment: select_remove.svg added

by floscher, 7 years ago

Attachment: select_way.svg added

by floscher, 7 years ago

Attachment: select_way_add.svg added

by floscher, 7 years ago

Attachment: select_way_remove.svg added

in reply to:  68 comment:69 by floscher, 7 years ago

Replying to Klumbumbus:

Did you accidently not pixel align the red node symbols at these three icons?

https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_add.png
https://josm.openstreetmap.de/export/13870/josm/trunk/images/cursor/modifier/select_node_remove.png

Yes, good catch, thank you! I corrected the icons. And I also made a small modification to the other select-modifiers, which shouldn't change the appearance in JOSM, but in some image viewing programs (e.g. Gimp) the dashed line should display correctly.

comment:70 by Klumbumbus, 7 years ago

In 13874/josm:

see #15240 - update 9 SVG icons made by floscher, add svg mime type

comment:71 by Don-vip, 7 years ago

@bastiK @michael2402 The test MapCSSRendererTest.testRender[area-fill-image] is both failing and slower on Jenkins since r13862, could you please have a look? Surprisingly the test is OK on my machine.

comment:72 by Klumbumbus, 7 years ago

Description: modified (diff)

add progress chart to ticket description, little motivation ;)

Note: See TracTickets for help on using tickets.