Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use diffrent icons for amenity=atm and amenity=bank #626

Closed
Dimitar5555 opened this issue Oct 28, 2022 · 1 comment · Fixed by #630
Closed

Use diffrent icons for amenity=atm and amenity=bank #626

Dimitar5555 opened this issue Oct 28, 2022 · 1 comment · Fixed by #630
Labels
new-icon

Comments

@Dimitar5555
Copy link
Contributor

Currently both presets use the same icon which makes them indistinguishable unless you click on both of them. It would be nice if they had different icons like it's done in OSM-carto.

image
image
@tyrasd tyrasd added new-icon waitfor-icon a matching icon is missing for this preset labels Oct 31, 2022
@tyrasd
Copy link
Member

tyrasd commented Oct 31, 2022

temaki has an icon for atm's, but I think it's not very usable, as it's quite detailed (doesn't work well when rendered at a small scale), and uses the $ sign as currency placeholder, while maki (the current icon for banks and atms) uses a "generic" blob instead. I like the icon by openstreetmap-carto better. Let's replace the one in temaki with that one?!

tyrasd added a commit to tyrasd/temaki that referenced this issue Oct 31, 2022
@tyrasd
replace atm icon with one inspired by OSM-carto
575a700 
but keeping the bill from "maki" for consistency

for openstreetmap/id-tagging-schema#626
@tyrasd tyrasd mentioned this issue Oct 31, 2022
Merged
tyrasd added a commit to tyrasd/temaki that referenced this issue Oct 31, 2022
@tyrasd
ad alternative atm icon, inspired by OSM-carto
0603e49 
contrary to OSM-carto this one keeps the bill from "maki" for consistency

for openstreetmap/id-tagging-schema#626
bhousel pushed a commit to rapideditor/temaki that referenced this issue Oct 31, 2022
@tyrasd @bhousel
ad alternative atm icon, inspired by OSM-carto
6bda938 
contrary to OSM-carto this one keeps the bill from "maki" for consistency

for openstreetmap/id-tagging-schema#626
@arch0345 arch0345 mentioned this issue Nov 3, 2022
Merged
@tyrasd tyrasd removed the waitfor-icon a matching icon is missing for this preset label Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
new-icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Tree Stump Preset and update ATM icon arch0345/id-tagging-schema
2 participants
@tyrasd @Dimitar5555