[Link] iteration

[aveline]

WHY are these changes introduced?

Fixes #8794 #8797

WHAT is this pull request doing?

• Removes BannerContext from Link component
• Removed external prop from Link component

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

Copy-paste this code in playground/Playground.tsx:

import React from 'react';
import {Page} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
    </Page>
  );
}

🎩 checklist

• Tested on mobile
• Tested on multiple browsers
• Tested for accessibility
• Updated the component's README.md with documentation changes
• Tophatted documentation changes in the style guide

[github-actions]

size-limit report 📦

Path	Size
polaris-react-cjs	242.89 KB (0%)
polaris-react-esm	157.14 KB (-0.04% 🔽)
polaris-react-esnext	219.69 KB (+0.02% 🔺)
polaris-react-css	47.5 KB (+0.12% 🔺)

[chazdean]

Changes look good to me! I like the CSS approach in Banner ✨

[aveline]

/snapit

[github-actions]

🫰✨ Thanks @aveline! Your snapshots have been published to npm.

Test the snapshots by updating your package.json with the newly published versions:

yarn add @shopify/polaris-cli@0.0.0-snapshot-release-20230515155511

yarn add @shopify/polaris-codemods@0.0.0-snapshot-release-20230515155511

yarn add @shopify/polaris-icons@0.0.0-snapshot-release-20230515155511

yarn add @shopify/polaris-migrator@0.0.0-snapshot-release-20230515155511

yarn add @shopify/polaris@0.0.0-snapshot-release-20230515155511

yarn add @shopify/polaris-tokens@0.0.0-snapshot-release-20230515155511

yarn add @shopify/stylelint-polaris@0.0.0-snapshot-release-20230515155511

[chloerice]

Looks great @aveline! We'll need to revert removing the external prop and deprecate instead.

[chloerice]

This prop will need to be deprecated first and then we can remove it in v12.

Suggested change

	/** Makes the link open in a new tab */
	external?: boolean;
	/**
	** @deprecated Set `target` to "_blank" instead
	** Makes the link open in a new tab
	*/
	external?: boolean;

[aveline]

Closing for #9223