From bbb1156c0e64c04c8489f8b237c7bcc4478be6d7 Mon Sep 17 00:00:00 2001
From: Sebastian Tschan <sebastian.tschan@gmail.com>
Date: Sat, 26 May 2012 22:40:35 +0200
Subject: [PATCH] Omit aria-valuenow for the indeterminate global progress bar.

Remove the unnecessary aria-valuetext attribute.
---
 index.html                 | 4 ++--
 js/jquery.fileupload-ui.js | 7 +++----
 test/index.html            | 4 ++--
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/index.html b/index.html
index ba9b6ebd7..3ceaf5280 100644
--- a/index.html
+++ b/index.html
@@ -94,7 +94,7 @@ <h1>jQuery File Upload Demo</h1>
             <!-- The global progress information -->
             <div class="span5 fileupload-progress fade">
                 <!-- The global progress bar -->
-                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0" aria-valuetext="0%">
+                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100">
                     <div class="bar" style="width:0%;"></div>
                 </div>
                 <!-- The extended global progress information -->
@@ -157,7 +157,7 @@ <h3 class="modal-title"></h3>
             <td class="error" colspan="2"><span class="label label-important">{%=locale.fileupload.error%}</span> {%=locale.fileupload.errors[file.error] || file.error%}</td>
         {% } else if (o.files.valid && !i) { %}
             <td>
-                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0" aria-valuetext="0%"><div class="bar" style="width:0%;"></div></div>
+                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0"><div class="bar" style="width:0%;"></div></div>
             </td>
             <td class="start">{% if (!o.options.autoUpload) { %}
                 <button class="btn btn-primary">
diff --git a/js/jquery.fileupload-ui.js b/js/jquery.fileupload-ui.js
index 978294727..a7b7d464c 100644
--- a/js/jquery.fileupload-ui.js
+++ b/js/jquery.fileupload-ui.js
@@ -129,7 +129,6 @@
                             !$.support.transition && 'progress-animated'
                         )
                         .attr('aria-valuenow', 100)
-                        .attr('aria-valuetext', '100%')
                         .find('.bar').css(
                             'width',
                             '100%'
@@ -232,7 +231,6 @@
                     var progress = parseInt(data.loaded / data.total * 100, 10);
                     data.context.find('.progress')
                         .attr('aria-valuenow', progress)
-                        .attr('aria-valuetext', progress + '%')
                         .find('.bar').css(
                             'width',
                             progress + '%'
@@ -254,7 +252,6 @@
                 globalProgressNode
                     .find('.progress')
                     .attr('aria-valuenow', progress)
-                    .attr('aria-valuetext', progress + '%')
                     .find('.bar').css(
                         'width',
                         progress + '%'
@@ -274,7 +271,9 @@
                 var that = $(this).data('fileupload');
                 that._transition($(this).find('.fileupload-progress')).done(
                     function () {
-                        $(this).find('.bar').css('width', '0%').parent().attr('aria-valuenow', '0').attr('aria-valuetext', '0%');
+                        $(this).find('.progress')
+                            .attr('aria-valuenow', '0')
+                            .find('.bar').css('width', '0%');
                         $(this).find('.progress-extended').html('&nbsp;');
                         that._trigger('stopped', e);
                     }
diff --git a/test/index.html b/test/index.html
index 297475483..ac5a29cdf 100644
--- a/test/index.html
+++ b/test/index.html
@@ -53,7 +53,7 @@ <h2 id="qunit-userAgent"></h2>
             <!-- The global progress information -->
             <div class="span5 fileupload-progress fade">
                 <!-- The global progress bar -->
-                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0" aria-valuetext="0%">
+                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100">
                     <div class="bar" style="width:0%;"></div>
                 </div>
                 <!-- The extended global progress information -->
@@ -78,7 +78,7 @@ <h2 id="qunit-userAgent"></h2>
             <td class="error" colspan="2"><span class="label label-important">{%=locale.fileupload.error%}</span> {%=locale.fileupload.errors[file.error] || file.error%}</td>
         {% } else if (o.files.valid && !i) { %}
             <td>
-                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0" aria-valuetext="0%"><div class="bar" style="width:0%;"></div></div>
+                <div class="progress progress-success progress-striped active" role="progressbar" aria-valuemin="0" aria-valuemax="100" aria-valuenow="0"><div class="bar" style="width:0%;"></div></div>
             </td>
             <td class="start">{% if (!o.options.autoUpload) { %}
                 <button class="btn btn-primary">