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

Issue #624: Add webapp manifest #628

Merged
merged 1 commit into from
Jun 10, 2015
Merged

Issue #624: Add webapp manifest #628

merged 1 commit into from
Jun 10, 2015

Conversation

miketaylr
Copy link
Member

So uh, I don't know who supports this yet. But @marcoscaceres asked for some live examples to test out his Gecko parser.

r? @magsout

@miketaylr
Copy link
Member Author

I suppose I need to configure nginx to serve .webmanifest files as application/json too.

@magsout
Copy link
Member

magsout commented Jun 10, 2015

@miketaylr

So uh, I don't know who supports this yet

you're talking about browser ?

@miketaylr
Copy link
Member Author

Yeah.

@magsout
Copy link
Member

magsout commented Jun 10, 2015

@magsout
Copy link
Member

magsout commented Jun 10, 2015

Maybe we can add css and js and font ?

@miketaylr
Copy link
Member Author

Ah, oops. This is actually a little different than a cache.manifest. The spec is here: https://w3c.github.io/manifest/

(and yes, offline is still a goal ^_^)

@magsout
Copy link
Member

magsout commented Jun 10, 2015

aaaaahh

@magsout
Copy link
Member

magsout commented Jun 10, 2015

LGTM even if I can't test ;p

magsout added a commit that referenced this pull request Jun 10, 2015
Issue #624: Add webapp manifest
@magsout magsout merged commit 8d1fda4 into master Jun 10, 2015
@magsout magsout deleted the issues/624/1 branch June 10, 2015 20:38
@miketaylr
Copy link
Member Author

Thanks! I guess Marcos will be testing for us. 😄

@marcoscaceres
Copy link

http://caniuse.com/#search=manifest
that's a different thing, btw ^_^

@marcoscaceres
Copy link

Ah, should read the whole thread before commenting... 😅

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.

3 participants