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

Adds screenshots to places docs #319

Merged
merged 3 commits into from
Mar 2, 2017
Merged

Adds screenshots to places docs #319

merged 3 commits into from
Mar 2, 2017

Conversation

ecgreb
Copy link
Collaborator

@ecgreb ecgreb commented Mar 1, 2017

Overview

Adds screenshots to places docs

Proposed Changes

Update autocomplete-ui.md and places.md

Fixes #313

@ecgreb ecgreb changed the title Adds autocomplete image Adds screenshots to places docs Mar 1, 2017
@ecgreb ecgreb requested a review from kkowalsky March 1, 2017 22:59
@ecgreb
Copy link
Collaborator Author

ecgreb commented Mar 1, 2017

@kkowalsky I added the same places screenshots we are using in the blog post to the docs pages. Let me know if you think these work where I added them and if its ok to use the same images from the blog hosted on s3.

@kkowalsky
Copy link
Member

This is great @ecgreb! 👍

@ecgreb ecgreb merged commit a1b9c02 into master Mar 2, 2017
@ecgreb ecgreb deleted the 313-places-screenshots branch March 2, 2017 19:25
@ecgreb ecgreb removed the in progress label Mar 2, 2017
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