From a63765f5d688ffdddda254426ea7cccb9f09aaa5 Mon Sep 17 00:00:00 2001
From: Rich Tabor <hi@richtabor.com>
Date: Wed, 24 Jan 2024 08:02:36 -0500
Subject: [PATCH 01/11] Add copySmall icon

---
 packages/icons/src/index.js              |  1 +
 packages/icons/src/library/copy-small.js | 16 ++++++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 packages/icons/src/library/copy-small.js

diff --git a/packages/icons/src/index.js b/packages/icons/src/index.js
index d743299d35a24..aa7a8ee90778c 100644
--- a/packages/icons/src/index.js
+++ b/packages/icons/src/index.js
@@ -52,6 +52,7 @@ export { default as color } from './library/color';
 export { default as column } from './library/column';
 export { default as columns } from './library/columns';
 export { default as copy } from './library/copy';
+export { default as copySmall } from './library/copy-small';
 export { default as comment } from './library/comment';
 export { default as commentAuthorAvatar } from './library/comment-author-avatar';
 export { default as commentAuthorName } from './library/comment-author-name';
diff --git a/packages/icons/src/library/copy-small.js b/packages/icons/src/library/copy-small.js
new file mode 100644
index 0000000000000..07f6bff15d59b
--- /dev/null
+++ b/packages/icons/src/library/copy-small.js
@@ -0,0 +1,16 @@
+/**
+ * WordPress dependencies
+ */
+import { SVG, Path } from '@wordpress/primitives';
+
+const copySmall = (
+	<SVG xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24">
+		<Path
+			fillRule="evenodd"
+			clipRule="evenodd"
+			d="M5.625 5.5h9.75c.069 0 .125.056.125.125v9.75a.125.125 0 0 1-.125.125h-9.75a.125.125 0 0 1-.125-.125v-9.75c0-.069.056-.125.125-.125ZM4 5.625C4 4.728 4.728 4 5.625 4h9.75C16.273 4 17 4.728 17 5.625v9.75c0 .898-.727 1.625-1.625 1.625h-9.75A1.625 1.625 0 0 1 4 15.375v-9.75Zm14.5 11.656v-9H20v9C20 18.8 18.77 20 17.251 20H6.25v-1.5h11.001c.69 0 1.249-.528 1.249-1.219Z"
+		/>
+	</SVG>
+);
+
+export default copySmall;

From 9c69dd600523b3ded81f73c71e6e7e304aec6f6c Mon Sep 17 00:00:00 2001
From: Rich Tabor <hi@richtabor.com>
Date: Wed, 24 Jan 2024 08:03:47 -0500
Subject: [PATCH 02/11] Remove additionalControls + consistent labels

---
 .../src/components/link-control/link-preview.js      | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/link-preview.js b/packages/block-editor/src/components/link-control/link-preview.js
index 17cc851c503a1..3f317429c08e5 100644
--- a/packages/block-editor/src/components/link-control/link-preview.js
+++ b/packages/block-editor/src/components/link-control/link-preview.js
@@ -15,7 +15,7 @@ import {
 } from '@wordpress/components';
 import { useCopyToClipboard } from '@wordpress/compose';
 import { filterURLForDisplay, safeDecodeURI } from '@wordpress/url';
-import { Icon, globe, info, linkOff, edit, copy } from '@wordpress/icons';
+import { Icon, globe, info, linkOff, edit, copySmall } from '@wordpress/icons';
 import { __unstableStripHTML as stripHTML } from '@wordpress/dom';
 import { useDispatch } from '@wordpress/data';
 import { store as noticesStore } from '@wordpress/notices';
@@ -33,7 +33,6 @@ export default function LinkPreview( {
 	hasRichPreviews = false,
 	hasUnlinkControl = false,
 	onRemove,
-	additionalControls,
 } ) {
 	// Avoid fetching if rich previews are not desired.
 	const showRichPreviews = hasRichPreviews ? value?.url : null;
@@ -127,7 +126,7 @@ export default function LinkPreview( {
 
 				<Button
 					icon={ edit }
-					label={ __( 'Edit' ) }
+					label={ __( 'Edit link' ) }
 					className="block-editor-link-control__search-item-action"
 					onClick={ onEditClick }
 					size="compact"
@@ -135,15 +134,15 @@ export default function LinkPreview( {
 				{ hasUnlinkControl && (
 					<Button
 						icon={ linkOff }
-						label={ __( 'Unlink' ) }
+						label={ __( 'Remove link' ) }
 						className="block-editor-link-control__search-item-action block-editor-link-control__unlink"
 						onClick={ onRemove }
 						size="compact"
 					/>
 				) }
 				<Button
-					icon={ copy }
-					label={ __( 'Copy URL' ) }
+					icon={ copySmall }
+					label={ __( 'Copy link' ) }
 					className="block-editor-link-control__search-item-action block-editor-link-control__copy"
 					ref={ ref }
 					disabled={ isEmptyURL }
@@ -151,7 +150,6 @@ export default function LinkPreview( {
 				/>
 				<ViewerSlot fillProps={ value } />
 			</div>
-			{ additionalControls && additionalControls() }
 		</div>
 	);
 }

From 29bc6d54cbab7da6340096104a404af96bdd7f31 Mon Sep 17 00:00:00 2001
From: scruffian <ben@scruffian.com>
Date: Thu, 25 Jan 2024 16:16:11 +0000
Subject: [PATCH 03/11] remove test we don't need

---
 .../src/components/link-control/test/index.js | 60 -------------------
 1 file changed, 60 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/test/index.js b/packages/block-editor/src/components/link-control/test/index.js
index 054f7a3ea8746..7cdbcf5f1aff6 100644
--- a/packages/block-editor/src/components/link-control/test/index.js
+++ b/packages/block-editor/src/components/link-control/test/index.js
@@ -1816,66 +1816,6 @@ describe( 'Selecting links', () => {
 } );
 
 describe( 'Addition Settings UI', () => {
-	it( 'should allow toggling the "Opens in new tab" setting control (only) on the link preview', async () => {
-		const user = userEvent.setup();
-		const selectedLink = fauxEntitySuggestions[ 0 ];
-		const mockOnChange = jest.fn();
-
-		const customSettings = [
-			{
-				id: 'opensInNewTab',
-				title: 'Open in new tab',
-			},
-			{
-				id: 'noFollow',
-				title: 'No follow',
-			},
-		];
-
-		const LinkControlConsumer = () => {
-			const [ link, setLink ] = useState( selectedLink );
-
-			return (
-				<LinkControl
-					value={ link }
-					settings={ customSettings }
-					onChange={ ( newVal ) => {
-						mockOnChange( newVal );
-						setLink( newVal );
-					} }
-				/>
-			);
-		};
-
-		render( <LinkControlConsumer /> );
-
-		const opensInNewTabField = screen.queryByRole( 'checkbox', {
-			name: 'Open in new tab',
-			checked: false,
-		} );
-
-		expect( opensInNewTabField ).toBeInTheDocument();
-
-		// No matter which settings are passed in only the `Opens in new tab`
-		// setting should be shown on the link preview (non-editing) state.
-		const noFollowField = screen.queryByRole( 'checkbox', {
-			name: 'No follow',
-		} );
-		expect( noFollowField ).not.toBeInTheDocument();
-
-		// Check that the link value is updated immediately upon checking
-		// the checkbox.
-		await user.click( opensInNewTabField );
-
-		expect( opensInNewTabField ).toBeChecked();
-
-		expect( mockOnChange ).toHaveBeenCalledTimes( 1 );
-		expect( mockOnChange ).toHaveBeenCalledWith( {
-			...selectedLink,
-			opensInNewTab: true,
-		} );
-	} );
-
 	it( 'should hide advanced link settings and toggle when not editing a link', async () => {
 		const selectedLink = fauxEntitySuggestions[ 0 ];
 

From b9cfbd8d62a4bb74a073abe8f351b36406866269 Mon Sep 17 00:00:00 2001
From: Rich Tabor <hi@richtabor.com>
Date: Thu, 25 Jan 2024 11:36:20 -0500
Subject: [PATCH 04/11] Remove classes/styles we don't need and tooltip, tidy
 snackbar

---
 .../components/link-control/link-preview.js   | 25 +++++++------------
 .../src/components/link-control/style.scss    | 10 --------
 2 files changed, 9 insertions(+), 26 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/link-preview.js b/packages/block-editor/src/components/link-control/link-preview.js
index 3f317429c08e5..169b06723355e 100644
--- a/packages/block-editor/src/components/link-control/link-preview.js
+++ b/packages/block-editor/src/components/link-control/link-preview.js
@@ -11,7 +11,6 @@ import {
 	Button,
 	ExternalLink,
 	__experimentalTruncate as Truncate,
-	Tooltip,
 } from '@wordpress/components';
 import { useCopyToClipboard } from '@wordpress/compose';
 import { filterURLForDisplay, safeDecodeURI } from '@wordpress/url';
@@ -65,7 +64,7 @@ export default function LinkPreview( {
 
 	const { createNotice } = useDispatch( noticesStore );
 	const ref = useCopyToClipboard( value.url, () => {
-		createNotice( 'info', __( 'Copied URL to clipboard.' ), {
+		createNotice( 'info', __( 'Link copied to clipboard.' ), {
 			isDismissible: true,
 			type: 'snackbar',
 		} );
@@ -98,16 +97,14 @@ export default function LinkPreview( {
 					<span className="block-editor-link-control__search-item-details">
 						{ ! isEmptyURL ? (
 							<>
-								<Tooltip text={ value.url }>
-									<ExternalLink
-										className="block-editor-link-control__search-item-title"
-										href={ value.url }
-									>
-										<Truncate numberOfLines={ 1 }>
-											{ displayTitle }
-										</Truncate>
-									</ExternalLink>
-								</Tooltip>
+								<ExternalLink
+									className="block-editor-link-control__search-item-title"
+									href={ value.url }
+								>
+									<Truncate numberOfLines={ 1 }>
+										{ displayTitle }
+									</Truncate>
+								</ExternalLink>
 								{ value?.url && displayTitle !== displayURL && (
 									<span className="block-editor-link-control__search-item-info">
 										<Truncate numberOfLines={ 1 }>
@@ -123,11 +120,9 @@ export default function LinkPreview( {
 						) }
 					</span>
 				</span>
-
 				<Button
 					icon={ edit }
 					label={ __( 'Edit link' ) }
-					className="block-editor-link-control__search-item-action"
 					onClick={ onEditClick }
 					size="compact"
 				/>
@@ -135,7 +130,6 @@ export default function LinkPreview( {
 					<Button
 						icon={ linkOff }
 						label={ __( 'Remove link' ) }
-						className="block-editor-link-control__search-item-action block-editor-link-control__unlink"
 						onClick={ onRemove }
 						size="compact"
 					/>
@@ -143,7 +137,6 @@ export default function LinkPreview( {
 				<Button
 					icon={ copySmall }
 					label={ __( 'Copy link' ) }
-					className="block-editor-link-control__search-item-action block-editor-link-control__copy"
 					ref={ ref }
 					disabled={ isEmptyURL }
 					size="compact"
diff --git a/packages/block-editor/src/components/link-control/style.scss b/packages/block-editor/src/components/link-control/style.scss
index 7b6470df43543..86dcf1b3e5ee5 100644
--- a/packages/block-editor/src/components/link-control/style.scss
+++ b/packages/block-editor/src/components/link-control/style.scss
@@ -352,11 +352,6 @@ $block-editor-link-control-number-of-actions: 1;
 	position: relative;
 }
 
-.block-editor-link-control__unlink {
-	padding-left: $grid-unit-20;
-	padding-right: $grid-unit-20;
-}
-
 .block-editor-link-control__setting {
 	margin-bottom: 0;
 	flex: 1;
@@ -425,8 +420,3 @@ $block-editor-link-control-number-of-actions: 1;
 	top: calc(50% + #{$spinner-size} / 4); // Add top spacing because this input has a visual label.
 	right: $grid-unit-15;
 }
-
-.block-editor-link-control__search-item-action {
-	margin-left: auto; // push to far right hand side
-	flex-shrink: 0;
-}

From 50106c4118c02d434b1c65e067599b80798c869f Mon Sep 17 00:00:00 2001
From: Rich Tabor <hi@richtabor.com>
Date: Thu, 25 Jan 2024 11:41:37 -0500
Subject: [PATCH 05/11] Fix focus style

---
 packages/block-editor/src/components/link-control/style.scss | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/packages/block-editor/src/components/link-control/style.scss b/packages/block-editor/src/components/link-control/style.scss
index 86dcf1b3e5ee5..8ba11f9ce3308 100644
--- a/packages/block-editor/src/components/link-control/style.scss
+++ b/packages/block-editor/src/components/link-control/style.scss
@@ -245,6 +245,10 @@ $block-editor-link-control-number-of-actions: 1;
 		border-radius: $radius-block-ui;
 		line-height: 1.1;
 
+		&:focus {
+			box-shadow: none;
+		}
+
 		&:focus-visible {
 			@include block-toolbar-button-style__focus();
 			text-decoration: none;

From 1d5eab17b9b6615dd9b9d55a7bf8032b593407d6 Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Thu, 25 Jan 2024 14:19:31 -0600
Subject: [PATCH 06/11] Fix test for pattern overrides

---
 test/e2e/specs/editor/various/pattern-overrides.spec.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/e2e/specs/editor/various/pattern-overrides.spec.js b/test/e2e/specs/editor/various/pattern-overrides.spec.js
index 8a680c351e4fc..1747b09e63d9f 100644
--- a/test/e2e/specs/editor/various/pattern-overrides.spec.js
+++ b/test/e2e/specs/editor/various/pattern-overrides.spec.js
@@ -305,7 +305,7 @@ test.describe( 'Pattern Overrides', () => {
 			exact: true,
 		} );
 		const editLinkButton = page.getByRole( 'button', {
-			name: 'Edit',
+			name: 'Edit link',
 			exact: true,
 		} );
 		const saveLinkButton = page.getByRole( 'button', {

From 6d33855c889ed75a585525988dfb6c5ff9b0b0ab Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Thu, 25 Jan 2024 14:30:00 -0600
Subject: [PATCH 07/11] Fix e2e tests

---
 test/e2e/specs/editor/blocks/links.spec.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/test/e2e/specs/editor/blocks/links.spec.js b/test/e2e/specs/editor/blocks/links.spec.js
index 2e6681c2d0701..7ebdb33b10353 100644
--- a/test/e2e/specs/editor/blocks/links.spec.js
+++ b/test/e2e/specs/editor/blocks/links.spec.js
@@ -228,7 +228,9 @@ test.describe( 'Links', () => {
 		await LinkUtils.createLink();
 
 		// Click on the Edit button.
-		await page.getByRole( 'button', { name: 'Edit', exact: true } ).click();
+		await page
+			.getByRole( 'button', { name: 'Edit link', exact: true } )
+			.click();
 
 		// Change the URL.
 		// getByPlaceholder required in order to handle Link Control component
@@ -255,7 +257,9 @@ test.describe( 'Links', () => {
 
 		const linkPopover = LinkUtils.getLinkPopover();
 
-		await linkPopover.getByRole( 'button', { name: 'Unlink' } ).click();
+		await linkPopover
+			.getByRole( 'button', { name: 'Remove link' } )
+			.click();
 
 		// The link should have been removed.
 		await expect.poll( editor.getBlocks ).toMatchObject( [

From ea8d9a29a3890d324cfe98a8d9d9e2c8e9868259 Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Tue, 6 Feb 2024 10:34:35 -0600
Subject: [PATCH 08/11] Add additionalControls back in. It can be removed in a
 separate PR if necessary.

---
 .../block-editor/src/components/link-control/link-preview.js    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/block-editor/src/components/link-control/link-preview.js b/packages/block-editor/src/components/link-control/link-preview.js
index 169b06723355e..f52cda30f9c14 100644
--- a/packages/block-editor/src/components/link-control/link-preview.js
+++ b/packages/block-editor/src/components/link-control/link-preview.js
@@ -32,6 +32,7 @@ export default function LinkPreview( {
 	hasRichPreviews = false,
 	hasUnlinkControl = false,
 	onRemove,
+	additionalControls,
 } ) {
 	// Avoid fetching if rich previews are not desired.
 	const showRichPreviews = hasRichPreviews ? value?.url : null;
@@ -143,6 +144,7 @@ export default function LinkPreview( {
 				/>
 				<ViewerSlot fillProps={ value } />
 			</div>
+			{ additionalControls && additionalControls() }
 		</div>
 	);
 }

From 682cfc40c429d45fa9751a55faf579a4c21f1ecb Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Tue, 6 Feb 2024 11:11:11 -0600
Subject: [PATCH 09/11] Rename Edit -> Edit link and Unlink -> Remove link to
 fix unit tests

---
 .../src/components/link-control/test/index.js | 41 +++++--------------
 .../media-replace-flow/test/index.js          |  2 +-
 2 files changed, 12 insertions(+), 31 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/test/index.js b/packages/block-editor/src/components/link-control/test/index.js
index 7cdbcf5f1aff6..411ae62d4eee1 100644
--- a/packages/block-editor/src/components/link-control/test/index.js
+++ b/packages/block-editor/src/components/link-control/test/index.js
@@ -321,7 +321,7 @@ describe( 'Basic rendering', () => {
 
 			// Click the "Edit" button to trigger into the editing mode.
 			const editButton = screen.queryByRole( 'button', {
-				name: 'Edit',
+				name: 'Edit link',
 			} );
 
 			await user.click( editButton );
@@ -379,7 +379,7 @@ describe( 'Basic rendering', () => {
 			render( <LinkControl value={ { url: 'https://example.com' } } /> );
 
 			const unLinkButton = screen.queryByRole( 'button', {
-				name: 'Unlink',
+				name: 'Remove link',
 			} );
 
 			expect( unLinkButton ).not.toBeInTheDocument();
@@ -397,7 +397,7 @@ describe( 'Basic rendering', () => {
 			);
 
 			const unLinkButton = screen.queryByRole( 'button', {
-				name: 'Unlink',
+				name: 'Remove link',
 			} );
 			expect( unLinkButton ).toBeVisible();
 
@@ -418,7 +418,7 @@ describe( 'Basic rendering', () => {
 			);
 
 			const unLinkButton = screen.queryByRole( 'button', {
-				name: 'Unlink',
+				name: 'Remove link',
 			} );
 			expect( unLinkButton ).toBeVisible();
 
@@ -822,7 +822,7 @@ describe( 'Manual link entry', () => {
 
 			// Click the "Edit" button to trigger into the editing mode.
 			let editButton = screen.queryByRole( 'button', {
-				name: 'Edit',
+				name: 'Edit link',
 			} );
 
 			await user.click( editButton );
@@ -856,7 +856,7 @@ describe( 'Manual link entry', () => {
 
 			// Re-query the edit button as it's been replaced.
 			editButton = screen.queryByRole( 'button', {
-				name: 'Edit',
+				name: 'Edit link',
 			} );
 
 			await user.click( editButton );
@@ -1060,7 +1060,7 @@ describe( 'Default search suggestions', () => {
 		// shown.
 		const currentLinkUI = screen.getByLabelText( 'Currently selected' );
 		const currentLinkBtn = within( currentLinkUI ).getByRole( 'button', {
-			name: 'Edit',
+			name: 'Edit link',
 		} );
 		await user.click( currentLinkBtn );
 
@@ -1502,7 +1502,7 @@ describe( 'Selecting links', () => {
 
 		expect( currentLink ).toBeVisible();
 		expect(
-			screen.queryByRole( 'button', { name: 'Edit' } )
+			screen.queryByRole( 'button', { name: 'Edit link' } )
 		).toBeVisible();
 		expect( currentLinkAnchor ).toBeVisible();
 	} );
@@ -1527,7 +1527,7 @@ describe( 'Selecting links', () => {
 		// Required in order to select the button below.
 		let currentLinkUI = screen.getByLabelText( 'Currently selected' );
 		const currentLinkBtn = within( currentLinkUI ).getByRole( 'button', {
-			name: 'Edit',
+			name: 'Edit link',
 		} );
 
 		// Simulate searching for a term.
@@ -1597,7 +1597,7 @@ describe( 'Selecting links', () => {
 
 				// Check that this suggestion is now shown as selected.
 				expect(
-					screen.getByRole( 'button', { name: 'Edit' } )
+					screen.getByRole( 'button', { name: 'Edit link' } )
 				).toBeVisible();
 				expect( currentLinkAnchor ).toBeVisible();
 			}
@@ -1709,7 +1709,7 @@ describe( 'Selecting links', () => {
 
 				expect( currentLink ).toBeVisible();
 				expect(
-					screen.getByRole( 'button', { name: 'Edit' } )
+					screen.getByRole( 'button', { name: 'Edit link' } )
 				).toBeVisible();
 				expect( currentLinkAnchor ).toBeVisible();
 			}
@@ -1996,25 +1996,6 @@ describe( 'Addition Settings UI', () => {
 			} )
 		);
 	} );
-
-	it( 'should show tooltip with full URL alongside filtered display', async () => {
-		const user = userEvent.setup();
-		const url =
-			'http://www.wordpress.org/wp-content/uploads/a-document.pdf';
-		render( <LinkControl value={ { url } } /> );
-
-		const link = screen.getByRole( 'link' );
-
-		expect( link ).toHaveTextContent( 'a-document.pdf' );
-
-		await user.hover( link );
-
-		expect( await screen.findByRole( 'tooltip' ) ).toHaveTextContent( url );
-
-		await user.unhover( link );
-
-		expect( screen.queryByRole( 'tooltip' ) ).not.toBeInTheDocument();
-	} );
 } );
 
 describe( 'Post types', () => {
diff --git a/packages/block-editor/src/components/media-replace-flow/test/index.js b/packages/block-editor/src/components/media-replace-flow/test/index.js
index 0af3ad1db6684..1d326804f0dcc 100644
--- a/packages/block-editor/src/components/media-replace-flow/test/index.js
+++ b/packages/block-editor/src/components/media-replace-flow/test/index.js
@@ -105,7 +105,7 @@ describe( 'General media replace flow', () => {
 
 		await user.click(
 			screen.getByRole( 'button', {
-				name: 'Edit',
+				name: 'Edit link',
 			} )
 		);
 

From 0e857f48ff2c2fb05489b75a9a6f17562cb87ad0 Mon Sep 17 00:00:00 2001
From: Jerry Jones <jones.jeremydavid@gmail.com>
Date: Tue, 6 Feb 2024 14:08:13 -0600
Subject: [PATCH 10/11] Add url to copy link button

---
 .../block-editor/src/components/link-control/link-preview.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/packages/block-editor/src/components/link-control/link-preview.js b/packages/block-editor/src/components/link-control/link-preview.js
index f52cda30f9c14..7919b9c31d7ef 100644
--- a/packages/block-editor/src/components/link-control/link-preview.js
+++ b/packages/block-editor/src/components/link-control/link-preview.js
@@ -137,7 +137,10 @@ export default function LinkPreview( {
 				) }
 				<Button
 					icon={ copySmall }
-					label={ __( 'Copy link' ) }
+					// Ends up looking like "Copy link: https://example.com"
+					label={ `${ __( 'Copy link' ) }${
+						isEmptyURL ? '' : `: ${ value.url }`
+					}` }
 					ref={ ref }
 					disabled={ isEmptyURL }
 					size="compact"

From e91e5050c325911c9358b84482ab2c1c0034d19f Mon Sep 17 00:00:00 2001
From: Dave Smith <getdavemail@gmail.com>
Date: Wed, 7 Feb 2024 10:52:00 +0000
Subject: [PATCH 11/11] Use sprintf for copy link

---
 .../src/components/link-control/link-preview.js      | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/packages/block-editor/src/components/link-control/link-preview.js b/packages/block-editor/src/components/link-control/link-preview.js
index 7919b9c31d7ef..05eeb3769a942 100644
--- a/packages/block-editor/src/components/link-control/link-preview.js
+++ b/packages/block-editor/src/components/link-control/link-preview.js
@@ -6,7 +6,7 @@ import classnames from 'classnames';
 /**
  * WordPress dependencies
  */
-import { __ } from '@wordpress/i18n';
+import { __, sprintf } from '@wordpress/i18n';
 import {
 	Button,
 	ExternalLink,
@@ -137,10 +137,12 @@ export default function LinkPreview( {
 				) }
 				<Button
 					icon={ copySmall }
-					// Ends up looking like "Copy link: https://example.com"
-					label={ `${ __( 'Copy link' ) }${
-						isEmptyURL ? '' : `: ${ value.url }`
-					}` }
+					label={ sprintf(
+						// Translators: %1$s is a placeholder for an optional colon, %2$s is a placeholder for the link URL (if present).
+						__( 'Copy link%1$s%2$s' ), // Ends up looking like "Copy link: https://example.com".
+						isEmptyURL ? '' : ': ',
+						value.url
+					) }
 					ref={ ref }
 					disabled={ isEmptyURL }
 					size="compact"