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

Add link to inline-css #2

Merged
merged 1 commit into from May 19, 2015
Merged

Add link to inline-css #2

merged 1 commit into from May 19, 2015

Conversation

nimitzshah
Copy link
Contributor

Used roots-yaml as reference.

@@ -22,6 +21,12 @@ a roots extension for inlining your CSS properties into the style attribute in a
extensions: [ rootsInlineCss(options) ]
```

### Usage

This extension provides a way for you to inline your html with css. By default all of your `html` files will be inlined however you can specify through options which files you want inlined either with a single file or a [glob expression](http://www.globtester.com/).
Copy link
Contributor

Choose a reason for hiding this comment

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

i'd rather we reference the actual https://github.com/isaacs/minimatch library, since it will provide the dev with api reference

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes sense.

@nimitzshah nimitzshah force-pushed the docs branch 2 times, most recently from ce1f55e to a8503b8 Compare May 19, 2015 15:57
@@ -10,9 +10,12 @@ a roots extension for inlining your CSS properties into the style attribute in a

- make sure you are in your roots project directory
- `npm install roots-inline-css --save`
-
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra whitespace

@@ -22,6 +24,10 @@ a roots extension for inlining your CSS properties into the style attribute in a
extensions: [ rootsInlineCss(options) ]
```

This extension provides a way for you to inline your html with css. By default all of your `html` files will be inlined however you can specify through options which files you want inlined either with a single file or a [glob expression]( https://github.com/isaacs/minimatch).
Copy link
Contributor

Choose a reason for hiding this comment

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

extra space in markdown anchor src

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This space was to separate the explanation from the code snippet. Should I still remove it?

Copy link
Contributor

Choose a reason for hiding this comment

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

extra space not extra newline.

[glob expression]( https://github.com/isaacs/minimatch)

should become

[glob expression](https://github.com/isaacs/minimatch)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed that arrays and hashes have a space between the bracket and element. Is this for readability?
[ rootsInlineCss(options) ] as an example.

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah

kylemac added a commit that referenced this pull request May 19, 2015
Add link to inline-css
@kylemac kylemac merged commit 7c3cca9 into master May 19, 2015
@kylemac kylemac deleted the docs branch May 19, 2015 19:42
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.

None yet

2 participants