Skip to content
This repository has been archived by the owner on May 8, 2019. It is now read-only.

Link to new screenshot #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dpilafian
Copy link

Issue #5

@@ -6,7 +6,7 @@

<p align="center">Slate helps you create beautiful, intelligent, responsive API documentation.</p>

<p align="center"><img src="https://dl.dropboxusercontent.com/u/95847291/github%20images/slate/slate_screenshot_new.png" width=700 alt="Screenshot of Example Documentation created with Slate"></p>
<p align="center"><img src="https://raw.githubusercontent.com/lord/img/master/screenshot-slate.png" width=700 alt="Screenshot of Example Documentation created with Slate"></p>

Choose a reason for hiding this comment

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

We should probably add a screen to our own repo rather than link to one owned by someone else. @sibartlett check it!

Copy link
Author

Choose a reason for hiding this comment

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

Sounds reasonable. How about screenshot.png in the project's root folder?

Copy link
Author

@dpilafian dpilafian Jul 5, 2017

Choose a reason for hiding this comment

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

The old (broken) link also points outside the repo. Maybe the best (and easiest) thing to do is for this pull request to go through as-is in order to fix the README.md and then open a new issue to move the image into the repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants