Merge pull request #2758 from benwbrum/accessibility-fixes

Accessibility fixes
benwbrum · Sep 13, 2021 · cb62adf · cb62adf
2 parents c892bf8 + 391632f
commit cb62adf
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/app/views/collection/_facet_form.html.slim b/app/views/collection/_facet_form.html.slim
@@ -26,18 +26,18 @@ h3 =t('.filter')
     -@collection.facet_configs.each do |facet_config|
       -unless facet_config.label.blank?
         -if facet_config.input_type == "date"
-          h4 =facet_config.label
           =form_for :facet_search, :url => collection_search_path(@collection.owner, @collection), remote: true, html: { class: "collection-search", id: "search#{count}" } do |f|
-            =f.text_field :date, readonly: true
-            =f.hidden_field :label, value: facet_config.label
-            =f.hidden_field :date_order, value: facet_config.order
+            h4 =f.label "facet_config_date_#{count}", facet_config.label
+            =f.text_field :date, readonly: true, id: "facet_search_facet_config_date_#{count}"
+            =f.hidden_field :label, value: facet_config.label, id: "facet_config_label_#{count}"
+            =f.hidden_field :date_order, value: facet_config.order, id: "facet_config_order_#{count}"
             =f.button "Search", id: "slider-search#{count}"
             div id="slider#{count += 1}"
 -@collection.facet_configs.order(:order).each do |facet_config|
   -if !facet_config.label.blank? && facet_config.input_type == "text"
     -i = facet_config.order.to_i
     h4
-      a(href="#" class="facet-label" onclick="return false;" id="import-iiif-manifest")               =facet_config[:label]
+      a(href="#" class="facet-label" onclick="return false;" id="#{(facet_config[:label]).parameterize.underscore}")               =facet_config[:label]
     section style="display: none"
       -selector = "s#{i}".to_sym
       -[selector].each do |filter_name|
@@ -74,8 +74,10 @@ h3 =t('.filter')
           slide: function( event, ui ) {
             $( this.closest("form")[0] ).val( ui.values[ 0 ] + " - " + ui.values[ 1 ] );
           }
+        });  //.attr('aria-label', 'mycustomname')
+        $(".ui-slider-handle").each(function() {
+          $(this).attr("aria-label", "handle");
         });
-
         $( this.closest("form")[0] ).val( $( this ).slider( "values", 0 ) + " - " + $( this ).slider( "values", 1 ) );
       });
     });

diff --git a/app/views/static/software.html.slim b/app/views/static/software.html.slim
@@ -9,21 +9,21 @@
       h3 Transcription
       p Add and edit transcriptions while viewing scanned images of a document. Collaborate with others to transcribe a document.
 
-    =image_tag 'static/software/transcription.png'
+    =image_tag('static/software/transcription.png', alt: "Transcribe a document")
 
   section#annotation
     .content
       h3 Indexing & Annotation
       p Index a document by tagging people, places, and other subjects of interest. Search by subject and add articles to tagged subjects to create annotations for collaborators.
 
-    =image_tag 'static/software/indexing-annotation.png'
+    =image_tag('static/software/indexing-annotation.png', alt: "Index and Annotate")
 
   section#dashboard
     .content
       h3 Dashboard
       p See all the activity within a project at a glance with customized project owner and collaborator views.
 
-    =image_tag 'static/software/dashboard.png'
+    =image_tag('static/software/dashboard.png', alt: "Project Dashboard")
 
   #features
     section#version_control
@@ -45,7 +45,7 @@
   section#video
     h2 Easy to Get Started
     #video_wrapper
-      <iframe width="560" height="315" src="https://www.youtube.com/embed/UcNXSY0q9uE?controls=0" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
+      <iframe title="Get Started" width="560" height="315" src="https://www.youtube.com/embed/UcNXSY0q9uE?controls=0" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
 
   section#pricing
     h2 FromThePage Pricing