Skip to content

Replaced 'browser-feature-detection.md' with gist 2285900 & moved it. #108

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

Closed
wants to merge 2 commits into from
Closed

Conversation

connor
Copy link
Contributor

@connor connor commented Apr 12, 2012

As per the discussion in the previous PR with @addyosmani and @ajpiano, the content has been updated and moved to a better directoy.

…wser-feature-detection` to /code-organization, instead of /using-jquery-core
### Other resources to check out re: feature detection

- [Polyfilling the HTML5 gap](http://addyosmani.com/polyfillthehtml5gaps/slides/#1) by Addy Osmani
- [Feature, browser, and form factor detection](http://addyosmani.com/polyfillthehtml5gaps/slides/#1) by Michael Mahemoff
Copy link
Member

Choose a reason for hiding this comment

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

bad link btw

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah - thanks, paul! fixed it. don't know what i was thinking... =/

@ajpiano
Copy link
Member

ajpiano commented Apr 13, 2012

Thanks @ConnorMontgomery! Looks good, but I can't help asking: What's the deal with the changes to the syntax highlighter CSS?

@connor
Copy link
Contributor Author

connor commented Apr 13, 2012

@ajpiano whoah - i didn't think I added it on the CLI. I saw that when I ran a git status, but for some reason it just snuck in there.

Honestly, I don't remember touching it, but will get that removed from this PR - sorry man!

@addyosmani
Copy link
Member

@connor yo. Could you update this PR to reflect the latest requests above? We could probably land after that. Thanks :)

@connor
Copy link
Contributor Author

connor commented May 17, 2012

For sure - had lost track of it! Will have it updated tonight :)

On May 17, 2012, at 2:18 PM, Addy Osmanireply@reply.github.com

@addyosmani
Copy link
Member

Thanks!

@connor
Copy link
Contributor Author

connor commented May 18, 2012

Closing this PR - it'll be easier to just make a new one.

@connor connor closed this May 18, 2012
@connor
Copy link
Contributor Author

connor commented May 18, 2012

This has been updated here.

<3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants