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

Adding hotel and hostel POI icons #1031

Merged
merged 2 commits into from
Feb 10, 2024
Merged

Conversation

whubsch
Copy link
Contributor

@whubsch whubsch commented Jan 26, 2024

This PR adds two icons, one for hotel, motel, and guest_house, and a bunk-bed style one for hostel. I previously had included dormitory with the hostel icon, but on the Slack we determined that because dormitories are not open or available for use by the public, they should not be included in this PR. I have the figures in the hostel icon sleeping in different directions primarily to avoid users mistaking the icon for two hotels right on top of each other, even though I know that's not how most people sleep 😄

This PR makes progress on #692. Open to any feedback or suggestions!

Localhost link
Screenshot 2024-01-26 at 6 17 57 AM

Screenshot 2024-01-26 at 6 17 44 AM

@ZeLonewolf
Copy link
Member

I think these icons look great! Let's add a test location as well to the samples list in test/. The spot you linked above is fine, as well as this spot in Manhattan.

Copy link
Member

@ZeLonewolf ZeLonewolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look great, thanks for contributing!

@ZeLonewolf ZeLonewolf merged commit c3859ce into osm-americana:main Feb 10, 2024
6 checks passed
@wmisener wmisener mentioned this pull request Jul 19, 2024
@whubsch whubsch deleted the hotel-poi branch September 4, 2024 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants