#47, set aria-disabled on next/prev links

diff --git a/extensions/navigation/deck.navigation.js b/extensions/navigation/deck.navigation.js
index bf53737..643cef8 100644 (file)
--- a/extensions/navigation/deck.navigation.js
+++ b/extensions/navigation/deck.navigation.js
@@ -1,6 +1,6 @@
 /*!
 Deck JS - deck.navigation
-Copyright (c) 2011 Caleb Troughton
+Copyright (c) 2011-2013 Caleb Troughton
 Dual licensed under the MIT license.
 https://github.com/imakewebthings/deck.js/blob/master/MIT-license.txt
 */
@@ -24,8 +24,10 @@ This module adds clickable previous and next links to the deck.
     var $nextButton = $(options.selectors.nextLink);
 
     $prevButton.toggleClass(options.classes.navDisabled, to === 0);
+    $prevButton.attr('aria-disabled', to === 0);
     $prevButton.attr('href', hrefBase + '#' + (prevId ? prevId : ''));
     $nextButton.toggleClass(options.classes.navDisabled, to === lastIndex);
+    $nextButton.attr('aria-disabled', to === lastIndex);
     $nextButton.attr('href', hrefBase + '#' + (nextId ? nextId : ''));
   };
 

diff --git a/test/spec.navigation.js b/test/spec.navigation.js
index 9fd180f..db04794 100644 (file)
--- a/test/spec.navigation.js
+++ b/test/spec.navigation.js
@@ -29,12 +29,22 @@ describe('Deck JS Navigation Buttons', function() {
     expect($(defaults.selectors.previousLink)).toHaveClass(defaults.classes.navDisabled);
   });
 
+  it('should add aria-disabled to previous link if on first slide', function() {
+    $.deck('go', 0);
+    expect($(defaults.selectors.previousLink)).toHaveAttr('aria-disabled', 'true');
+  });
+
   it('should add the disabled class to the next link if on last slide', function() {
     expect($(defaults.selectors.nextLink)).not.toHaveClass(defaults.classes.navDisabled);
     $.deck('go', $.deck('getSlides').length - 1);
     expect($(defaults.selectors.nextLink)).toHaveClass(defaults.classes.navDisabled);
   });
 
+  it('should add aria-disabled to next link if on last slide', function() {
+    $.deck('go', $.deck('getSlides').length - 1);
+    expect($(defaults.selectors.nextLink)).toHaveAttr('aria-disabled', 'true');
+  });
+
   it('should not start disabled if deck initialized in the middle', function() {
     $.deck('go', 2);
     $.deck('.slide');
@@ -50,4 +60,4 @@ describe('Deck JS Navigation Buttons', function() {
     expect($(defaults.selectors.previousLink).attr('href')).toMatch('#custom-id');
     expect($(defaults.selectors.nextLink).attr('href')).toMatch('#slide-3');
   });
-});
\ No newline at end of file
+});