From aa87242b9053008e681a08b29845962c9b0e50bd Mon Sep 17 00:00:00 2001
From: Richard Harris <richard.a.harris@gmail.com>
Date: Sun, 27 Oct 2019 18:26:06 -0400
Subject: [PATCH] make modal more accessible

---
 .../15-composition/04-modal/Modal.svelte      | 81 ++++++++++++-------
 1 file changed, 54 insertions(+), 27 deletions(-)

diff --git a/site/content/examples/15-composition/04-modal/Modal.svelte b/site/content/examples/15-composition/04-modal/Modal.svelte
index 44f79c902d..8d73045dd6 100644
--- a/site/content/examples/15-composition/04-modal/Modal.svelte
+++ b/site/content/examples/15-composition/04-modal/Modal.svelte
@@ -1,25 +1,62 @@
 <script>
-	import { createEventDispatcher } from 'svelte';
+	import { createEventDispatcher, onDestroy } from 'svelte';
 
 	const dispatch = createEventDispatcher();
-	
-	const handleClose = () => dispatch('close')
-
-	function handleKeydown(event) {
-		if (event.key == 'Escape') {
-			handleClose()
-		} else if (event.key == 'Tab') {
-			event.preventDefault()
+	const close = () => dispatch('close');
+
+	let modal;
+
+	const handle_keydown = e => {
+		if (e.key === 'Escape') {
+			close();
+			return;
 		}
-	}
 
-	let closeButton
-	onMount(() => {
-		closeButton.focus()
-	})
+		if (e.key === 'Tab') {
+			// trap focus
+			const nodes = modal.querySelectorAll('*');
+			const active = document.activeElement;
+
+			const tabbable = Array.from(nodes).filter(n => n.tabIndex >= 0);
+			const first = tabbable[0];
+			const last = tabbable[tabbable.length - 1];
 
+			let index = tabbable.indexOf(document.activeElement);
+			if (index === -1 && e.shiftKey) index = 0;
+
+			index += tabbable.length + (e.shiftKey ? -1 : 1);
+			index %= tabbable.length;
+
+			tabbable[index].focus();
+			e.preventDefault();
+		}
+	};
+
+	const previously_focused = typeof document !== 'undefined' && document.activeElement;
+
+	if (previously_focused) {
+		onDestroy(() => {
+			previously_focused.focus();
+		});
+	}
 </script>
 
+<svelte:window on:keydown={handle_keydown}/>
+
+<div class="modal-background" on:click={close}></div>
+
+<div class="modal" role="dialog" aria-modal="true" bind:this={modal}>
+	<slot name="header"></slot>
+	<hr>
+	<slot></slot>
+	<hr>
+
+	<!-- svelte-ignore a11y-autofocus -->
+	<button autofocus on:click={close}>close modal</button>
+
+	<a href="argh">argh</a>
+</div>
+
 <style>
 	.modal-background {
 		position: fixed;
@@ -32,7 +69,8 @@
 
 	.modal {
 		position: absolute;
-		left: 50%;	top: 50%;
+		left: 50%;
+		top: 50%;
 		width: calc(100vw - 4em);
 		max-width: 32em;
 		max-height: calc(100vh - 4em);
@@ -46,15 +84,4 @@
 	button {
 		display: block;
 	}
-</style>
-
-<div class='modal-background' on:click='{handleClose}'></div>
-
-<div class='modal'>
-	<slot name='header'></slot>
-	<hr>
-	<slot></slot>
-	<hr>
-
-	<button on:click='{() => dispatch("close")}' bind:this={closeButton}>close modal</button>
-</div>
+</style>
\ No newline at end of file