cybrespace
/
pinafore
1
0
Fork 1
Code Issues Pull Requests Releases Wiki Activity

fix: carousel should use ul/li for a11y (#956)

This commit is contained in:
Nolan Lawson 2019-02-09 18:06:58 -08:00 committed by GitHub
parent 9cb15a3396
commit 2884955d67
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/routes/_components/dialog/components/MediaDialog.html
View File

@ -6,9 +6,9 @@
className="media-modal-dialog" className="media-modal-dialog"
> >
<div class="media-container"> <div class="media-container">
<div class="media-scroll" ref:scroller> <ul class="media-scroll" ref:scroller>
{#each mediaItems as media} {#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">
<div class="media-scroll-item-inner-inner"> <div class="media-scroll-item-inner-inner">
{#if canPinchZoom} {#if canPinchZoom}
@ -20,9 +20,9 @@
{/if} {/if}
</div> </div>
</div> </div>
</div> </li>
{/each} {/each}
</div> </ul>
<div class="media-controls-outside"> <div class="media-controls-outside">
{#if canPinchZoom} {#if canPinchZoom}
<IconButton <IconButton
@ -98,6 +98,12 @@
-ms-overflow-style: none; -ms-overflow-style: none;
} }
ul.media-scroll {
padding: 0;
margin: 0;
list-style: none;
}
.media-scroll::-webkit-scrollbar { .media-scroll::-webkit-scrollbar {
display: none; display: none;
} }