Opened 11 years ago

Last modified 11 years ago

#10900 closed enhancement

[patch] bicycle_repair_station preset and icon — at Version 6

Reported by: brycenesbitt Owned by: team
Priority: normal Milestone: 15.01
Component: Internal preset Version:
Keywords: Cc:

Description (last modified by brycenesbitt)

Here's a preset for bicycle_repair_station with an icon.

If you're ever out cycling, and have a problem, you'll want one of these.
Note that some of them are associated with vending machines for tubes and patch kits.

Change History (13)

by brycenesbitt, 11 years ago

Attachment: wrench-icon.png added

by brycenesbitt, 11 years ago

Attachment: wrench-icon.svg added

comment:1 by brycenesbitt, 11 years ago

Summary: PATCH: bicycle_repair_station preset and icon[patch] bicycle_repair_station preset and icon

in reply to:  description comment:2 by Klumbumbus, 11 years ago

Replying to brycenesbitt:

Note I would have preferred to use "<check>" rather than "<combo>",
but I could not figure out how to support entering a negative like
"Pump=no".

A check has three states: key not set, yes and no.

comment:3 by Klumbumbus, 11 years ago

Milestone: 15.01

The keys make_model=* and service:bicycle:chaintool=* and not well documented in the wiki and not widely used. So they do not fulfill the requirements for the internal preset. See DevelopersGuide/DefaultPresets. indoor=* is also not documented and controversial.

service:bicycle:diy doens't make sense to me, since such a bicycle repair station is always diy. Otherwise it would be shop=bicycle + service:bicycle:repair=yes.

Can you provide a svg icon with both a bicycle and a wrench (maybe similar to amenity=bicycle_rental)? Also make sure, that it is visible on black background and license compatible, see DevelopersGuide/DefaultPresets#NewIcons.

in reply to:  description comment:4 by skyper, 11 years ago

Replying to brycenesbitt:

Note I would have preferred to use "<check>" rather than "<combo>",
but I could not figure out how to support entering a negative like
"Pump=no".

See TaggingPresets#Attributeentryandselectiondialogboxes and the paragraph below (TaggingPresets#Attributes). Think you are looking for value_on plus value_off.

by brycenesbitt, 11 years ago

Attachment: wrench-bike-icon.svg added

Wrench over bicycle icon, SVG format

by brycenesbitt, 11 years ago

Attachment: wrench-bike-icon_24x24.png added

by brycenesbitt, 11 years ago

Attachment: wrench-bike-icon-white.svg added

Render wrench and bike in white, with a transparent background. To see in inkscape draw a solid color shape behind this.

by brycenesbitt, 11 years ago

Removed indoor, used check

by brycenesbitt, 11 years ago

Attachment: bicycle_repair_station.diff added

Removed indoor, used <check>

comment:5 by brycenesbitt, 11 years ago

  • Updated the icon. It is licence compatible.
  • Removed the DIY tag. I was following practice, but it does not make any sense.
  • Recolored the icon to white.
  • Moved to using <check>

comment:6 by brycenesbitt, 11 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.