From 2884955d679f4c4763290b78687b1f01f7a73dc5 Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Sat, 9 Feb 2019 18:06:58 -0800
Subject: [PATCH] fix: carousel should use ul/li for a11y (#956)

---
 .../_components/dialog/components/MediaDialog.html | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/routes/_components/dialog/components/MediaDialog.html b/src/routes/_components/dialog/components/MediaDialog.html
index d12abdd..dfa5743 100644
--- a/src/routes/_components/dialog/components/MediaDialog.html
+++ b/src/routes/_components/dialog/components/MediaDialog.html
@@ -6,9 +6,9 @@
   className="media-modal-dialog"
 >
   <div class="media-container">
-    <div class="media-scroll" ref:scroller>
+    <ul class="media-scroll" ref:scroller>
       {#each mediaItems as media}
-        <div class="media-scroll-item">
+        <li class="media-scroll-item">
           <div class="media-scroll-item-inner">
             <div class="media-scroll-item-inner-inner">
                 {#if canPinchZoom}
@@ -20,9 +20,9 @@
                 {/if}
             </div>
           </div>
-        </div>
+        </li>
       {/each}
-    </div>
+    </ul>
     <div class="media-controls-outside">
       {#if canPinchZoom}
         <IconButton
@@ -98,6 +98,12 @@
     -ms-overflow-style: none;
   }
 
+  ul.media-scroll {
+    padding: 0;
+    margin: 0;
+    list-style: none;
+  }
+
   .media-scroll::-webkit-scrollbar {
     display: none;
   }