From 98bf58369388622646c95e34802c56bd124fd84f Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 11:18:17 +0200
Subject: [PATCH 1/9] fix: ESC should close opened Popup/Dialog if focus is
 lost and moved to body

---
 .../react/src/components/Dialog/Dialog.tsx     | 18 ++++++++++++++++++
 packages/react/src/components/Popup/Popup.tsx  |  9 +++++++++
 2 files changed, 27 insertions(+)

diff --git a/packages/react/src/components/Dialog/Dialog.tsx b/packages/react/src/components/Dialog/Dialog.tsx
index 3f95815b6..129d05d53 100644
--- a/packages/react/src/components/Dialog/Dialog.tsx
+++ b/packages/react/src/components/Dialog/Dialog.tsx
@@ -5,6 +5,7 @@ import * as customPropTypes from '@stardust-ui/react-proptypes'
 import * as _ from 'lodash'
 import * as PropTypes from 'prop-types'
 import * as React from 'react'
+import * as keyboardKey from 'keyboard-key'
 
 import {
   UIComponentProps,
@@ -203,6 +204,17 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
     }
   }
 
+  handleDocumentKeydown = e => {
+    // if focus was lost from Dialog and moved to body, for e.g. when click on Dialog's content
+    // and ESC is pressed, the opened Dialog should get closed and the trigger should get focus
+    const bodyHasFocus: boolean = document.activeElement === document.body
+    const keyCode = keyboardKey.getCode(e)
+    if (keyCode === keyboardKey.Escape && bodyHasFocus && this.state.open) {
+      this.handleDialogCancel(e)
+      _.invoke(this.triggerRef, 'current.focus')
+    }
+  }
+
   renderComponent({ accessibility, classes, ElementType, styles, unhandledProps }) {
     const {
       actions,
@@ -310,6 +322,12 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
                 type="click"
                 capture
               />
+              <EventListener
+                listener={this.handleDocumentKeydown}
+                targetRef={documentRef}
+                type="keydown"
+                capture
+              />
             </>
           )}
         </Unstable_NestingAuto>
diff --git a/packages/react/src/components/Popup/Popup.tsx b/packages/react/src/components/Popup/Popup.tsx
index ad8320f37..a81565fee 100644
--- a/packages/react/src/components/Popup/Popup.tsx
+++ b/packages/react/src/components/Popup/Popup.tsx
@@ -270,6 +270,15 @@ export default class Popup extends AutoControlledComponent<PopupProps, PopupStat
     if (isMatchingKey && this.isOutsidePopupElementAndOutsideTriggerElement(getRefs(), e)) {
       this.trySetOpen(false, e)
     }
+
+    // if focus was lost from Popup and moved to body, for e.g. when click on popup content
+    // and ESC is pressed, the last opened Popup should get closed and the trigger should get focus
+    const isLastOpenedPopup: boolean = getRefs().length === 1
+    const bodyHasFocus: boolean = document.activeElement === document.body
+
+    if (keyCode === keyboardKey.Escape && bodyHasFocus && isLastOpenedPopup) {
+      this.close(e, () => _.invoke(this.triggerFocusableDomElement, 'focus'))
+    }
   }
 
   isOutsidePopupElementAndOutsideTriggerElement(refs: NodeRef[], e) {

From 3cdeb8ed9cd3d771a19114c1990db12be2644819 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 13:13:43 +0200
Subject: [PATCH 2/9] Add e2e test for popup

---
 e2e/tests/popupInPopup-example.tsx            | 26 +++++++++++++
 e2e/tests/popupInPopup-test.ts                | 39 +++++++++++++++++++
 .../react/src/components/Dialog/Dialog.tsx    |  5 +--
 packages/react/src/components/Popup/Popup.tsx |  4 +-
 4 files changed, 70 insertions(+), 4 deletions(-)
 create mode 100644 e2e/tests/popupInPopup-example.tsx
 create mode 100644 e2e/tests/popupInPopup-test.ts

diff --git a/e2e/tests/popupInPopup-example.tsx b/e2e/tests/popupInPopup-example.tsx
new file mode 100644
index 000000000..9c22e5cff
--- /dev/null
+++ b/e2e/tests/popupInPopup-example.tsx
@@ -0,0 +1,26 @@
+import * as React from 'react'
+import { Button, Popup } from '@stardust-ui/react'
+
+export const selectors = {
+  popupContentId: 'popup-content-id',
+  popupTriggerId: 'popup-trigger-id',
+  popupContentNestedId: 'popup-content-nested-id',
+  popupTriggerNestedId: 'popup-trigger-nested-id',
+}
+
+const PopupInPopupExample = () => (
+  <Popup
+    content={
+      <>
+        <div id={selectors.popupContentId}>content</div>
+        <Popup
+          content={<div id={selectors.popupContentNestedId}>nested content</div>}
+          trigger={<Button id={selectors.popupTriggerNestedId} content="Open a nested popup" />}
+        />
+      </>
+    }
+    trigger={<Button id={selectors.popupTriggerId} content="Open a popup" />}
+  />
+)
+
+export default PopupInPopupExample
diff --git a/e2e/tests/popupInPopup-test.ts b/e2e/tests/popupInPopup-test.ts
new file mode 100644
index 000000000..26654a67a
--- /dev/null
+++ b/e2e/tests/popupInPopup-test.ts
@@ -0,0 +1,39 @@
+import { selectors } from './popupInPopup-example'
+
+const popupTrigger = `#${selectors.popupTriggerId}`
+const popupContent = `#${selectors.popupContentId}`
+const popupTriggerNested = `#${selectors.popupTriggerNestedId}`
+const popupContentNested = `#${selectors.popupContentNestedId}`
+
+describe('Nested Popups', () => {
+  beforeEach(async () => {
+    await e2e.gotoTestCase(__filename, popupTrigger)
+  })
+
+  it('the last opened Popup is closed when press ESC and focus moved to body', async () => {
+    await e2e.clickOn(popupTrigger) // opens popup
+    expect(await e2e.exists(popupContent)).toBe(true)
+
+    await e2e.clickOn(popupTriggerNested) // opens nested popup
+    expect(await e2e.exists(popupContentNested)).toBe(true)
+
+    await e2e.clickOn(popupContentNested)
+
+    // check that focus moved to body after clicking on Popup content
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC and check if nested popup is closed and focus is on nested trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(popupContentNested)).toBe(false)
+    expect(await e2e.isFocused(popupTriggerNested)).toBe(true)
+
+    // click on popup content to move focus to body
+    await e2e.clickOn(popupContent)
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC again and check if the last popup is closed and focus is on trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(popupContent)).toBe(false)
+    expect(await e2e.isFocused(popupTrigger)).toBe(true)
+  })
+})
diff --git a/packages/react/src/components/Dialog/Dialog.tsx b/packages/react/src/components/Dialog/Dialog.tsx
index 129d05d53..31e325ec1 100644
--- a/packages/react/src/components/Dialog/Dialog.tsx
+++ b/packages/react/src/components/Dialog/Dialog.tsx
@@ -205,11 +205,10 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
   }
 
   handleDocumentKeydown = e => {
-    // if focus was lost from Dialog and moved to body, for e.g. when click on Dialog's content
+    // if focus was lost from Dialog, for e.g. when click on Dialog's content
     // and ESC is pressed, the opened Dialog should get closed and the trigger should get focus
-    const bodyHasFocus: boolean = document.activeElement === document.body
     const keyCode = keyboardKey.getCode(e)
-    if (keyCode === keyboardKey.Escape && bodyHasFocus && this.state.open) {
+    if (keyCode === keyboardKey.Escape && this.state.open) {
       this.handleDialogCancel(e)
       _.invoke(this.triggerRef, 'current.focus')
     }
diff --git a/packages/react/src/components/Popup/Popup.tsx b/packages/react/src/components/Popup/Popup.tsx
index a81565fee..2285f8d7a 100644
--- a/packages/react/src/components/Popup/Popup.tsx
+++ b/packages/react/src/components/Popup/Popup.tsx
@@ -273,7 +273,9 @@ export default class Popup extends AutoControlledComponent<PopupProps, PopupStat
 
     // if focus was lost from Popup and moved to body, for e.g. when click on popup content
     // and ESC is pressed, the last opened Popup should get closed and the trigger should get focus
-    const isLastOpenedPopup: boolean = getRefs().length === 1
+    const lastContentRef = getRefs().pop()
+    const isLastOpenedPopup: boolean =
+      lastContentRef && lastContentRef.current === this.popupDomElement
     const bodyHasFocus: boolean = document.activeElement === document.body
 
     if (keyCode === keyboardKey.Escape && bodyHasFocus && isLastOpenedPopup) {

From fadb20169b74b7b801d2d6004dd3f6dbfbfed518 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 13:20:51 +0200
Subject: [PATCH 3/9] fix for nested Dialogs

---
 packages/react/src/components/Dialog/Dialog.tsx | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/packages/react/src/components/Dialog/Dialog.tsx b/packages/react/src/components/Dialog/Dialog.tsx
index 31e325ec1..d6d619916 100644
--- a/packages/react/src/components/Dialog/Dialog.tsx
+++ b/packages/react/src/components/Dialog/Dialog.tsx
@@ -204,11 +204,15 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
     }
   }
 
-  handleDocumentKeydown = e => {
+  handleDocumentKeydown = (getRefs: Function) => e => {
     // if focus was lost from Dialog, for e.g. when click on Dialog's content
     // and ESC is pressed, the opened Dialog should get closed and the trigger should get focus
+    const lastOverlayRef = getRefs().pop()
+    const isLastOpenedDialog: boolean =
+      lastOverlayRef && lastOverlayRef.current === this.overlayRef.current
+
     const keyCode = keyboardKey.getCode(e)
-    if (keyCode === keyboardKey.Escape && this.state.open) {
+    if (keyCode === keyboardKey.Escape && isLastOpenedDialog && this.state.open) {
       this.handleDialogCancel(e)
       _.invoke(this.triggerRef, 'current.focus')
     }
@@ -322,7 +326,7 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
                 capture
               />
               <EventListener
-                listener={this.handleDocumentKeydown}
+                listener={this.handleDocumentKeydown(getRefs)}
                 targetRef={documentRef}
                 type="keydown"
                 capture

From 79ec1be6dfb95b025dd4db1a75ba6bb3ad03aad2 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 13:32:26 +0200
Subject: [PATCH 4/9] add test for nested dialogs

---
 e2e/tests/dialogInDialog-test.ts | 27 +++++++++++++++++++++++++++
 e2e/tests/popupInPopup-test.ts   |  4 ++--
 2 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/e2e/tests/dialogInDialog-test.ts b/e2e/tests/dialogInDialog-test.ts
index 6f9ae05b2..0bfc19ced 100644
--- a/e2e/tests/dialogInDialog-test.ts
+++ b/e2e/tests/dialogInDialog-test.ts
@@ -59,4 +59,31 @@ describe('Dialog in Dialog', () => {
     expect(await e2e.exists(outerHeader)).toBe(false)
     expect(await e2e.exists(innerHeader)).toBe(false)
   })
+
+  it('A click on content and pressing ESC button should close the last opened dialog', async () => {
+    await e2e.clickOn(outerTrigger) // opens popup
+    expect(await e2e.exists(outerHeader)).toBe(true)
+
+    await e2e.clickOn(innerTrigger) // opens nested popup
+    expect(await e2e.exists(innerHeader)).toBe(true)
+
+    await e2e.clickOn(innerHeader)
+
+    // check that focus moved to body after clicking on Popup content
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC and check if nested popup is closed and focus is on nested trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(innerHeader)).toBe(false)
+    expect(await e2e.isFocused(innerTrigger)).toBe(true)
+
+    // click on popup content to move focus to body
+    await e2e.clickOn(outerHeader)
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC again and check if the last popup is closed and focus is on trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(outerHeader)).toBe(false)
+    expect(await e2e.isFocused(outerTrigger)).toBe(true)
+  })
 })
diff --git a/e2e/tests/popupInPopup-test.ts b/e2e/tests/popupInPopup-test.ts
index 26654a67a..46fe3ec81 100644
--- a/e2e/tests/popupInPopup-test.ts
+++ b/e2e/tests/popupInPopup-test.ts
@@ -5,12 +5,12 @@ const popupContent = `#${selectors.popupContentId}`
 const popupTriggerNested = `#${selectors.popupTriggerNestedId}`
 const popupContentNested = `#${selectors.popupContentNestedId}`
 
-describe('Nested Popups', () => {
+describe('Popup in Popup', () => {
   beforeEach(async () => {
     await e2e.gotoTestCase(__filename, popupTrigger)
   })
 
-  it('the last opened Popup is closed when press ESC and focus moved to body', async () => {
+  it('A click on content and pressing ESC button should close the last opened popup', async () => {
     await e2e.clickOn(popupTrigger) // opens popup
     expect(await e2e.exists(popupContent)).toBe(true)
 

From 2587f6a852cd28976ac0505dca3c138a50678cf1 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 13:42:36 +0200
Subject: [PATCH 5/9] add test case fo dialog in popup

---
 e2e/tests/dialogInDialog-test.ts    | 12 ++++++------
 e2e/tests/dialogInPopup-example.tsx | 15 +++++++++------
 e2e/tests/dialogInPopup-test.ts     | 29 ++++++++++++++++++++++++++++-
 3 files changed, 43 insertions(+), 13 deletions(-)

diff --git a/e2e/tests/dialogInDialog-test.ts b/e2e/tests/dialogInDialog-test.ts
index 0bfc19ced..a47a55bd4 100644
--- a/e2e/tests/dialogInDialog-test.ts
+++ b/e2e/tests/dialogInDialog-test.ts
@@ -61,27 +61,27 @@ describe('Dialog in Dialog', () => {
   })
 
   it('A click on content and pressing ESC button should close the last opened dialog', async () => {
-    await e2e.clickOn(outerTrigger) // opens popup
+    await e2e.clickOn(outerTrigger) // opens dialog
     expect(await e2e.exists(outerHeader)).toBe(true)
 
-    await e2e.clickOn(innerTrigger) // opens nested popup
+    await e2e.clickOn(innerTrigger) // opens nested dialog
     expect(await e2e.exists(innerHeader)).toBe(true)
 
     await e2e.clickOn(innerHeader)
 
-    // check that focus moved to body after clicking on Popup content
+    // check that focus moved to body after clicking on Dialog content
     expect(await e2e.isFocused('body')).toBe(true)
 
-    // press ESC and check if nested popup is closed and focus is on nested trigger
+    // press ESC and check if nested dialog is closed and focus is on nested trigger
     await e2e.pressKey('Escape')
     expect(await e2e.exists(innerHeader)).toBe(false)
     expect(await e2e.isFocused(innerTrigger)).toBe(true)
 
-    // click on popup content to move focus to body
+    // click on dialog content to move focus to body
     await e2e.clickOn(outerHeader)
     expect(await e2e.isFocused('body')).toBe(true)
 
-    // press ESC again and check if the last popup is closed and focus is on trigger
+    // press ESC again and check if the last dialog is closed and focus is on trigger
     await e2e.pressKey('Escape')
     expect(await e2e.exists(outerHeader)).toBe(false)
     expect(await e2e.isFocused(outerTrigger)).toBe(true)
diff --git a/e2e/tests/dialogInPopup-example.tsx b/e2e/tests/dialogInPopup-example.tsx
index 9426385c9..61b79348c 100644
--- a/e2e/tests/dialogInPopup-example.tsx
+++ b/e2e/tests/dialogInPopup-example.tsx
@@ -6,18 +6,21 @@ export const selectors = {
   dialogHeader: Dialog.slotClassNames.header,
   dialogOverlay: Dialog.slotClassNames.overlay,
   dialogTrigger: 'dialog-trigger',
-  popupContent: Popup.Content.className,
+  popupContent: 'popup-content',
   popupTrigger: 'popup-trigger',
 }
 
 const DialogInPopupExample = () => (
   <Popup
     content={
-      <Dialog
-        cancelButton={{ content: 'Close', id: selectors.dialogCancel }}
-        header="A dialog"
-        trigger={<Button id={selectors.dialogTrigger} content="Open a dialog" />}
-      />
+      <>
+        <div id={selectors.popupContent}>Popup content</div>
+        <Dialog
+          cancelButton={{ content: 'Close', id: selectors.dialogCancel }}
+          header="A dialog"
+          trigger={<Button id={selectors.dialogTrigger} content="Open a dialog" />}
+        />
+      </>
     }
     trigger={<Button id={selectors.popupTrigger} content="Open a popup" />}
   />
diff --git a/e2e/tests/dialogInPopup-test.ts b/e2e/tests/dialogInPopup-test.ts
index ba87eaa0a..4d91f8f4a 100644
--- a/e2e/tests/dialogInPopup-test.ts
+++ b/e2e/tests/dialogInPopup-test.ts
@@ -4,7 +4,7 @@ const dialogCancel = `#${selectors.dialogCancel}`
 const dialogHeader = `.${selectors.dialogHeader}`
 const dialogOverlay = `.${selectors.dialogOverlay}`
 const dialogTrigger = `#${selectors.dialogTrigger}`
-const popupContent = `.${selectors.popupContent}`
+const popupContent = `#${selectors.popupContent}`
 const popupTrigger = `#${selectors.popupTrigger}`
 
 // https://github.com/stardust-ui/react/issues/1674
@@ -48,4 +48,31 @@ describe('Dialog in Popup', () => {
     expect(await e2e.exists(popupContent)).toBe(true)
     expect(await e2e.exists(dialogHeader)).toBe(false)
   })
+
+  it('A click on content and pressing ESC button should close the first opened dialog', async () => {
+    await e2e.clickOn(popupTrigger) // opens popup
+    await e2e.clickOn(dialogTrigger) // opens dialog
+
+    expect(await e2e.exists(popupContent)).toBe(true)
+    expect(await e2e.exists(dialogHeader)).toBe(true)
+
+    await e2e.clickOn(dialogHeader)
+
+    // check that focus moved to body after clicking on Dialog content
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC and check if nested popup is closed and focus is on nested trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(dialogHeader)).toBe(false)
+    expect(await e2e.isFocused(dialogTrigger)).toBe(true)
+
+    // click on popup content to move focus to body
+    await e2e.clickOn(popupContent)
+    expect(await e2e.isFocused('body')).toBe(true)
+
+    // press ESC again and check if the last popup is closed and focus is on trigger
+    await e2e.pressKey('Escape')
+    expect(await e2e.exists(popupContent)).toBe(false)
+    expect(await e2e.isFocused(popupTrigger)).toBe(true)
+  })
 })

From 2ecf95ff99851c40d7242bcb4e69764586638918 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Fri, 16 Aug 2019 13:45:28 +0200
Subject: [PATCH 6/9] update changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 7fbd32f42..18b3f443c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -24,6 +24,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 - Make `Embed` focusable via keyboard @lucivpav ([#1758](https://github.com/stardust-ui/react/pull/1758))
 - Update screener tests for `Input` to only capture Teams themes @codepretty ([#1801](https://github.com/stardust-ui/react/pull/1801))
 - Fix `Checkbox` style bug in background color [redlines] @bcalvery ([#1796](https://github.com/stardust-ui/react/pull/1796))
+- ESC key should close the last opened `Popup` or `Dialog` if body has focus @sophieH29 ([#1807](https://github.com/stardust-ui/react/pull/1807))
 
 ### Features
 - Add `overwrite` prop to `Provider` @layershifter ([#1780](https://github.com/stardust-ui/react/pull/1780))

From b1fc629c872c5767792e293ae7bf5e6e0bf8a892 Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Mon, 19 Aug 2019 11:12:04 +0200
Subject: [PATCH 7/9] improvements after CR comments

---
 packages/react/src/components/Dialog/Dialog.tsx | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/packages/react/src/components/Dialog/Dialog.tsx b/packages/react/src/components/Dialog/Dialog.tsx
index 57ee7cdc5..bdcd8dc5b 100644
--- a/packages/react/src/components/Dialog/Dialog.tsx
+++ b/packages/react/src/components/Dialog/Dialog.tsx
@@ -204,15 +204,14 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
     }
   }
 
-  handleDocumentKeydown = (getRefs: Function) => e => {
+  handleDocumentKeydown = (getRefs: Function) => (e: KeyboardEvent) => {
     // if focus was lost from Dialog, for e.g. when click on Dialog's content
     // and ESC is pressed, the opened Dialog should get closed and the trigger should get focus
     const lastOverlayRef = getRefs().pop()
     const isLastOpenedDialog: boolean =
       lastOverlayRef && lastOverlayRef.current === this.overlayRef.current
 
-    const keyCode = keyboardKey.getCode(e)
-    if (keyCode === keyboardKey.Escape && isLastOpenedDialog && this.state.open) {
+    if (keyboardKey.getCode(e) === keyboardKey.Escape && isLastOpenedDialog) {
       this.handleDialogCancel(e)
       _.invoke(this.triggerRef, 'current.focus')
     }

From 16525be8758b38d6a1fe1a7cdbea7a6c17846433 Mon Sep 17 00:00:00 2001
From: Oleksandr Fediashov <olfedias@microsoft.com>
Date: Mon, 19 Aug 2019 12:06:51 +0200
Subject: [PATCH 8/9] Update packages/react/src/components/Dialog/Dialog.tsx

---
 packages/react/src/components/Dialog/Dialog.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/react/src/components/Dialog/Dialog.tsx b/packages/react/src/components/Dialog/Dialog.tsx
index fedd04559..ab4656d3d 100644
--- a/packages/react/src/components/Dialog/Dialog.tsx
+++ b/packages/react/src/components/Dialog/Dialog.tsx
@@ -328,7 +328,7 @@ class Dialog extends AutoControlledComponent<WithAsProp<DialogProps>, DialogStat
               />
               <EventListener
                 listener={this.handleDocumentKeydown(getRefs)}
-                targetRef={documentRef}
+                targetRef={targetRef}
                 type="keydown"
                 capture
               />

From 0d4229106f0fd2269a8f99ac47c5f7e13cae8e1f Mon Sep 17 00:00:00 2001
From: Sofiya Huts <sohuts@microsoft.com>
Date: Mon, 19 Aug 2019 12:15:38 +0200
Subject: [PATCH 9/9] small improvement to trigger build