Modify ↓

Opened 3 years ago

Closed 3 years ago

#5276 closed defect (fixed)

[PATCH] Preset/Icon for man_made=flagpole

Reported by: jcollie Owned by: christeck
Priority: normal Component: Internal preset
Version: Keywords:
Cc:

Description

Here's a patch and icon for man_made=flagpole

Attachments (2)

flagpole.diff​ (1.6 KB) - added by jcollie 3 years ago.
patch
flagpole.png​ (457 bytes) - added by jcollie 3 years ago.
icon

Download all attachments as: .zip

Change History (11)

Changed 3 years ago by jcollie

patch

Changed 3 years ago by jcollie

icon

comment:1 Changed 3 years ago by bastiK

  • Component changed from Core to Internal preset
  • Owner changed from team to christeck
  • Summary changed from Preset/Icon for man_made=flagpole to [PATCH] Preset/Icon for man_made=flagpole

comment:2 Changed 3 years ago by ce

  • Resolution set to fixed
  • Status changed from new to closed

(In [3391]) fixes #5276 Current build is broken so I cannot test whether the preset tags the elements correctly and whether the nodes are shown in the viewport with the correct icon. I assume yes for both.

comment:3 Changed 3 years ago by bastiK

  • Resolution fixed deleted
  • Status changed from closed to reopened

Does not work.

Build is not broken, JOSM source is in Java 6, now.

comment:4 Changed 3 years ago by jcollie

Is there a reason that the key was changed to man_made=flagpost? As far as I can tell man_made=flagpost is not used whereas man_made=flagpole has 89 uses. Also why was the icon I provided not used? The "presets/flag.png" is a 32x32 icon which is larger than the 16x16 icons generally used on the map.

comment:5 Changed 3 years ago by ce

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [3395]) preset icon cut to 32x32. added mapstyle icon 16x16. adjusted elemstyles accordingly. fixes #5276

comment:6 Changed 3 years ago by bastiK

Why is this icon so big (32x32)? Normally mappaint icons are smaller.

comment:7 Changed 3 years ago by bastiK

  • Resolution fixed deleted
  • Status changed from closed to reopened

see above

comment:8 Changed 3 years ago by christeck

The tag is set correctly now, the mappaint icon has been separated by the presets icon and shows up at a size of 16px as mappaint icons sould do.

May I ask which problem still persists?

comment:9 Changed 3 years ago by bastiK

  • Resolution set to fixed
  • Status changed from reopened to closed

Sorry, it's alright. I guess I had an old version in my local svn when I tested it...

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.