Skip to content

Updates the extensions README to include added tags#1834

Closed
Meggin wants to merge 1 commit intoampproject:masterfrom
Meggin:extensions-readme
Closed

Updates the extensions README to include added tags#1834
Meggin wants to merge 1 commit intoampproject:masterfrom
Meggin:extensions-readme

Conversation

@Meggin
Copy link
Copy Markdown
Contributor

@Meggin Meggin commented Feb 8, 2016

New pull request that cherry picks updates to extensions table only.

| [`amp-install-serviceworker`](amp-install-serviceworker/amp-install-serviceworker.md) | Installs a ServiceWorker. |
| [`amp-lightbox`](amp-lightbox/amp-lightbox.md) | Allows for a “lightbox” or similar experience. |
| [`amp-list`](amp-list/amp-list.md) | Dynamically downloads data and creates list items using a template. |
| [`amp-mustache`](amp-mustache/amp-mustache.md) | Allows rendering of [`Mustache.js`](https://github.com/janl/mustache.js/) templates. |
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.

Don't think Mustache.js needs to be formatted as code (...).

@rudygalfi
Copy link
Copy Markdown
Contributor

One really tiny thing. LGTM

@Meggin
Copy link
Copy Markdown
Contributor Author

Meggin commented Feb 16, 2016

Not only did this have merge conflicts, but I noticed more new components to add. So I went and did that in this pull request which has been merged: #2052. So closing this one.

@Meggin Meggin closed this Feb 16, 2016
@Meggin Meggin deleted the extensions-readme branch February 16, 2016 20:05
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