Skip to content

Commit 5ee4fae

Browse files
authored
Merge pull request #258 from rgieseke/ignore-brush-a11y-warning
Remove half-working ARIA code and A11Y warning
2 parents 5801e51 + dc93471 commit 5ee4fae

File tree

1 file changed

+2
-20
lines changed

1 file changed

+2
-20
lines changed

src/_components/Brush.html.svelte

Lines changed: 2 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -91,50 +91,32 @@
9191
$: right = 100 * (1 - max);
9292
</script>
9393

94+
<!-- TODO Add keyboard accessibility. See https://github.com/mhkeller/layercake/pull/258 -->
95+
<!-- svelte-ignore a11y_no_static_element_interactions -->
9496
<div
9597
bind:this={brush}
9698
class="brush-outer"
9799
on:mousedown|stopPropagation={reset}
98100
on:touchstart|stopPropagation={reset}
99-
role="slider"
100-
aria-valuemin={min}
101-
aria-valuemax={max}
102-
aria-valuetext="{min} to {max}"
103-
tabindex="0"
104101
>
105102
{#if min !== null}
106103
<div
107104
class="brush-inner"
108105
on:mousedown|stopPropagation={move}
109106
on:touchstart|stopPropagation={move}
110107
style="left: {left}%; right: {right}%"
111-
role="slider"
112-
aria-valuemin={min}
113-
aria-valuemax={max}
114-
aria-valuetext="{min} to {max}"
115-
tabindex="0"
116108
></div>
117109
<div
118110
class="brush-handle"
119111
on:mousedown|stopPropagation={adjust_min}
120112
on:touchstart|stopPropagation={adjust_min}
121113
style="left: {left}%"
122-
role="slider"
123-
aria-valuemin={min}
124-
aria-valuemax={max}
125-
aria-valuetext="{min} to {max}"
126-
tabindex="0"
127114
></div>
128115
<div
129116
class="brush-handle"
130117
on:mousedown|stopPropagation={adjust_max}
131118
on:touchstart|stopPropagation={adjust_max}
132119
style="right: {right}%"
133-
role="slider"
134-
aria-valuemin={min}
135-
aria-valuemax={max}
136-
aria-valuetext="{min} to {max}"
137-
tabindex="0"
138120
></div>
139121
{/if}
140122
</div>

0 commit comments

Comments
 (0)