Modify ↓

Opened 14 years ago

Closed 14 years ago

#5276 closed defect (fixed)

[PATCH] Preset/Icon for man_made=flagpole

Reported by: jcollie Owned by: ce
Priority: normal Milestone:
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 14 years ago.
patch
flagpole.png​ (457 bytes ) - added by jcollie 14 years ago.
icon

Download all attachments as: .zip

Change History (11)

by jcollie, 14 years ago

Attachment: flagpole.diff​ added

patch

by jcollie, 14 years ago

Attachment: flagpole.png​ added

icon

comment:1 by bastiK, 14 years ago

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

comment:2 by ce, 14 years ago

Resolution: → fixed
Status: new → 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 by bastiK, 14 years ago

Resolution: fixed
Status: closed → reopened

Does not work.

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

comment:4 by jcollie, 14 years ago

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 by ce, 14 years ago

Resolution: → fixed
Status: reopened → closed

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

comment:6 by bastiK, 14 years ago

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

comment:7 by bastiK, 14 years ago

Resolution: fixed
Status: closed → reopened

see above

comment:8 by ce, 14 years ago

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 by bastiK, 14 years ago

Resolution: → fixed
Status: reopened → closed

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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain ce.
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.