From 242f2d6c964111dc3d9a3dd958f80537a6223f13 Mon Sep 17 00:00:00 2001
From: marcysutton <holla@marcysutton.com>
Date: Fri, 5 Jun 2015 14:02:46 -0400
Subject: [PATCH] accessibility: controls as buttons, not divs

---
 css/reveal.css  | 25 ++++++++++++++-----------
 css/reveal.scss | 26 ++++++++++++++------------
 js/reveal.js    |  8 ++++----
 3 files changed, 32 insertions(+), 27 deletions(-)

diff --git a/css/reveal.css b/css/reveal.css
index 258e975..a83c155 100644
--- a/css/reveal.css
+++ b/css/reveal.css
@@ -156,58 +156,61 @@ body {
   bottom: 10px;
   -webkit-user-select: none; }
 
-.reveal .controls div {
+.reveal .controls button {
+  padding: 0;
   position: absolute;
   opacity: 0.05;
   width: 0;
   height: 0;
+  background-color: transparent;
   border: 12px solid transparent;
   -webkit-transform: scale(0.9999);
       -ms-transform: scale(0.9999);
           transform: scale(0.9999);
   -webkit-transition: all 0.2s ease;
           transition: all 0.2s ease;
+  -webkit-appearance: none;
   -webkit-tap-highlight-color: rgba(0, 0, 0, 0); }
 
-.reveal .controls div.enabled {
+.reveal .controls button.enabled {
   opacity: 0.7;
   cursor: pointer; }
 
-.reveal .controls div.enabled:active {
+.reveal .controls button.enabled:active {
   margin-top: 1px; }
 
-.reveal .controls div.navigate-left {
+.reveal .controls button.navigate-left {
   top: 42px;
   border-right-width: 22px;
   border-right-color: #000; }
 
-.reveal .controls div.navigate-left.fragmented {
+.reveal .controls button.navigate-left.fragmented {
   opacity: 0.3; }
 
-.reveal .controls div.navigate-right {
+.reveal .controls button.navigate-right {
   left: 74px;
   top: 42px;
   border-left-width: 22px;
   border-left-color: #000; }
 
-.reveal .controls div.navigate-right.fragmented {
+.reveal .controls button.navigate-right.fragmented {
   opacity: 0.3; }
 
-.reveal .controls div.navigate-up {
+.reveal .controls button.navigate-up {
   left: 42px;
   border-bottom-width: 22px;
   border-bottom-color: #000; }
 
-.reveal .controls div.navigate-up.fragmented {
+.reveal .controls button.navigate-up.fragmented {
   opacity: 0.3; }
 
-.reveal .controls div.navigate-down {
+.reveal .controls button.navigate-down {
   left: 42px;
   top: 74px;
   border-top-width: 22px;
   border-top-color: #000; }
 
-.reveal .controls div.navigate-down.fragmented {
+.reveal .controls button.navigate-down.fragmented {
   opacity: 0.3; }
 
 /*********************************************
diff --git a/css/reveal.scss b/css/reveal.scss
index 3321c98..a8558a9 100644
--- a/css/reveal.scss
+++ b/css/reveal.scss
@@ -214,66 +214,68 @@ body {
 	-webkit-user-select: none;
 }
 
-.reveal .controls div {
+.reveal .controls button {
+	padding: 0;
 	position: absolute;
 	opacity: 0.05;
 	width: 0;
 	height: 0;
+	background-color: transparent;
 	border: 12px solid transparent;
 	transform: scale(.9999);
 	transition: all 0.2s ease;
-
+	-webkit-appearance: none;
 	-webkit-tap-highlight-color: rgba( 0, 0, 0, 0 );
 }
 
-.reveal .controls div.enabled {
+.reveal .controls button.enabled {
 	opacity: 0.7;
 	cursor: pointer;
 }
 
-.reveal .controls div.enabled:active {
+.reveal .controls button.enabled:active {
 	margin-top: 1px;
 }
 
-	.reveal .controls div.navigate-left {
+	.reveal .controls button.navigate-left {
 		top: 42px;
 
 		border-right-width: 22px;
 		border-right-color: #000;
 	}
-		.reveal .controls div.navigate-left.fragmented {
+		.reveal .controls button.navigate-left.fragmented {
 			opacity: 0.3;
 		}
 
-	.reveal .controls div.navigate-right {
+	.reveal .controls button.navigate-right {
 		left: 74px;
 		top: 42px;
 
 		border-left-width: 22px;
 		border-left-color: #000;
 	}
-		.reveal .controls div.navigate-right.fragmented {
+		.reveal .controls button.navigate-right.fragmented {
 			opacity: 0.3;
 		}
 
-	.reveal .controls div.navigate-up {
+	.reveal .controls button.navigate-up {
 		left: 42px;
 
 		border-bottom-width: 22px;
 		border-bottom-color: #000;
 	}
-		.reveal .controls div.navigate-up.fragmented {
+		.reveal .controls button.navigate-up.fragmented {
 			opacity: 0.3;
 		}
 
-	.reveal .controls div.navigate-down {
+	.reveal .controls button.navigate-down {
 		left: 42px;
 		top: 74px;
 
 		border-top-width: 22px;
 		border-top-color: #000;
 	}
-		.reveal .controls div.navigate-down.fragmented {
+		.reveal .controls button.navigate-down.fragmented {
 			opacity: 0.3;
 		}
 
diff --git a/js/reveal.js b/js/reveal.js
index ff5ea53..8964a0c 100644
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -457,10 +457,10 @@
 
 		// Arrow controls
 		createSingletonNode( dom.wrapper, 'aside', 'controls',
-			'<div class="navigate-left"></div>' +
-			'<div class="navigate-right"></div>' +
-			'<div class="navigate-up"></div>' +
-			'<div class="navigate-down"></div>' );
+			'<button class="navigate-left" aria-label="previous slide"></button>' +
+			'<button class="navigate-right" aria-label="next slide"></button>' +
+			'<button class="navigate-up" aria-label="above slide"></button>' +
+			'<button class="navigate-down" aria-label="below slide"></button>' );
 
 		// Slide number
 		dom.slideNumber = createSingletonNode( dom.wrapper, 'div', 'slide-number', '' );