Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

reorganize Readme #4375

Merged
merged 4 commits into from
Jul 18, 2013
Merged

reorganize Readme #4375

merged 4 commits into from
Jul 18, 2013

Conversation

tomByrer
Copy link
Contributor

@tomByrer tomByrer commented Jul 1, 2013

Moved paragraphs around so least technical (most user-oriented) is on top, then tech/contributing (hacker info) is near the end. This allowed to streamline the prose, but I tried to avoid changing the text as much as possible.

Linked blog in welcome section (great for finding features; helps users decide if Brackets is for them).

Used H4 instead of bold for sub-headers.
cheers!

tomByrer added 3 commits July 1, 2013 06:08
Moved paragraphs around so least technical (most user-oriented) is on top, then tech/contributing (hacker info) is near the end.  This allowed to streamline the prose, but I tried to avoid changing the text as much as possible.

Linked blog in welcome section (great for finding features; helps users decide if Brackets is for them).

Used H4 vs **bold** for sub-headers.
(note to self; don't Push with only 4 hours of sleep)
@tomByrer
Copy link
Contributor Author

tomByrer commented Jul 3, 2013

(got feedback)

@tomByrer
Copy link
Contributor Author

tomByrer commented Jul 9, 2013

used insight, thanks @njx !

Look Good enough to merge?

@ghost ghost assigned njx Jul 10, 2013
@@ -1,8 +1,6 @@
Welcome to Brackets! [![Build Status](https://travis-ci.org/adobe/brackets.png?branch=master)](https://travis-ci.org/adobe/brackets)
-------------------

Installers for the latest build can be [downloaded here](http://download.brackets.io/).
Copy link

Choose a reason for hiding this comment

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

We want to keep this at the top since it's important that people know how to get an installed build (it's not so easy for people to get started with Brackets just from the github repos, since they have to build the shell, etc.).

Copy link

Choose a reason for hiding this comment

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

Never mind, I see that it's in a clearly marked section below.

@njx
Copy link

njx commented Jul 18, 2013

This is great, thanks for doing this! I'm going to go ahead and merge it as is, then just make the tweaks I mentioned above myself. Thanks again.

njx pushed a commit that referenced this pull request Jul 18, 2013
@njx njx merged commit 5a28176 into adobe:master Jul 18, 2013
@tomByrer tomByrer deleted the patch-1 branch July 18, 2013 03:01
@tomByrer
Copy link
Contributor Author

Thank you @njx for both the merge & critique. I'll keep your comments in mind for future edits.

I hope in the future it is mentioned that "extensions" are brackets' "plug-ins"; will help some understand better IMHO.

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