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

Place card shows website and phone number when available #268

Merged
merged 3 commits into from
Apr 4, 2023

Conversation

michaelkirk
Copy link
Member

This is some progress on #178 - starting with just the web and phone info.

I'm starting there because it's easy - it's already part of our pelias response.

I'm going to follow up with some rudimentary edit support - I think, for now anyway, this is going to look like "link back to OSM to edit", but I want to do a little more research about the best way to communicate that, especially for people who aren't familiar with OSM.

before
Screenshot 2023-04-03 at 2 28 07 PM

after

mobile

Screenshot 2023-04-03 at 2 27 00 PM

desktop

Screenshot 2023-04-03 at 2 26 28 PM

@michaelkirk michaelkirk merged commit fa07013 into main Apr 4, 2023
@michaelkirk michaelkirk deleted the mkirk/poi-details branch April 4, 2023 23:21
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.

1 participant