Skip to content
This repository has been archived by the owner on May 5, 2022. It is now read-only.

Added feature detection example #39

Closed
wants to merge 1 commit into from

Conversation

yoavweiss
Copy link
Contributor

Closes #7

@igrigorik - Can you take a look?

@yoavweiss
Copy link
Contributor Author

PRed the example in DOM. Will link to it from here once it's in.

@igrigorik
Copy link
Member

Great, thanks!

@toddreifsteck
Copy link
Member

LGTM

@yoavweiss yoavweiss mentioned this pull request Jan 7, 2016
@igrigorik
Copy link
Member

@yoavweiss what's the resolution here? are we going to drop in a non-normative note to link to the example?

@yoavweiss
Copy link
Contributor Author

The example in DOM as currently suggested may not be of great value to developers in my view.

So, we can link to an external example and/or simply write a short blog post that would explain the use of supports().

@igrigorik
Copy link
Member

@yoavweiss let's add a non-normative note in the spec and link to the gist (which, btw, should probably be .html to get the syntax highlighting). Once you have a blog post you can link it from the gist as well.

@igrigorik
Copy link
Member

@yoavweiss looks like the HTML spec PR landed, can we update this and resolve it? :)

@igrigorik
Copy link
Member

Closing, resolved via #52.

@igrigorik igrigorik closed this Jan 26, 2016
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