From e266f20925e3fe1272ecf0a628530abf747136f2 Mon Sep 17 00:00:00 2001
From: Aniket Handa <anihan@microsoft.com>
Date: Wed, 8 May 2019 17:13:14 -0700
Subject: [PATCH 1/3] Remove aria-hidden

---
 .../src/components/FocusTrapZone/FocusTrapZone.tsx               | 1 -
 1 file changed, 1 deletion(-)

diff --git a/packages/office-ui-fabric-react/src/components/FocusTrapZone/FocusTrapZone.tsx b/packages/office-ui-fabric-react/src/components/FocusTrapZone/FocusTrapZone.tsx
index 5506d7b444ce9b..af8216b6312873 100644
--- a/packages/office-ui-fabric-react/src/components/FocusTrapZone/FocusTrapZone.tsx
+++ b/packages/office-ui-fabric-react/src/components/FocusTrapZone/FocusTrapZone.tsx
@@ -82,7 +82,6 @@ export class FocusTrapZone extends React.Component<IFocusTrapZoneProps, {}> impl
         position: 'fixed' // 'fixed' prevents browsers from scrolling to bumpers when viewport does not contain them
       },
       tabIndex: disabled ? -1 : 0, // make bumpers tabbable only when enabled
-      'aria-hidden': true,
       'data-is-visible': true
     } as React.HTMLAttributes<HTMLDivElement>;
 

From 74af9fabece1205b050f9280892fda24cc524c04 Mon Sep 17 00:00:00 2001
From: Aniket Handa <anihan@microsoft.com>
Date: Wed, 8 May 2019 17:29:05 -0700
Subject: [PATCH 2/3] Update snapshots

---
 .../HoverCard/__snapshots__/HoverCard.test.tsx.snap    |  4 ----
 .../components/Modal/__snapshots__/Modal.test.tsx.snap |  6 ------
 .../components/Panel/__snapshots__/Panel.test.tsx.snap |  2 --
 .../CommandBar.CommandBarButtonAs.Example.tsx.shot     |  2 --
 .../FocusTrapZone.Box.Click.Example.tsx.shot           |  2 --
 .../__snapshots__/FocusTrapZone.Box.Example.tsx.shot   |  2 --
 ...sTrapZone.Box.FocusOnCustomElement.Example.tsx.shot |  2 --
 .../FocusTrapZone.FocusZone.Example.tsx.shot           |  2 --
 .../FocusTrapZone.Nested.Example.tsx.shot              | 10 ----------
 .../Panel.HiddenOnDismiss.Example.tsx.shot             |  2 --
 10 files changed, 34 deletions(-)

diff --git a/packages/office-ui-fabric-react/src/components/HoverCard/__snapshots__/HoverCard.test.tsx.snap b/packages/office-ui-fabric-react/src/components/HoverCard/__snapshots__/HoverCard.test.tsx.snap
index 80d068b9800fef..799b23f87734ad 100644
--- a/packages/office-ui-fabric-react/src/components/HoverCard/__snapshots__/HoverCard.test.tsx.snap
+++ b/packages/office-ui-fabric-react/src/components/HoverCard/__snapshots__/HoverCard.test.tsx.snap
@@ -142,7 +142,6 @@ exports[`HoverCard renders ExpandingCard correctly 1`] = `
             onFocusCapture={[Function]}
           >
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -170,7 +169,6 @@ exports[`HoverCard renders ExpandingCard correctly 1`] = `
               </div>
             </div>
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -329,7 +327,6 @@ exports[`HoverCard renders PlainCard correctly 1`] = `
             onFocusCapture={[Function]}
           >
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -355,7 +352,6 @@ exports[`HoverCard renders PlainCard correctly 1`] = `
               </div>
             </div>
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
diff --git a/packages/office-ui-fabric-react/src/components/Modal/__snapshots__/Modal.test.tsx.snap b/packages/office-ui-fabric-react/src/components/Modal/__snapshots__/Modal.test.tsx.snap
index 7431ebbe3f1886..227f2331dd8fdf 100644
--- a/packages/office-ui-fabric-react/src/components/Modal/__snapshots__/Modal.test.tsx.snap
+++ b/packages/office-ui-fabric-react/src/components/Modal/__snapshots__/Modal.test.tsx.snap
@@ -126,7 +126,6 @@ exports[`Modal renders Draggable Modal correctly 1`] = `
           }
         >
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -153,7 +152,6 @@ exports[`Modal renders Draggable Modal correctly 1`] = `
             Test Content
           </div>
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -306,7 +304,6 @@ exports[`Modal renders Modal correctly 1`] = `
           onFocusCapture={[Function]}
         >
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -333,7 +330,6 @@ exports[`Modal renders Modal correctly 1`] = `
             Test Content
           </div>
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -465,7 +461,6 @@ exports[`Modal renders Modeless Modal correctly 1`] = `
           onFocusCapture={[Function]}
         >
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -492,7 +487,6 @@ exports[`Modal renders Modeless Modal correctly 1`] = `
             Test Content
           </div>
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
diff --git a/packages/office-ui-fabric-react/src/components/Panel/__snapshots__/Panel.test.tsx.snap b/packages/office-ui-fabric-react/src/components/Panel/__snapshots__/Panel.test.tsx.snap
index b4a965d3f2be8c..7ca8b56351e9eb 100644
--- a/packages/office-ui-fabric-react/src/components/Panel/__snapshots__/Panel.test.tsx.snap
+++ b/packages/office-ui-fabric-react/src/components/Panel/__snapshots__/Panel.test.tsx.snap
@@ -155,7 +155,6 @@ exports[`Panel renders Panel correctly 1`] = `
           style={Object {}}
         >
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
@@ -404,7 +403,6 @@ exports[`Panel renders Panel correctly 1`] = `
             </div>
           </div>
           <div
-            aria-hidden={true}
             data-is-visible={true}
             onFocus={[Function]}
             style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/CommandBar.CommandBarButtonAs.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/CommandBar.CommandBarButtonAs.Example.tsx.shot
index b070830f5c9e69..262ec473cfe499 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/CommandBar.CommandBarButtonAs.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/CommandBar.CommandBarButtonAs.Example.tsx.shot
@@ -2119,7 +2119,6 @@ exports[`Component Examples renders CommandBar.CommandBarButtonAs.Example.tsx co
                                 onFocusCapture={[Function]}
                               >
                                 <div
-                                  aria-hidden={true}
                                   data-is-visible={true}
                                   onFocus={[Function]}
                                   style={
@@ -2356,7 +2355,6 @@ exports[`Component Examples renders CommandBar.CommandBarButtonAs.Example.tsx co
                                   </div>
                                 </div>
                                 <div
-                                  aria-hidden={true}
                                   data-is-visible={true}
                                   onFocus={[Function]}
                                   style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Click.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Click.Example.tsx.shot
index e73090a94375ef..570f027ee3e22b 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Click.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Click.Example.tsx.shot
@@ -7,7 +7,6 @@ exports[`Component Examples renders FocusTrapZone.Box.Click.Example.tsx correctl
   onFocusCapture={[Function]}
 >
   <div
-    aria-hidden={true}
     data-is-visible={true}
     onFocus={[Function]}
     style={
@@ -401,7 +400,6 @@ exports[`Component Examples renders FocusTrapZone.Box.Click.Example.tsx correctl
     </a>
   </div>
   <div
-    aria-hidden={true}
     data-is-visible={true}
     onFocus={[Function]}
     style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Example.tsx.shot
index b31458f09c497b..56b17a9013f7dc 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.Example.tsx.shot
@@ -189,7 +189,6 @@ exports[`Component Examples renders FocusTrapZone.Box.Example.tsx correctly 1`]
     onFocusCapture={[Function]}
   >
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
@@ -583,7 +582,6 @@ exports[`Component Examples renders FocusTrapZone.Box.Example.tsx correctly 1`]
       </a>
     </div>
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.FocusOnCustomElement.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.FocusOnCustomElement.Example.tsx.shot
index dca8f46c5f8f05..c638980ae85ffc 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.FocusOnCustomElement.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Box.FocusOnCustomElement.Example.tsx.shot
@@ -189,7 +189,6 @@ exports[`Component Examples renders FocusTrapZone.Box.FocusOnCustomElement.Examp
     onFocusCapture={[Function]}
   >
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
@@ -584,7 +583,6 @@ exports[`Component Examples renders FocusTrapZone.Box.FocusOnCustomElement.Examp
       </a>
     </div>
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.FocusZone.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.FocusZone.Example.tsx.shot
index bb364370b429ca..353ece4f7872dc 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.FocusZone.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.FocusZone.Example.tsx.shot
@@ -7,7 +7,6 @@ exports[`Component Examples renders FocusTrapZone.FocusZone.Example.tsx correctl
   onFocusCapture={[Function]}
 >
   <div
-    aria-hidden={true}
     data-is-visible={true}
     onFocus={[Function]}
     style={
@@ -1094,7 +1093,6 @@ exports[`Component Examples renders FocusTrapZone.FocusZone.Example.tsx correctl
     </div>
   </div>
   <div
-    aria-hidden={true}
     data-is-visible={true}
     onFocus={[Function]}
     style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Nested.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Nested.Example.tsx.shot
index 1ac9d20b0f13de..701ba6e1bfac28 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Nested.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/FocusTrapZone.Nested.Example.tsx.shot
@@ -123,7 +123,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
     onFocusCapture={[Function]}
   >
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
@@ -448,7 +447,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
         onFocusCapture={[Function]}
       >
         <div
-          aria-hidden={true}
           data-is-visible={true}
           onFocus={[Function]}
           style={
@@ -774,7 +772,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
             onFocusCapture={[Function]}
           >
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -1096,7 +1093,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
               </button>
             </div>
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -1114,7 +1110,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
             onFocusCapture={[Function]}
           >
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -1436,7 +1431,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
               </button>
             </div>
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -1450,7 +1444,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
           </div>
         </div>
         <div
-          aria-hidden={true}
           data-is-visible={true}
           onFocus={[Function]}
           style={
@@ -1468,7 +1461,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
         onFocusCapture={[Function]}
       >
         <div
-          aria-hidden={true}
           data-is-visible={true}
           onFocus={[Function]}
           style={
@@ -1789,7 +1781,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
           </button>
         </div>
         <div
-          aria-hidden={true}
           data-is-visible={true}
           onFocus={[Function]}
           style={
@@ -1803,7 +1794,6 @@ exports[`Component Examples renders FocusTrapZone.Nested.Example.tsx correctly 1
       </div>
     </div>
     <div
-      aria-hidden={true}
       data-is-visible={true}
       onFocus={[Function]}
       style={
diff --git a/packages/office-ui-fabric-react/src/components/__snapshots__/Panel.HiddenOnDismiss.Example.tsx.shot b/packages/office-ui-fabric-react/src/components/__snapshots__/Panel.HiddenOnDismiss.Example.tsx.shot
index d12eff40b8ddec..4e85b16549026d 100644
--- a/packages/office-ui-fabric-react/src/components/__snapshots__/Panel.HiddenOnDismiss.Example.tsx.shot
+++ b/packages/office-ui-fabric-react/src/components/__snapshots__/Panel.HiddenOnDismiss.Example.tsx.shot
@@ -248,7 +248,6 @@ exports[`Component Examples renders Panel.HiddenOnDismiss.Example.tsx correctly
             style={Object {}}
           >
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={
@@ -497,7 +496,6 @@ exports[`Component Examples renders Panel.HiddenOnDismiss.Example.tsx correctly
               </div>
             </div>
             <div
-              aria-hidden={true}
               data-is-visible={true}
               onFocus={[Function]}
               style={

From 154d2a88e613ca67e7686a112f639aac19ae059c Mon Sep 17 00:00:00 2001
From: Aniket Handa <anihan@microsoft.com>
Date: Wed, 8 May 2019 17:32:32 -0700
Subject: [PATCH 3/3] Add rush change

---
 .../panel-focus_2019-05-09-00-32.json                 | 11 +++++++++++
 1 file changed, 11 insertions(+)
 create mode 100644 common/changes/office-ui-fabric-react/panel-focus_2019-05-09-00-32.json

diff --git a/common/changes/office-ui-fabric-react/panel-focus_2019-05-09-00-32.json b/common/changes/office-ui-fabric-react/panel-focus_2019-05-09-00-32.json
new file mode 100644
index 00000000000000..0b8fff499ec185
--- /dev/null
+++ b/common/changes/office-ui-fabric-react/panel-focus_2019-05-09-00-32.json
@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "Fix ARIA issue in Surfaces",
+      "type": "patch"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "anihan@microsoft.com"
+}
\ No newline at end of file