[PANGOO-2594][BpkNavigationTabGroup]Fix BPK NavTabGroup Warning

[kerrie625]

There is warning For BPK NavTabGroup.So need to fix.

Remember to include the following changes:

• Ensure the PR title includes the name of the component you are changing so it's clear in the release notes for consumers of the changes in the version e.g [KOA-123][BpkButton] Updating the colour
• README.md (If you have created a new component)
• Component README.md
• Tests
• Accessibility tests
  • The following checks were performed:
    • Ability to navigate using a keyboard only
    • Zoom functionality (Deque University explanation):
      • The page SHOULD be functional AND readable when only the text is magnified to 200% of its initial size
      • Pages must reflow as zoom increases up to 400% so that content continues to be presented in only one column i.e. Content MUST NOT require scrolling in two directions (both vertically and horizontally)
    • Ability to navigate using a screen reader only
• Storybook examples created/updated
• For breaking changes or deprecating components/properties, migration guides added to the description of the PR. If the guide has large changes, consider creating a new Markdown page inside the component's docs folder and link it here

[github-actions]

Visit https://backpack.github.io/storybook-prs/3618 to see this build running in a browser.

[github-actions]

	Warnings
⚠️	Package source files (e.g. packages/package-name/src/Component.js) were updated, but snapshots weren't. Have you checked that the tests still pass?
⚠️	Package source files (e.g. packages/package-name/src/Component.tsx) were updated, but type files weren't. Have you checked that no types have changed?

Browser support

If this is a visual change, make sure you've tested it in multiple browsers.

Generated by 🚫 dangerJS against 930fd3f

[amburrrshand]

Why did you decide to remove the ariaLabel?

[kerrie625]

Thanks @amburrrshand .Not remove it, Just change ariaLabel as optional. And this example is used for test props without ariaLabel

[olliecurtis]

What is the reasoning to make it optional?

Surely by doing this we are decreasing Accessibility for travellers?

[kerrie625]

When checking header tabs on production, I found the navigation tabs did not include ariaLabel before. So I think it needs to change this ariaLabel to option. Is it reasonable for consumer decide to whether pass this ariaLabel?

[amburrrshand]

In this case, I think it's best practice for accessibility standards to not have it as optional

[kerrie625]

@olliecurtis @amburrrshand Thank you for your advice. I have reverted the ariaLabel as required.

[github-actions]

Visit https://backpack.github.io/storybook-prs/3618 to see this build running in a browser.

[github-actions]

Visit https://backpack.github.io/storybook-prs/3618 to see this build running in a browser.

[amburrrshand]

LGTM