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.

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

add alternate atm icon, which is inspired by OSM-carto #78

Merged
merged 1 commit into from
Oct 31, 2022

Conversation

tyrasd
Copy link
Contributor

@tyrasd tyrasd commented Oct 31, 2022

Replaces the current atm icon () with a more abstract one which should work better at small scale (especially the $ sign is problematic in the old version):

This icon is based on openstreetmap-carto (), but keeps the bill from "maki" () for consistency with the icon used for the bank preset.

See openstreetmap/id-tagging-schema#626

@bhousel
Copy link
Collaborator

bhousel commented Oct 31, 2022

What we've done in the past is to make an alternate icon named like atm2 rather than removing the existing one, in case people still want to use it.

contrary to OSM-carto this one keeps the bill from "maki" for consistency

for openstreetmap/id-tagging-schema#626
@tyrasd
Copy link
Contributor Author

tyrasd commented Oct 31, 2022

that fine with me, I've updated the PR accordingly

@tyrasd tyrasd changed the title replace atm icon with one inspired by OSM-carto add alternate atm icon, which is inspired by OSM-carto Oct 31, 2022
@bhousel bhousel merged commit 6bda938 into rapideditor:main Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants