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

Improves Guides index#882

Merged
michaelrkn merged 1 commit into
masterfrom
improve-index
Oct 28, 2015
Merged

Improves Guides index#882
michaelrkn merged 1 commit into
masterfrom
improve-index

Conversation

@locks
Copy link
Copy Markdown
Contributor

@locks locks commented Oct 15, 2015

I await feedback.

  • Replace TDK
  • Tidy it up
  • Squash

@locks locks changed the title Improves Guides index [WIP] Improves Guides index Oct 15, 2015
Comment thread source/index.md
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

JSON? HTML?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use of a keyboard and mouse? ;)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, Handlebars syntax.

@michaelrkn
Copy link
Copy Markdown
Contributor

Looking good! Line notes to follow.

Comment thread source/index.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nitpicky, but since this is the front page, I think it's worth being nitpicky: technically this sentence contains a comma splice that should be fixed.

@locks locks force-pushed the improve-index branch 2 times, most recently from 05a97a6 to 401c8b5 Compare October 19, 2015 15:27
@locks
Copy link
Copy Markdown
Contributor Author

locks commented Oct 19, 2015

Handlebars syntax missing but alas, that's in another castle.

@locks locks changed the title [WIP] Improves Guides index Improves Guides index Oct 19, 2015
Comment thread source/index.md Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

link?

michaelrkn added a commit that referenced this pull request Oct 28, 2015
@michaelrkn michaelrkn merged commit da3e2d9 into master Oct 28, 2015
@michaelrkn michaelrkn deleted the improve-index branch October 28, 2015 05:48
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.

3 participants