Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove base url from link control search results #54553

Merged
merged 14 commits into from
Sep 21, 2023
+58 −7
Merged

Remove base url from link control search results #54553

Changes from 5 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
158c841
Expose baseURL setting on Post and Site Editors via block settings
getdave Sep 18, 2023
37e9bb1
Strip baseURL from rendered search results
getdave Sep 18, 2023
25c576a
Only fetch baseURL once in top level component
getdave Sep 18, 2023
bb545d8
Simplify implementation to utilise URL parse functions
getdave Sep 18, 2023
1e95736
Improve comment wording to avoid referencing undefined var
getdave Sep 18, 2023
9b84981
Remove superfluous conditional
getdave Sep 19, 2023
0d12f6f
Decode URL prior to operations
getdave Sep 19, 2023
5808ca7
Refactor for readability
getdave Sep 19, 2023
956152a
Fix where url is not defined
getdave Sep 19, 2023
dd9e6c4
Revert change to filter util
getdave Sep 19, 2023
724bffc
Ensure that filterURLForDisplay always receives a string as an arg
getdave Sep 19, 2023
d2e54f9
Make e2e test locator less strict
getdave Sep 19, 2023
67a4b67
Prefer pipe
getdave Sep 19, 2023
868b3e4
Force remove trailing slash
getdave Sep 21, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions packages/block-editor/src/components/link-control/search-item.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import {
file,
} from '@wordpress/icons';
import { __unstableStripHTML as stripHTML } from '@wordpress/dom';
import { safeDecodeURI, filterURLForDisplay } from '@wordpress/url';
import { safeDecodeURI, filterURLForDisplay, getPath } from '@wordpress/url';

const ICONS_MAP = {
post: postList,
Expand Down Expand Up @@ -52,9 +52,15 @@ export const LinkControlSearchItem = ( {
isURL = false,
shouldShowType = false,
} ) => {
// Remove the site URL from any suggestion that is not a manually entered URL.
// This is to reduce visual noise in the search results.
const suggestionURL = isURL
? suggestion?.url
: '/' + getPath( suggestion?.url ) + '/';

Mamaduka marked this conversation as resolved.
Show resolved Hide resolved
const info = isURL
? __( 'Press ENTER to add this link' )
: filterURLForDisplay( safeDecodeURI( suggestion?.url ), 24 );
: filterURLForDisplay( safeDecodeURI( suggestionURL ), 24 );

return (
<MenuItem
Expand Down
Loading