Current:

Optionally specify an alternative URL by which this content can be accessed. For example, type "about" when writing an about page. Use a relative path and don't add a trailing slash or the URL alias won't work.

Proposed:

The alternative URL for this content. For example, type "about" when creating an about page. Use a relative path without a trailing slash.
Files: 

Comments

yoroy’s picture

Status: Active » Needs review
FileSize
886 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,759 pass(es). View

Related: [1325976]

Trying a patch…

xjm’s picture

Status: Needs review » Needs work
+++ b/modules/path/path.moduleundefined
@@ -136,7 +136,7 @@ function path_form_node_form_alter(&$form, $form_state) {
+		'#description' => t('The alternative URL for this content. Use a relative path without a trailing slash. For example, type "about" for the about page.'),

Oops, some goofy indentation here.

The text change seems like an improvement to me. Except, maybe we should replace the word "type" with "enter"? We don't care whether they enter it by typing or not.

Two good novice tasks here are:

  1. Rerolling the patch with the suggested changes, using the updated directory structure for Drupal 8.x
  2. Posting "before" and "after" screenshots of this text change.

Note that the Drupal 8.x patch will need to be rerolled, because the core directory structure for Drupal 8 has now changed. (For more information, see #22336: Move all core Drupal files under a /core folder to improve usability and upgrades). When the patch has been rerolled, please set the issue back to "Needs Review."

If you need help rerolling this patch, you can come to core office hours or ask in #drupal-gitsupport on IRC.

yoroy’s picture

Agreed on using 'enter'. Hope to see someone take this on and re-roll :)

mallezie’s picture

FileSize
28.53 KB
28.96 KB
909 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,748 pass(es). View

I rerolled the patch.
Corrected the indentation and changed "type" with "enter".

Before screenshot
Before patch
After screenshot
After patch
And patch attached

xjm’s picture

Status: Needs work » Needs review
Issue tags: -Novice

Thanks! Setting NR so testbot picks it up.

mallezie’s picture

Issue tags: +Novice
FileSize
909 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,762 pass(es). View

If im not mistaken test bots has troubles with dots, so renaming patch and rerolling.
and i forget to set to needs review.

kathyh’s picture

Status: Needs review » Reviewed & tested by the community

Verified patch applied cleanly and the new text is seen when new content is added.

xjm’s picture

Issue tags: +string freeze

#7: Well, in addition to verifying that the patch does what it says, we also need to decide whether the change should be made and whether the proposed text is what we want. Typically we want consensus from a few people for these sorts of changes.

In this case, though, I think this is a clear text improvement and I'd agree this is RTBC.

kathyh’s picture

Status: Reviewed & tested by the community » Needs review
xjm’s picture

Status: Needs review » Reviewed & tested by the community

Nah, this is good. :) I was just clarifying the usual practice for RTBCing patches that introduce changes, based on your comment text. (yoroy is one of the UX topic maintainers.)

catch’s picture

Status: Reviewed & tested by the community » Fixed

Looks good. Committed/pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

ro-29’s picture

Title: Rewrite description for URL alias on node/add/* » Rewrite description for URL alias on node/add/* Patch
Issue summary: View changes
FileSize
908 bytes
4.6 KB
srikanthk16’s picture

#14 rerolled url alias description

Lord_of_Codes’s picture

Submission of patch for Google-Code-In.

Lord_of_Codes’s picture