Permalink
Browse files

remove duplicate notifications test.

  • Loading branch information...
1 parent c15d314 commit f37592ec9e4a2c3c987e1ff90c2933b3eaf58043 @paulirish paulirish committed Jan 30, 2012
Showing with 3 additions and 14 deletions.
  1. +2 −1 feature-detects/notification.js
  2. +0 −12 feature-detects/notifications.js
  3. +1 −1 modernizr.js
@@ -3,7 +3,8 @@
// window.webkitNotifications is only used by Chrome
// http://www.html5rocks.com/en/tutorials/notifications/quick/
+
// window.Notification only exist in the draft specs
// http://dev.w3.org/2006/webapi/WebNotifications/publish/Notifications.html#idl-if-Notification
-Modernizr.addTest('notification', !!Modernizr.prefixed('Notifications', window) || !!window.Notification);
+Modernizr.addTest('notification', !!Modernizr.prefixed('Notifications', window));
@@ -1,12 +0,0 @@
-// http://dev.w3.org/2006/webapi/WebNotifications/publish/Notifications.html
-// http://dev.chromium.org/developers/design-documents/desktop-notifications/api-specification
-
-// the current webkit implementation is `webkitNotifications.createHTMLNotification`
-// however the spec uses `new Notification()`
-
-// we test for both, it up to you to use the right one with the correct signature.
-
-
-Modernizr.addTest('notifications', function(){
- return !!Modernizr.prefixed('Notification', window) || !!Modernizr.prefixed('notifications', window);
-}
View
@@ -917,7 +917,7 @@ window.Modernizr = (function( window, document, undefined ) {
modElem = inputElem = null;
//>>BEGIN IEPP
- // Enable HTML 5 elements for styling in IE.
+ // Enable HTML 5 elements for styling in IE & add HTML5 css
/*! HTML5 Shiv v3.1 | @jon_neal @afarkas @rem | MIT/GPL2 Licensed */
(function (win, doc) {

0 comments on commit f37592e

Please sign in to comment.