From d3869ff4c9bf90f4d68348ecae022b476acbd8b1 Mon Sep 17 00:00:00 2001
From: Marcy Sutton <marcy.sutton@deque.com>
Date: Tue, 30 Jul 2019 16:47:26 -0500
Subject: [PATCH] updates/fixes from the day

---
 examples/unit-testing/dropdown.test.js | 24 +++++++++++++++++++++---
 src/components/site-chrome/layout.scss |  2 +-
 src/slides/index.mdx                   |  1 +
 3 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/examples/unit-testing/dropdown.test.js b/examples/unit-testing/dropdown.test.js
index b0d2b07..412bb53 100644
--- a/examples/unit-testing/dropdown.test.js
+++ b/examples/unit-testing/dropdown.test.js
@@ -12,10 +12,28 @@ describe(`Dropdown`, () => {
 
     expect(text).toBeInTheDocument()
   })
-  it(`renders a focusable activator element`, () => {
+  it(`renders a focusable button that activates the dropdown`, () => {
+    const activatorText = `Dogs`
+    const items = [{
+      text: 'item 1',
+      url: '#'
+    }, {
+      text: 'item 2',
+      url: '#'
+    }]
+    const dropdown = render(<Dropdown activatorText={activatorText} items={items} />)
 
-  })
-  it(`sends focus to dropdown content when active`, () => {
+    const activator = dropdown.getByTestId('dropdown-activator')
+    activator.focus()
+
+    expect(activator).toHaveFocus()
+    
+    fireEvent.click(activator)
+
+    const dropdownList = dropdown.getByTestId('dropdown-itemList')
 
+    const firstAnchor = dropdownList.querySelector('a')
+    expect(firstAnchor).toHaveFocus()
+    
   })
 })
diff --git a/src/components/site-chrome/layout.scss b/src/components/site-chrome/layout.scss
index 7d97243..c8203bf 100644
--- a/src/components/site-chrome/layout.scss
+++ b/src/components/site-chrome/layout.scss
@@ -21,7 +21,7 @@ body {
 a, a:link, a:visited {
   color: #333;
 }
-.js-focus-visible :focus {
+.js-focus-visible :focus:not(.focus-visible) {
   outline: none;
 }
 .js-focus-visible :focus-visible {
diff --git a/src/slides/index.mdx b/src/slides/index.mdx
index f90cc34..223990e 100644
--- a/src/slides/index.mdx
+++ b/src/slides/index.mdx
@@ -79,6 +79,7 @@ http://bit.ly/microsoft-inclusive-toolkit
 - Accessibility debugging
 - Accessibility in JavaScript apps
     - Focus management
+    - Client-side routing
     - Announcements
     - Semantic HTML
     - Unobtrusive motion