From bfb6f107b9474fa42257398e357d3cce2c41cd60 Mon Sep 17 00:00:00 2001
From: George Hotelling <george.hotelling@automattic.com>
Date: Mon, 8 Feb 2021 17:15:03 -0500
Subject: [PATCH] Prototyping a new Link Nav Block Interaction

Working on #28440

Co-authored-by: Kerry Liu <kerry.liu@automattic.com>
---
 .../block-library/src/navigation-link/edit.js | 56 ++++++++++++-------
 .../src/navigation-link/style.scss            | 11 ++++
 2 files changed, 48 insertions(+), 19 deletions(-)

diff --git a/packages/block-library/src/navigation-link/edit.js b/packages/block-library/src/navigation-link/edit.js
index c6dd37884807e..545f4a79e3879 100644
--- a/packages/block-library/src/navigation-link/edit.js
+++ b/packages/block-library/src/navigation-link/edit.js
@@ -10,6 +10,7 @@ import { escape } from 'lodash';
 import { createBlock } from '@wordpress/blocks';
 import { useSelect, useDispatch } from '@wordpress/data';
 import {
+	__experimentalInputControl as InputControl,
 	KeyboardShortcuts,
 	PanelBody,
 	Popover,
@@ -232,21 +233,21 @@ export default function NavigationLinkEdit( {
 	}, [ isSelected ] );
 
 	// If the LinkControl popover is open and the URL has changed, close the LinkControl and focus the label text.
-	useEffect( () => {
-		if ( isLinkOpen && url ) {
-			// Does this look like a URL and have something TLD-ish?
-			if (
-				isURL( prependHTTP( label ) ) &&
-				/^.+\.[a-z]+/.test( label )
-			) {
-				// Focus and select the label text.
-				selectLabelText();
-			} else {
-				// Focus it (but do not select).
-				placeCaretAtHorizontalEdge( ref.current, true );
-			}
-		}
-	}, [ url ] );
+	// useEffect( () => {
+	// 	if ( isLinkOpen && url ) {
+	// 		// Does this look like a URL and have something TLD-ish?
+	// 		if (
+	// 			isURL( prependHTTP( label ) ) &&
+	// 			/^.+\.[a-z]+/.test( label )
+	// 		) {
+	// 			// Focus and select the label text.
+	// 			selectLabelText();
+	// 		} else {
+	// 			// Focus it (but do not select).
+	// 			placeCaretAtHorizontalEdge( ref.current, true );
+	// 		}
+	// 	}
+	// }, [ url ] );
 
 	/**
 	 * Focus the Link label text and select it.
@@ -302,6 +303,7 @@ export default function NavigationLinkEdit( {
 			[ `has-${ textColor }-color` ]: !! textColor,
 			'has-background': !! backgroundColor || !! style?.color?.background,
 			[ `has-${ backgroundColor }-background-color` ]: !! backgroundColor,
+			'is-missing-link': ! url,
 		} ),
 		style: {
 			color: style?.color?.text,
@@ -415,13 +417,31 @@ export default function NavigationLinkEdit( {
 							'core/image',
 							'core/strikethrough',
 						] }
+						onClick={ () => {
+							if ( ! url ) {
+								setIsLinkOpen( true );
+							}
+						} }
 					/>
 					{ isLinkOpen && (
 						<Popover
 							position="bottom center"
 							onClose={ () => setIsLinkOpen( false ) }
+							className="wp-block-navigation-link__popover-link-input"
 						>
-							<LinkControl
+							<InputControl
+								label={ __( 'Text' ) }
+								onChange={ ( label ) =>
+									setAttributes( { label } )
+								}
+							/>
+							<InputControl
+								label={ __( 'URL' ) }
+								onChange={ ( url ) =>
+									setAttributes( { url: encodeURI( url ) } )
+								}
+							/>
+							{ /* <LinkControl
 								className="wp-block-navigation-link__inline-link-input"
 								value={ link }
 								showInitialSuggestions={ true }
@@ -430,12 +450,10 @@ export default function NavigationLinkEdit( {
 								createSuggestionButtonText={ ( searchTerm ) => {
 									let format;
 									if ( type === 'post' ) {
-										/* translators: %s: search term. */
 										format = __(
 											'Create draft post: <mark>%s</mark>'
 										);
 									} else {
-										/* translators: %s: search term. */
 										format = __(
 											'Create draft page: <mark>%s</mark>'
 										);
@@ -482,7 +500,7 @@ export default function NavigationLinkEdit( {
 										id,
 									} )
 								}
-							/>
+							/> */ }
 						</Popover>
 					) }
 				</div>
diff --git a/packages/block-library/src/navigation-link/style.scss b/packages/block-library/src/navigation-link/style.scss
index 34558950e3391..8e2f9c97ea68e 100644
--- a/packages/block-library/src/navigation-link/style.scss
+++ b/packages/block-library/src/navigation-link/style.scss
@@ -26,6 +26,11 @@
 	}
 }
 
+// foo
+.is-missing-link {
+	border: solid 1px red;
+}
+
 // Styles for submenu flyout
 .has-child {
 	> .wp-block-navigation-link__content {
@@ -160,3 +165,9 @@
 		}
 	}
 }
+
+.wp-block-navigation-link__popover-link-input {
+	max-width: 360px;
+	min-width: auto;
+	width: 90vw;
+}