Modify

Opened 10 years ago

Closed 4 years ago

Last modified 4 years ago

#9164 closed enhancement (fixed)

[PATCH] add more values for historic=*

Reported by: mkoniecz Owned by: team
Priority: normal Milestone: 20.07
Component: Internal preset Version:
Keywords: historic church Cc: lutz, escada, Klumbumbus

Description (last modified by skyper)

tags/historic=church is not really popular (barely over 1000 in existence), but historic key is already in presets and church tag makes sense, and AFAIK there is no better alternative for historic churches (both in presets and among more popular tags).

Attachments (4)

#9164_--_Historic_presets.patch (12.8 KB ) - added by hiddewie 4 years ago.
#9164_--_Historic_presets1.patch (31.8 KB ) - added by hiddewie 4 years ago.
9164_milestone.png (372 bytes ) - added by Klumbumbus 4 years ago.
#9164_--_Historic_presets2.patch (37.0 KB ) - added by hiddewie 4 years ago.
with milestone from nodist

Download all attachments as: .zip

Change History (31)

comment:1 by Don-vip, 10 years ago

Component: CoreInternal preset

comment:2 by skyper, 10 years ago

There exists an external presets for historic objects.

comment:3 by skyper, 10 years ago

Cc: lutz escada added
Keywords: historic added

Maybe, we can get the auther(s) of the external preset to submit a patch for the internal preset. There are quite some duplicate entries ATM and at least the entries which are already in the internal preset could be updated/expended in favour of the duplicat entries in the external preset.

comment:4 by Don-vip, 7 years ago

Summary: add historic=church to presetsadd historic=church

comment:5 by Don-vip, 7 years ago

Keywords: church added

comment:6 by skyper, 4 years ago

Description: modified (diff)
Summary: add historic=churchadd more values for historic=*

While fort is in defaultpresets some values with higher number and wiki pages are missing:

  1. charcoal_pile
  2. citywalls
  3. heritage
  4. mine_shaft
  5. shieling
  6. mine
  7. manor
  8. milestone
  9. church
  10. city_gate

by hiddewie, 4 years ago

comment:8 by hiddewie, 4 years ago

Patch added for all historic items with enough tag usage.

Icons:

comment:9 by hiddewie, 4 years ago

Summary: add more values for historic=*[PATCH] add more values for historic=*

comment:10 by simon04, 4 years ago

Cc: Klumbumbus added
Milestone: 20.06

comment:11 by Klumbumbus, 4 years ago

Every new item needs an own icon and not share an icon with another item thats confusing.

comment:12 by Klumbumbus, 4 years ago

Also the milestone icon doesn't work within JOSM.

comment:13 by Klumbumbus, 4 years ago

Milestone: 20.0620.07

comment:14 by hiddewie, 4 years ago

I have fixed some of the icons. Also they have been deduplicated from their non-historic counterparts.

The mine and mineshaft (historic) still use the same icon, just like mine and mineshaft (non-historic) use the same icon.

by hiddewie, 4 years ago

comment:15 by Klumbumbus, 4 years ago

Thx. The milestone icon still doesn't work well within JOSM though.

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

by Klumbumbus, 4 years ago

Attachment: 9164_milestone.png added

comment:16 by hiddewie, 4 years ago

OK that is strange. I took the milestone SVG from nodist, which does work well.

by hiddewie, 4 years ago

with milestone from nodist

comment:17 by Klumbumbus, 4 years ago

Resolution: fixed
Status: newclosed

In 16748/josm:

fix #9164 - Add more historic presets (patch by hiddewie, modified)

  • historic=citywalls (internal icon reused)
  • historic=city_gate (icon https://wiki.openstreetmap.org/wiki/File%3ACity-gate-14.svg, color changed, PD and CC0 licensed)
  • historic=manor (icon from https://wiki.openstreetmap.org/wiki/File:Manor-14.svg, PD and CC0 licensed)
  • historic=church (icon derived from internal icon, PD and CC0 licensed)
  • historic=charcoal_pile (icon derived from internal icon, PD and CC0 licensed)
  • historic=mine (icon derived from internal icon, PD and CC0 licensed)
  • historic=mine_shaft (icon derived from internal icon, PD and CC0 licensed)
  • historic=shieling (icon derived from internal icon, PD and CC0 licensed)
  • historic=milestone (icon self created, PD and CC0 licensed)

comment:18 by Klumbumbus, 4 years ago

I created a new milestone icon because we used that icon from nodist for railway=milestone or highway=milestone in the past and that might be confusing. I changed the historic mine icon, flipped it. I saw this on printed maps for disused mines.

What are the icon sources of church, charcoal_pile, mine and shieling? Or which did you create yourself?

comment:19 by Klumbumbus, 4 years ago

Resolution: fixed
Status: closedreopened

comment:20 by hiddewie, 4 years ago

Perfect, thanks.

Church, charcoal_pile, mine + mine_shaft and shieling are all modified icons from the JOSM code base, so under the same license.

comment:21 by Klumbumbus, 4 years ago

Thx.

comment:22 by Klumbumbus, 4 years ago

Resolution: fixed
Status: reopenedclosed

comment:23 by Klumbumbus, 4 years ago

Resolution: fixed
Status: closedreopened

Some icons do not work. Under investigation...

comment:24 by Klumbumbus, 4 years ago

In 16753/josm:

see #9164 - fix icon display

comment:25 by Klumbumbus, 4 years ago

Resolution: fixed
Status: reopenedclosed

In 16754/josm:

fix #9164 - fix icon display for real

comment:26 by Klumbumbus, 4 years ago

That was tricky ;)

in reply to:  18 comment:27 by gaben, 4 years ago

Replying to Klumbumbus:

I changed the historic mine icon, flipped it.

:D

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.