From b1b483472f860a98169e8a8f5735efd9b06856b5 Mon Sep 17 00:00:00 2001
From: Eric Liu <ericyl.us@gmail.com>
Date: Thu, 21 Jan 2021 14:34:11 -0800
Subject: [PATCH] fix(ui-shell): deprecate ariaLabel prop in HeaderNav

- remove menubar role from HeaderNav
- obtain "aria-label" and "aria-labelledby" from $$props
---
 src/UIShell/GlobalHeader/HeaderNav.svelte | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/UIShell/GlobalHeader/HeaderNav.svelte b/src/UIShell/GlobalHeader/HeaderNav.svelte
index 85f19fc3..d7dd5e98 100644
--- a/src/UIShell/GlobalHeader/HeaderNav.svelte
+++ b/src/UIShell/GlobalHeader/HeaderNav.svelte
@@ -1,17 +1,19 @@
 <script>
   /**
    * Specify the ARIA label for the nav
+   * @deprecated use "aria-label" instead
    * @type {string}
    */
   export let ariaLabel = undefined;
+
+  $: props = {
+    "aria-label": ariaLabel || $$props["aria-label"],
+    "aria-labelledby": $$props["aria-labelledby"],
+  };
 </script>
 
-<nav aria-label="{ariaLabel}" class:bx--header__nav="{true}" {...$$restProps}>
-  <ul
-    role="menubar"
-    aria-label="{ariaLabel}"
-    class:bx--header__menu-bar="{true}"
-  >
+<nav {...props} class:bx--header__nav="{true}" {...$$restProps}>
+  <ul {...props} class:bx--header__menu-bar="{true}">
     <slot />
   </ul>
 </nav>