Skip to content

feat: clarify label & add help text with link #70451

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

Merged
Mamaduka merged 17 commits into from
Sep 16, 2025
Merged

feat: clarify label & add help text with link #70451

Changes from 3 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
9a1a576
feat: restore code
coderGtm Jun 10, 2025
95ebc0d
feat: clarify label & add help text with link
coderGtm Jun 17, 2025
7a0e65c
i18n: remove en-US preset from doc link
coderGtm Jun 17, 2025
87d4657
Merge branch 'WordPress:trunk' into trunk
coderGtm Jun 20, 2025
3c8418a
Merge branch 'WordPress:trunk' into fix/link-rel-text
coderGtm Jun 20, 2025
df5efda
Merge remote-tracking branch 'upstream/trunk' into trunk
coderGtm Sep 1, 2025
0047bd8
Merge branch 'trunk' into fix/link-rel-text
coderGtm Sep 1, 2025
6288f0e
Merge branch 'WordPress:trunk' into fix/link-rel-text
coderGtm Sep 1, 2025
6d46aee
Merge branch 'fix/link-rel-text' of https://github.com/coderGtm/guten…
coderGtm Sep 1, 2025
9c17788
fix: replace span with react fragment
coderGtm Sep 1, 2025
fcf900b
fix: remove unrelated changes
coderGtm Sep 5, 2025
31b697b
Merge remote-tracking branch 'upstream/trunk' into fix/link-rel-text
coderGtm Sep 8, 2025
2b71099
feat: update all 'Link rel' label to 'Link Relation' and add help text
coderGtm Sep 8, 2025
866c08d
Merge remote-tracking branch 'upstream/trunk' into fix/link-rel-text
coderGtm Sep 8, 2025
a2bafbd
refactor: update help text for 'Link Relation' to use interpolated el…
coderGtm Sep 10, 2025
83ef20b
fix: standardize label casing for 'Link Relation' to 'Link relation'
coderGtm Sep 15, 2025
2c610ef
refactor: use 'a' as the link placeholder instead of 'linkRel'
coderGtm Sep 15, 2025
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
13 changes: 12 additions & 1 deletion packages/block-editor/src/components/url-popover/image-url-input-ui.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import {
ToggleControl,
TextControl,
__experimentalVStack as VStack,
ExternalLink,
} from '@wordpress/components';
import {
Icon,
Expand Down Expand Up @@ -229,9 +230,19 @@ const ImageURLInputUI = ( {
<TextControl
__next40pxDefaultSize
__nextHasNoMarginBottom
label={ __( 'Link rel' ) }
label={ __( 'Link Relation' ) }

@Mamaduka Mamaduka Sep 9, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
label={ __( 'Link Relation' ) }
label={ __( 'Link relation' ) }

I think similar labels use sentence case. See: https://github.com/WordPress/gutenberg/blob/trunk/docs/contributors/documentation/copy-guide.md#five-pay-attention-to-capitalization

value={ rel ?? '' }
onChange={ onSetLinkRel }
help={
<span>
Comment thread
Mamaduka marked this conversation as resolved.
Resolved
{ __(
'The Link Relation attribute defines the relationship between a linked resource and the current document.'

@Mamaduka Mamaduka Sep 9, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'The Link Relation attribute defines the relationship between a linked resource and the current document.'
'The link relation attribute defines the relationship between a linked resource and the current document.'

I think we should use standard casing here. This matches casing in the MDN article.

Note: Label and text changes should be the same everywhere.

) }{ ' ' }
<ExternalLink href="https://developer.mozilla.org/docs/Web/HTML/Attributes/rel">
{ __( 'Learn more.' ) }
</ExternalLink>
</span>
}
/>
<TextControl
__next40pxDefaultSize
Expand Down