Modify ↓

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12372 closed enhancement (fixed)

Include shop=perfumery in presets

Reported by: naoliv Owned by: team
Priority: normal Milestone: 16.02
Component: Internal preset Version:
Keywords: Cc:

Description

Could we have ​shop=perfumery included in the presets, please?

taginfo gives us 590 objects with shop=perfumery

Attachments (2)

JOSM_svg_icons_beauty_cosmetics_perfumery__CC0_Public_Domain.7z​ (2.7 KB ) - added by zermes 9 years ago.
JOSM_icons_beauty_cosmetics_perfumery.png​ (1.8 KB ) - added by zermes 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by Klumbumbus, 9 years ago

Milestone: → 16.02

If we include, then we should also include the more used ​shop=cosmetics. For perfumery we could use ​http://wiki.openstreetmap.org/wiki/File:Perfumery-14.svg, but we need an icon for cosmetics

comment:2 by zermes, 9 years ago

I will make one, although we already have for cosmetics ​https://wiki.openstreetmap.org/wiki/File:Beauty-14.svg but is not optimized for 16px nor aligned to the grid.

comment:3 by Klumbumbus, 9 years ago

Yes. However this icon is used for shop=beauty in osm carto. it would be confusing to use it for cosmetics in josm.

by zermes, 9 years ago

Attachment: JOSM_svg_icons_beauty_cosmetics_perfumery__CC0_Public_Domain.7z​ added

by zermes, 9 years ago

Attachment: JOSM_icons_beauty_cosmetics_perfumery.png​ added

comment:4 by zermes, 9 years ago

You are right. Didn't pay attention to the file name. Here they are with one more for shop=beauty, because when I finished the cosmetics icon with lipstick I noticed there was already one in JOSM with a lipstick.


comment:5 by naoliv, 9 years ago

Resolution: → fixed
Status: new → closed

Fixed in [9550].
See 12297#comment:38

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

comment:6 by Klumbumbus, 9 years ago

In 9550/josm:

fix #12372 - add shop=cosmetics and shop=perfumery to preset and mappaint style (icons by zermes), move old icons to nodist folder; see also [o32009]

comment:7 by Klumbumbus, 9 years ago

uups, typo in ticket number. corrected.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.