diff options
author | Frederic Hemberger <mail@frederic-hemberger.de> | 2011-09-21 14:48:38 +0200 |
---|---|---|
committer | Frederic Hemberger <mail@frederic-hemberger.de> | 2011-09-21 14:48:38 +0200 |
commit | a9a901c1362f6a4d6004f445629077e7633abb28 (patch) | |
tree | 62e6381618f59b82986eccf8ef59f590529e7844 /.themes | |
parent | 2a30d9a1eb5219e5335d1afdd214204d61ea6367 (diff) | |
download | my_new_personal_website-a9a901c1362f6a4d6004f445629077e7633abb28.tar.xz my_new_personal_website-a9a901c1362f6a4d6004f445629077e7633abb28.zip |
Fixed misuse of ARIA roles (see http://dev.opera.com/articles/view/introduction-to-wai-aria/)
Diffstat (limited to '')
-rw-r--r-- | .themes/classic/sass/base/_layout.scss | 12 | ||||
-rw-r--r-- | .themes/classic/sass/base/_theme.scss | 2 | ||||
-rw-r--r-- | .themes/classic/sass/partials/_blog.scss | 2 | ||||
-rw-r--r-- | .themes/classic/sass/partials/_navigation.scss | 8 | ||||
-rw-r--r-- | .themes/classic/sass/partials/sidebar/_base.scss | 2 | ||||
-rw-r--r-- | .themes/classic/source/_includes/custom/navigation.html | 2 | ||||
-rw-r--r-- | .themes/classic/source/_includes/navigation.html | 4 | ||||
-rw-r--r-- | .themes/classic/source/_layouts/default.html | 6 | ||||
-rw-r--r-- | .themes/classic/source/_layouts/page.html | 4 | ||||
-rw-r--r-- | .themes/classic/source/_layouts/post.html | 4 | ||||
-rw-r--r-- | .themes/classic/source/index.html | 4 | ||||
-rw-r--r-- | .themes/classic/source/javascripts/octopress.js | 8 |
12 files changed, 29 insertions, 29 deletions
diff --git a/.themes/classic/sass/base/_layout.scss b/.themes/classic/sass/base/_layout.scss index 4a3c780f..c347af16 100644 --- a/.themes/classic/sass/base/_layout.scss +++ b/.themes/classic/sass/base/_layout.scss @@ -41,7 +41,7 @@ body { max-width: $max-width; position: relative; margin: 0 auto; - > header, > nav, > footer, #content > article, #content > div > article, #content > div > section, nav[role=pagination] { + > header, > nav, > footer, #content > article, #content > div > article, #content > div > section, nav.pagination { @extend .group; padding-left: $pad-min; padding-right: $pad-min; @@ -67,7 +67,7 @@ body { #content { > div, > article { width: 100%; }} -aside[role=sidebar] { +aside.sidebar { float: none; padding: 0 $pad-min 1px; background-color: lighten($sidebar-bg, 2); @@ -88,7 +88,7 @@ aside[role=sidebar] { body.sidebar-footer { @media only screen and (min-width: 750px) { - aside[role=sidebar]{ @include collapse-sidebar; } + aside.sidebar{ @include collapse-sidebar; } } #content { margin-right: 0px; } .toggle-sidebar { display: none; } @@ -98,7 +98,7 @@ body.sidebar-footer { body > header { font-size: 1em; } } @media only screen and (min-width: 750px) { - aside[role=sidebar] { @include collapse-sidebar; } + aside.sidebar { @include collapse-sidebar; } } @media only screen and (min-width: 768px) { body { -webkit-text-size-adjust: auto; } @@ -120,7 +120,7 @@ body.sidebar-footer { float: left; } } - aside[role=sidebar] { + aside.sidebar { @extend .group; width: $sidebar-width-medium - $sidebar-pad-medium*2; padding: 0 $sidebar-pad-medium $sidebar-pad-medium; @@ -147,7 +147,7 @@ body.sidebar-footer { padding-bottom: $pad-wide/2; } } - aside[role=sidebar] { + aside.sidebar { width: $sidebar-width-wide - $sidebar-pad-wide*2; padding: 1.2em $sidebar-pad-wide $sidebar-pad-wide; .collapse-sidebar & { diff --git a/.themes/classic/sass/base/_theme.scss b/.themes/classic/sass/base/_theme.scss index fce47e53..e5539b1f 100644 --- a/.themes/classic/sass/base/_theme.scss +++ b/.themes/classic/sass/base/_theme.scss @@ -60,7 +60,7 @@ $page-border-bottom: darken($footer-bg, 5) !default; a { @include link-colors($link-color, $hover: $link-color-hover, $focus: $link-color-hover, $visited: $link-color-visited, $active: $link-color-active); } -aside[role=sidebar] a { +aside.sidebar a { @include link-colors($sidebar-link-color, $hover: $sidebar-link-color-hover, $focus: $sidebar-link-color-hover, $active: $sidebar-link-color-active); } a { diff --git a/.themes/classic/sass/partials/_blog.scss b/.themes/classic/sass/partials/_blog.scss index 558f0a22..88c71e9d 100644 --- a/.themes/classic/sass/partials/_blog.scss +++ b/.themes/classic/sass/partials/_blog.scss @@ -121,7 +121,7 @@ article + article { display: inline-block; } -#content nav[role=pagination] { +#content nav.pagination { text-align: center; font-size: .95em; div { diff --git a/.themes/classic/sass/partials/_navigation.scss b/.themes/classic/sass/partials/_navigation.scss index 7af9f9f7..43ae469e 100644 --- a/.themes/classic/sass/partials/_navigation.scss +++ b/.themes/classic/sass/partials/_navigation.scss @@ -31,8 +31,8 @@ body > nav { } } } - fieldset[role=site-search]{ float: right; width: 48%; } - fieldset[role=mobile-nav]{ float: left; width: 48%; + fieldset[role=search]{ float: right; width: 48%; } + fieldset.mobile-nav{ float: left; width: 48%; select{ width: 100%; font-size: .8em; border: 1px solid #888;} } ul { display: none; } @@ -85,8 +85,8 @@ body > nav { width: $sidebar-width-medium - $pad-medium*2 - $sidebar-pad-medium - 58px; .search { width: 91%; } } - fieldset[role=mobile-nav] { display: none; } - fieldset[role=site-search]{ width: 99%; } + fieldset.mobile-nav { display: none; } + fieldset[role=search]{ width: 99%; } } @media only screen and (min-width: 992px) { form { diff --git a/.themes/classic/sass/partials/sidebar/_base.scss b/.themes/classic/sass/partials/sidebar/_base.scss index a0d1c09e..afda01c7 100644 --- a/.themes/classic/sass/partials/sidebar/_base.scss +++ b/.themes/classic/sass/partials/sidebar/_base.scss @@ -1,7 +1,7 @@ .side-shadow-border { @include box-shadow(lighten($sidebar-bg, 5) 0 1px); } -aside[role=sidebar] { +aside.sidebar { color: $sidebar-color; text-shadow: lighten($sidebar-bg, 8) 0 1px; a { @extend .force-wrap; } diff --git a/.themes/classic/source/_includes/custom/navigation.html b/.themes/classic/source/_includes/custom/navigation.html index 8f091e16..d6bd4246 100644 --- a/.themes/classic/source/_includes/custom/navigation.html +++ b/.themes/classic/source/_includes/custom/navigation.html @@ -1,4 +1,4 @@ -<ul role=main-navigation> +<ul class="main-navigation"> <li><a href="{{ root_url }}/">Blog</a></li> <li><a href="{{ root_url }}/blog/archives">Archives</a></li> </ul> diff --git a/.themes/classic/source/_includes/navigation.html b/.themes/classic/source/_includes/navigation.html index c26908f8..9b87017a 100644 --- a/.themes/classic/source/_includes/navigation.html +++ b/.themes/classic/source/_includes/navigation.html @@ -1,11 +1,11 @@ -<ul role=subscription data-subscription="rss{% if site.subscribe_email %} email{% endif %}"> +<ul class="subscription" data-subscription="rss{% if site.subscribe_email %} email{% endif %}"> <li><a href="{{ site.subscribe_rss }}" rel="subscribe-rss" title="subscribe via RSS">RSS</a></li> {% if site.subscribe_email %} <li><a href="{{ site.subscribe_email }}" rel="subscribe-email" title="subscribe via email">Email</a></li> {% endif %} </ul> <form action="{{ site.simple_search }}" method="get"> - <fieldset role="site-search"> + <fieldset role="search"> <input type="hidden" name="q" value="site:{{ site.url | shorthand_url }}" /> <input class="search" type="text" name="q" results="0" placeholder="Search"/> </fieldset> diff --git a/.themes/classic/source/_layouts/default.html b/.themes/classic/source/_layouts/default.html index d84da44c..17c4e664 100644 --- a/.themes/classic/source/_layouts/default.html +++ b/.themes/classic/source/_layouts/default.html @@ -1,14 +1,14 @@ {% capture root_url %}{{ site.root | strip_slash }}{% endcapture %} {% include head.html %} <body {% if page.body_id %} id="{{ page.body_id }}" {% endif %} {% if page.sidebar == false %} class="no-sidebar" {% endif %}> - <header>{% include header.html %}</header> - <nav role=navigation>{% include navigation.html %}</nav> + <header role="banner">{% include header.html %}</header> + <nav role="navigation">{% include navigation.html %}</nav> <div id="main"> <div id="content"> {{ content | expand_urls: root_url }} </div> </div> - <footer>{% include footer.html %}</footer> + <footer role="contentinfo">{% include footer.html %}</footer> {% include disqus.html %} {% include google_analytics.html %} {% include google_plus_one.html %} diff --git a/.themes/classic/source/_layouts/page.html b/.themes/classic/source/_layouts/page.html index 20c17452..4edd3ed5 100644 --- a/.themes/classic/source/_layouts/page.html +++ b/.themes/classic/source/_layouts/page.html @@ -3,7 +3,7 @@ layout: default --- <div> -<article> +<article role="article"> {% if page.title %} <header> <h1 class="entry-title">{{ page.title | titlecase }}</h1> @@ -32,7 +32,7 @@ layout: default {% endif %} </div> {% unless page.sidebar == false %} -<aside role=sidebar> +<aside class="sidebar"> {% if site.page_asides.size %} {% include_array page_asides %} {% else %} diff --git a/.themes/classic/source/_layouts/post.html b/.themes/classic/source/_layouts/post.html index e81cc787..3b1208da 100644 --- a/.themes/classic/source/_layouts/post.html +++ b/.themes/classic/source/_layouts/post.html @@ -4,7 +4,7 @@ single: true --- <div> -<article class="hentry"> +<article class="hentry" role="article"> {% include article.html %} <footer> <p class="meta"> @@ -25,7 +25,7 @@ single: true {% endif %} </div> {% unless page.sidebar == false %} -<aside role=sidebar> +<aside class="sidebar"> {% if site.post_asides.size %} {% include_array post_asides %} {% else %} diff --git a/.themes/classic/source/index.html b/.themes/classic/source/index.html index 1e896b0d..e2eb6211 100644 --- a/.themes/classic/source/index.html +++ b/.themes/classic/source/index.html @@ -10,7 +10,7 @@ layout: default {% include article.html %} </article> {% endfor %} - <nav role="pagination"> + <nav class="pagination"> <div> {% if paginator.next_page %} <a class="prev" href="{{paginator.next_page}}">← Older</a> @@ -22,7 +22,7 @@ layout: default </div> </nav> </div> -<aside role=sidebar> +<aside class="sidebar"> {% if site.blog_index_asides.size %} {% include_array blog_index_asides %} {% else %} diff --git a/.themes/classic/source/javascripts/octopress.js b/.themes/classic/source/javascripts/octopress.js index d944e7eb..a23ab502 100644 --- a/.themes/classic/source/javascripts/octopress.js +++ b/.themes/classic/source/javascripts/octopress.js @@ -1,7 +1,7 @@ function getNav() { - var mobileNav = $('nav[role=navigation] fieldset[role=site-search]').after('<fieldset role="mobile-nav"></fieldset>').next().append('<select></select>'); + var mobileNav = $('nav[role=navigation] fieldset[role=search]').after('<fieldset class="mobile-nav"></fieldset>').next().append('<select></select>'); mobileNav.children('select').append('<option value="">Navigate…</option>'); - $('ul[role=main-navigation] a').each(function(link) { + $('ul.main-navigation a').each(function(link) { mobileNav.children('select').append('<option value="'+link.href+'">• '+link.text+'</option>'); }); mobileNav.children('select').bind('change', function(event) { @@ -19,7 +19,7 @@ function addSidebarToggler() { $('body').addClass('collapse-sidebar'); } }); - var sections = $('aside[role=sidebar] > section'); + var sections = $('aside.sidebar > section'); if (sections.length > 1) { sections.each(function(section, index){ if ((sections.length >= 3) && index % 3 === 0) { @@ -29,7 +29,7 @@ function addSidebarToggler() { $(section).addClass(count); }); } - if (sections.length >= 3){ $('aside[role=sidebar]').addClass('thirds'); } + if (sections.length >= 3){ $('aside.sidebar').addClass('thirds'); } } function testFeatures() { |