Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#55947 closed enhancement (fixed)

Permalinks screen, add guiding introduction to structure selection.

Reported by: kebbet's profile kebbet Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.1 Priority: normal
Severity: normal Version:
Component: Permalinks Keywords: has-patch has-screenshots needs-copy-review needs-user-docs
Focuses: ui, administration, ui-copy Cc:

Description

Enhancement-suggestion to add a guiding paragraph below the heading ”Common Settings”. Suggested phrasing:

Select the permalink structure for your website. A structure that includes the post name makes sharing links to others more understandable.

This is a breakout from #55498 and is partly related to #55557 which all focus on enhancing the Permalink-screen.

Attachments (6)

55947-permalink-guidence.diff (690 bytes) - added by kebbet 2 years ago.
55947-permalink-guidence.png (446.9 KB) - added by kebbet 2 years ago.
55947-permalink-guidence-v2.diff (689 bytes) - added by kebbet 2 years ago.
55947-permalink-guidence-v2.diff-on-mobile.jpg (60.4 KB) - added by costdev 2 years ago.
55947-permalink-guidence-v2.diff on mobile.
55947-3.diff (800 bytes) - added by kebbet 2 years ago.
55947-4.diff (770 bytes) - added by kebbet 2 years ago.

Download all attachments as: .zip

Change History (23)

#1 @kebbet
2 years ago

Ping @sabernhardt

#2 @SergeyBiryukov
2 years ago

  • Milestone changed from Awaiting Review to 6.1

#3 @SergeyBiryukov
2 years ago

  • Component changed from Administration to Permalinks

#4 @hasanuzzamanshamim
2 years ago

As per the previous discussion ref
https://github.com/WordPress/wordpress-develop/pull/2557#discussion_r847052998

The short guiding paragraph would be nice from my point of view.

#5 @mukesh27
2 years ago

  • Keywords needs-refresh added

Hi there!

Thanks for the ticket and patch.

Can you please remove the single space before _e in you 55947-permalink-guidence.diff patch because right now it has two spaces before it?

#6 @kebbet
2 years ago

  • Keywords needs-refresh removed

Thanks for noticing @mukesh27! New patch is on its way.

55947-permalink-guidence-v2.diff removes the extra single space before _e

Last edited 2 years ago by kebbet (previous) (diff)

#7 @kebbet
2 years ago

  • Keywords needs-copy-review needs-testing added

Steps to Test Patch

  1. Navigate to Settings > Permalinks.
  2. Observe the added paragraph between the heading Common Settings and the structure selection form below.


#8 @kebbet
2 years ago

  • Keywords needs-testing removed

@costdev
2 years ago

55947-permalink-guidence-v2.diff on mobile.

#9 @costdev
2 years ago

Attached a mobile screenshot of 55947-permalink-guidence-v2.diff to show there are no issues on smaller screens.


Select the permalink structure for your website. A structure that includes the post name makes sharing links to others more understandable.

  • The second sentence means that sharing links is more understandable, but it is the link itself that is easier to understand.
  • It may be useful to change "post name" to <code>%postname%</code> so the user knows exactly what to use. This would mean changing from _e() to something like:
printf(
    /* translators: %s: the %postname% tag. */
    __( '... includes the %s tag ...' ),
    '<code>%postname%</code>'
);
  • The post name can also help with SEO.

If you agree with the above, what about this variation?

Select the permalink structure for your website. Including the %postname% tag makes links easy to understand, and can help your posts rank higher in search engines.`

This ticket was mentioned in PR #2947 on WordPress/wordpress-develop by kebbet.


2 years ago
#10

Trac ticket:

@kebbet
2 years ago

#11 @kebbet
2 years ago

Thanks for reviewing and the feedback. I do agree on all points, a new patch is attached to the ticket. The linked PR at Github was made to make sure tests pass.

#12 @costdev
2 years ago

@kebbet LGTM 👍

#13 @kebbet
2 years ago

Linked PR (https://github.com/WordPress/wordpress-develop/pull/2947) is now updated after [53706] was merged to trunk.

@kebbet
2 years ago

#14 @kebbet
2 years ago

55947-4.diff and linked PR at GitHub updated after [53710].

#15 @SergeyBiryukov
2 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 53711:

Permalinks: Add a guiding paragraph on the Permalink Settings screen.

This is intended as an introduction to permalink structure selection.

Props kebbet, costdev, hasanuzzamanshamim, mukesh27.
Fixes #55947.

SergeyBiryukov commented on PR #2947:


2 years ago
#16

Thanks for the PR! Merged in r53711.

#17 @milana_cap
2 years ago

  • Keywords needs-user-docs added

Might need updating screenshot at HelpHub.

Note: See TracTickets for help on using tickets.