From abf0fe992e22e03babe2d8d3630e4fdad14fed79 Mon Sep 17 00:00:00 2001
From: decabeza <alberto@decabeza.es>
Date: Fri, 8 Jun 2018 19:32:57 +0200
Subject: [PATCH] Removes tablist and tab roles

---
 app/views/admin/poll/booth_assignments/show.html.erb   | 10 +++++-----
 app/views/admin/poll/questions/index.html.erb          |  6 +++---
 app/views/budgets/investments/_filter_subnav.html.erb  |  1 -
 app/views/budgets/investments/_milestones.html.erb     |  2 +-
 app/views/budgets/investments/show.html.erb            |  4 ++--
 .../legislation/proposals/_filter_subnav.html.erb      |  1 -
 .../legislation/proposals/_notifications.html.erb      |  2 +-
 app/views/legislation/proposals/show.html.erb          |  4 ++--
 app/views/polls/show.html.erb                          |  4 ++--
 app/views/proposals/_filter_subnav.html.erb            |  1 -
 app/views/proposals/_notifications.html.erb            |  2 +-
 app/views/proposals/show.html.erb                      |  4 ++--
 app/views/topics/show.html.erb                         |  4 ++--
 13 files changed, 21 insertions(+), 24 deletions(-)

diff --git a/app/views/admin/poll/booth_assignments/show.html.erb b/app/views/admin/poll/booth_assignments/show.html.erb
index e9dafe8f1..4e395c6aa 100644
--- a/app/views/admin/poll/booth_assignments/show.html.erb
+++ b/app/views/admin/poll/booth_assignments/show.html.erb
@@ -12,7 +12,7 @@
   </p>
 <% end %>
 
-<ul class="tabs" id="booths_tabs" role="tablist"
+<ul class="tabs" id="booths_tabs"
     data-deep-link="true"
     data-update-history="true"
     data-deep-link-smudge="true"
@@ -29,8 +29,8 @@
   </li>
 </ul>
 
-<div class="tabs-content" data-tabs-content="booths_tabs" role="tablist">
-  <div class="tabs-panel" id="tab-officers" role="tab">
+<div class="tabs-content" data-tabs-content="booths_tabs">
+  <div class="tabs-panel" id="tab-officers">
     <% if @booth_assignment.officers.empty? %>
       <div class="callout primary margin-top">
         <%= t("admin.poll_booth_assignments.show.no_officers") %>
@@ -51,7 +51,7 @@
     <% end %>
   </div>
 
-  <div class="tabs-panel" id="tab-recounts" role="tab">
+  <div class="tabs-panel" id="tab-recounts">
     <h3><%= t("admin.poll_booth_assignments.show.recounts_list") %></h3>
 
     <table id="totals">
@@ -91,7 +91,7 @@
       </tbody>
     </table>
   </div>
-  <div class="tabs-panel is-active" id="tab-results" role="tab">
+  <div class="tabs-panel is-active" id="tab-results">
     <%= render "results" %>
   </div>
 </div>
diff --git a/app/views/admin/poll/questions/index.html.erb b/app/views/admin/poll/questions/index.html.erb
index e19811253..5a326d8f4 100644
--- a/app/views/admin/poll/questions/index.html.erb
+++ b/app/views/admin/poll/questions/index.html.erb
@@ -7,14 +7,14 @@
   <%= render 'search' %>
 </div>
 
-<div class="tabs-content" data-tabs-content="questions-tabs" role="tablist">
+<div class="tabs-content" data-tabs-content="questions-tabs">
   <%= render "filter_subnav" %>
 
-  <div class="tabs-panel is-active" id="tab-questions" role="tab">
+  <div class="tabs-panel is-active" id="tab-questions">
     <%= render "questions" %>
   </div>
 
-  <div class="tabs-panel" id="tab-successful-proposals" role="tab">
+  <div class="tabs-panel" id="tab-successful-proposals">
     <%= render "successful_proposals" %>
   </div>
 </div>
diff --git a/app/views/budgets/investments/_filter_subnav.html.erb b/app/views/budgets/investments/_filter_subnav.html.erb
index 048302b2e..6dd7cdff7 100644
--- a/app/views/budgets/investments/_filter_subnav.html.erb
+++ b/app/views/budgets/investments/_filter_subnav.html.erb
@@ -1,6 +1,5 @@
 <ul class="tabs"
     id="investments_tabs"
-    role="tablist"
     data-deep-link="true"
     data-update-history="true"
     data-deep-link-smudge="true"
diff --git a/app/views/budgets/investments/_milestones.html.erb b/app/views/budgets/investments/_milestones.html.erb
index 366d5d65a..60b52f46e 100644
--- a/app/views/budgets/investments/_milestones.html.erb
+++ b/app/views/budgets/investments/_milestones.html.erb
@@ -1,4 +1,4 @@
-<div class="tabs-panel tab-milestones" id="tab-milestones" role="tab">
+<div class="tabs-panel tab-milestones" id="tab-milestones">
   <div class="row">
     <div class="small-12 column">
       <% if @investment.milestones.blank? %>
diff --git a/app/views/budgets/investments/show.html.erb b/app/views/budgets/investments/show.html.erb
index 3efb1f7d4..175e08854 100644
--- a/app/views/budgets/investments/show.html.erb
+++ b/app/views/budgets/investments/show.html.erb
@@ -16,8 +16,8 @@
   </div>
 </div>
 
-<div class="tabs-content" data-tabs-content="investments_tabs" role="tablist">
-  <div class="tabs-panel is-active" id="tab-comments" role="tab">
+<div class="tabs-content" data-tabs-content="investments_tabs">
+  <div class="tabs-panel is-active" id="tab-comments">
     <%= render partial: '/comments/comment_tree', locals: { comment_tree: @comment_tree,
                                                             comment_flags: @comment_flags,
                                                             display_comments_count: false } %>
diff --git a/app/views/legislation/proposals/_filter_subnav.html.erb b/app/views/legislation/proposals/_filter_subnav.html.erb
index 66a65fed7..6cbe91045 100644
--- a/app/views/legislation/proposals/_filter_subnav.html.erb
+++ b/app/views/legislation/proposals/_filter_subnav.html.erb
@@ -1,6 +1,5 @@
 <ul class="tabs"
     id="legislation_proposals_tabs"
-    role="tablist"
     data-deep-link="true"
     data-update-history="true"
     data-deep-link-smudge="true"
diff --git a/app/views/legislation/proposals/_notifications.html.erb b/app/views/legislation/proposals/_notifications.html.erb
index 6a693c601..69389b2c5 100644
--- a/app/views/legislation/proposals/_notifications.html.erb
+++ b/app/views/legislation/proposals/_notifications.html.erb
@@ -1,4 +1,4 @@
-<div class="tabs-panel" id="tab-notifications" role="tab">
+<div class="tabs-panel" id="tab-notifications">
   <div class="row">
     <div class="small-12 column">
       <% if @notifications.blank? %>
diff --git a/app/views/legislation/proposals/show.html.erb b/app/views/legislation/proposals/show.html.erb
index 4f7b08fe7..3c88dc307 100644
--- a/app/views/legislation/proposals/show.html.erb
+++ b/app/views/legislation/proposals/show.html.erb
@@ -119,8 +119,8 @@
   </div>
 </div>
 
-<div class="tabs-content" data-tabs-content="legislation_proposals_tabs" role="tablist">
-  <div class="tabs-panel is-active" id="tab-comments" role="tab">
+<div class="tabs-content" data-tabs-content="legislation_proposals_tabs">
+  <div class="tabs-panel is-active" id="tab-comments">
     <%= render "legislation/proposals/comments" %>
   </div>
 </div>
diff --git a/app/views/polls/show.html.erb b/app/views/polls/show.html.erb
index 7a2730a8d..d6e992dc1 100644
--- a/app/views/polls/show.html.erb
+++ b/app/views/polls/show.html.erb
@@ -124,10 +124,10 @@
 
   </div>
 
-  <div class="tabs-content" data-tabs-content="polls_tabs" role="tablist">
+  <div class="tabs-content" data-tabs-content="polls_tabs">
     <%= render "filter_subnav" %>
 
-    <div class="tabs-panel is-active" id="tab-comments" role="tab">
+    <div class="tabs-panel is-active" id="tab-comments">
       <%= render "comments" %>
     </div>
   </div>
diff --git a/app/views/proposals/_filter_subnav.html.erb b/app/views/proposals/_filter_subnav.html.erb
index 2132be6e5..47709c8aa 100644
--- a/app/views/proposals/_filter_subnav.html.erb
+++ b/app/views/proposals/_filter_subnav.html.erb
@@ -1,6 +1,5 @@
 <ul class="tabs"
     id="proposals_tabs"
-    role="tablist"
     data-deep-link="true"
     data-update-history="true"
     data-deep-link-smudge="true"
diff --git a/app/views/proposals/_notifications.html.erb b/app/views/proposals/_notifications.html.erb
index fe96ba741..fc684b8f0 100644
--- a/app/views/proposals/_notifications.html.erb
+++ b/app/views/proposals/_notifications.html.erb
@@ -1,4 +1,4 @@
-<div class="tabs-panel" id="tab-notifications" role="tab">
+<div class="tabs-panel" id="tab-notifications">
   <div class="row">
     <div id="proposal_notifications" class="small-12 column notification-body">
       <% if @notifications.blank? %>
diff --git a/app/views/proposals/show.html.erb b/app/views/proposals/show.html.erb
index 36e35920c..ed385ab05 100644
--- a/app/views/proposals/show.html.erb
+++ b/app/views/proposals/show.html.erb
@@ -224,8 +224,8 @@
   </div>
 </div>
 
-<div class="tabs-content" data-tabs-content="proposals_tabs" role="tablist">
-  <div class="tabs-panel is-active" id="tab-comments" role="tab">
+<div class="tabs-content" data-tabs-content="proposals_tabs">
+  <div class="tabs-panel is-active" id="tab-comments">
     <%= render "proposals/comments" %>
   </div>
 
diff --git a/app/views/topics/show.html.erb b/app/views/topics/show.html.erb
index 4f2660840..62c8a14ee 100644
--- a/app/views/topics/show.html.erb
+++ b/app/views/topics/show.html.erb
@@ -38,9 +38,9 @@
   </div>
 
 
-  <div class="tabs-content" data-tabs-content="topics-tabs" role="tablist">
+  <div class="tabs-content" data-tabs-content="topics-tabs">
     <%= render "topics/filter_subnav" %>
-    <div class="tabs-panel is-active" id="tab-comments" role="tab">
+    <div class="tabs-panel is-active" id="tab-comments">
       <%= render "topics/comments" %>
     </div>
   </div>