Merge pull request #371 from alphagov/fix-progress-aria

Update the aria-valuenow on batch upload
alphagov · Aug 5, 2014 · 4286303 · 4286303
2 parents 2b63594 + 008a7b3
commit 4286303
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/app/assets/javascripts/modules/batch_progress.js b/app/assets/javascripts/modules/batch_progress.js
@@ -36,6 +36,7 @@
         percent = (progress.done/progress.total * 100).toFixed(0);
         percentDone.text(percent);
         bar.css('width', percent + '%');
+        bar.attr('aria-valuenow', percent);
         timeout = setTimeout(requestProgress, 1000);
       }
 

diff --git a/spec/javascripts/spec/batch_progress.spec.js b/spec/javascripts/spec/batch_progress.spec.js
@@ -50,6 +50,10 @@ describe('A batch progress module', function() {
       expect(element.find('.js-progress-bar').attr('style')).toMatch('width: 10%;');
     });
 
+    it('updates the aria value now attribute', function() {
+      expect(element.find('.js-progress-bar').attr('aria-valuenow')).toMatch(10);
+    });
+
     it('updates the count of done mappings', function() {
       expect(element.find('.js-progress-message').text()).toBe('1 of 10 mappings added');
     });