Add ARIA landmark roles

ARIA landmark roles provide useful navigation features for assistive technology and helps people understand the content structure.
thoughtbot · May 6, 2017 · b445299 · b445299
1 parent 8e957f5
commit b445299
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/app/views/administrate/application/_navigation.html.erb b/app/views/administrate/application/_navigation.html.erb
@@ -7,7 +7,7 @@ for all resources in the admin dashboard,
 as defined by the routes in the `admin/` namespace
 %>
 
-<nav class="navigation">
+<nav class="navigation" role="navigation">
   <% Administrate::Namespace.new(namespace).resources.each do |resource| %>
     <%= link_to(
       display_resource_name(resource),

diff --git a/app/views/administrate/application/_search.html.erb b/app/views/administrate/application/_search.html.erb
@@ -1,4 +1,4 @@
-<form class="search">
+<form class="search" role="search">
   <span class="search__icon">
     <%= svg_tag "administrate/search.svg", "search", width: 16, height: 16 %>
   </span>

diff --git a/app/views/administrate/application/edit.html.erb b/app/views/administrate/application/edit.html.erb
@@ -17,7 +17,7 @@ It displays a header, and renders the `_form` partial to do the heavy lifting.
 
 <% content_for(:title) { "#{t("administrate.actions.edit")} #{page.page_title}" } %>
 
-<header class="header">
+<header class="header" role="banner">
   <h1 class="header__heading"><%= content_for(:title) %></h1>
   <div class="header__actions">
     <%= link_to(

diff --git a/app/views/administrate/application/index.html.erb b/app/views/administrate/application/index.html.erb
@@ -33,7 +33,7 @@ It renders the `_table` partial to display details about the resources.
   <% end %>
 <% end %>
 
-<header class="header">
+<header class="header" role="banner">
   <h1 class="header__heading" id="page-title"><%= content_for(:title) %></h1>
   <div class="header__actions">
     <%= link_to(

diff --git a/app/views/administrate/application/new.html.erb b/app/views/administrate/application/new.html.erb
@@ -17,7 +17,7 @@ to do the heavy lifting.
 
 <% content_for(:title) { "#{t("administrate.actions.new")} #{page.resource_name.titleize}" } %>
 
-<header class="header">
+<header class="header" role="banner">
   <h1 class="header__heading"><%= content_for(:title) %></h1>
   <div class="header__actions">
     <%= link_to t("administrate.actions.back"), :back, class: "button" %>

diff --git a/app/views/administrate/application/show.html.erb b/app/views/administrate/application/show.html.erb
@@ -18,7 +18,7 @@ as well as a link to its edit page.
 
 <% content_for(:title) { "#{t("administrate.actions.show")} #{page.page_title}" } %>
 
-<header class="header">
+<header class="header" role="banner">
   <h1 class="header__heading"><%= content_for(:title) %></h1>
   <div class="header__actions">
     <%= link_to(