Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#12035 closed enhancement (fixed)

Add office=* items to internal presets

Reported by: Don-vip Owned by: team
Priority: normal Milestone: 15.11
Component: Internal preset Version:
Keywords: office license Cc: bastiK

Description (last modified by Don-vip)

office is widely used (150k occurences) but yet missing from the internal presets, although some values are rendered by the default map style.

We have examples in Presets/NewTags.

Some ideas for icons:

Tag 16px 24px
`office=accountant` https://cdn4.iconfinder.com/data/icons/24x24-free-application-icons/24/Accounting.png https://cdn4.iconfinder.com/data/icons/24x24-free-application-icons/24/Accounting.png
`office=advertising_agency` https://cdn3.iconfinder.com/data/icons/fugue/icon/megaphone.png https://cdn3.iconfinder.com/data/icons/fugue/bonus/icons-24/megaphone.png
`office=employment_agency` https://cdn4.iconfinder.com/data/icons/cologne/16x16/free-for-job.png https://cdn4.iconfinder.com/data/icons/cologne/32x32/free-for-job.png
`office=government` https://cdn4.iconfinder.com/data/icons/Project_Icons___Version_1_1_9_by_bogo_d/PNG/Library.png https://cdn4.iconfinder.com/data/icons/Project_Icons___Version_1_1_9_by_bogo_d/PNG/Library.png
`office=insurance` https://cdn4.iconfinder.com/data/icons/24x24-free-pixel-icons/24/Shield.png https://cdn4.iconfinder.com/data/icons/24x24-free-pixel-icons/24/Shield.png
`office=it` https://cdn2.iconfinder.com/data/icons/gnomeicontheme/16x16/devices/gnome-dev-computer.png https://cdn2.iconfinder.com/data/icons/gnomeicontheme/24x24/devices/gnome-dev-computer.png
`office=newspaper` https://cdn2.iconfinder.com/data/icons/fugue/icon/newspaper.png https://cdn2.iconfinder.com/data/icons/fugue/bonus/icons-24/newspaper.png
`office=research` https://cdn2.iconfinder.com/data/icons/gnomeicontheme/16x16/categories/applications-science.png https://cdn2.iconfinder.com/data/icons/gnomeicontheme/24x24/categories/applications-science.png

Attachments (0)

Change History (24)

comment:1 by Klumbumbus, 8 years ago

Owner: changed from team to Klumbumbus
Status: newassigned

comment:2 by Klumbumbus, 8 years ago

In 9011/josm:

see #12035 - add office=* items to internal preset and mappaint style

comment:3 by Klumbumbus, 8 years ago

This is with generic icon for now.

I added all values from the table at Key:office except:

I did not add the following values because they are duplicates: realtor and real_estate_agent (duplicate of estate_agent), tax (duplicate of government), travel_agent (duplicate of shop=travel_agency)

I did not add the following values because of low usage numbers: forestry (146) , guide (28), private_investigator (6), quango (148), register (134), water_utility (65)

(numbers from taginfo from today)

comment:4 by aceman, 8 years ago

Thanks!

comment:5 by aceman, 8 years ago

May there be strings clashes here with existing strings which cause problems for translators. I'm thinking e.g. "administrative" (boundary), telecommunication (tower:type), religion (religion key). Is a context needed?

in reply to:  5 comment:6 by Klumbumbus, 8 years ago

Replying to aceman:

Is a context needed?

It is already in there. name_context="office"

comment:7 by aceman, 8 years ago

Ah yes, sorry.

comment:8 by Klumbumbus, 8 years ago

Owner: changed from Klumbumbus to team
Status: assignednew

If individual icons are wanted then I would need some help, because finding/editing a good icon takes a lot of time sometimes. The icons in Presets/NewTags are problematic, because the source and license is unclear and some are german specific.

Please see icon guidelines => DevelopersGuide/DefaultPresets

comment:10 by Don-vip, 8 years ago

Description: modified (diff)

comment:11 by Don-vip, 8 years ago

Description: modified (diff)

comment:12 by Don-vip, 8 years ago

Description: modified (diff)

comment:13 by Don-vip, 8 years ago

Description: modified (diff)

comment:14 by Don-vip, 8 years ago

Description: modified (diff)

comment:15 by Don-vip, 8 years ago

Description: modified (diff)

comment:16 by Don-vip, 8 years ago

Description: modified (diff)

comment:17 by Don-vip, 8 years ago

Description: modified (diff)

comment:18 by Don-vip, 8 years ago

Description: modified (diff)

comment:19 by Klumbumbus, 8 years ago

What about the license? I see that your linked icons are GPL or CC-BY. Do we need to add text to the about window. when using icons with such a license?

comment:20 by Don-vip, 8 years ago

Cc: bastiK added
Keywords: license added

I assume a mention in the last section of CONTRIBUTION file would be enough?

comment:21 by Don-vip, 8 years ago

Paul, is it OK to add CC-BY icons?

comment:22 by Don-vip, 8 years ago

Resolution: fixed
Status: newclosed

Let's continue the discussion to #12136.

comment:23 by Don-vip, 8 years ago

Ticket #12213 has been marked as a duplicate of this ticket.

comment:24 by Klumbumbus, 8 years ago

In 9681/josm:

see #12227 - add craft=* to preset and mappaint style, warn about craft=yes; fix office area style (see #12035); add traffic_sign to validator ignore list

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 .
 
Note: See TracTickets for help on using tickets.