Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
user interface text
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
29 Oct 2011 at 19:53 UTC
Updated:
31 Dec 2019 at 11:38 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
yoroy commentedRelated: [1325976]
Trying a patch…
Comment #2
xjmOops, 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:
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.
Comment #3
yoroy commentedAgreed on using 'enter'. Hope to see someone take this on and re-roll :)
Comment #4
mallezieI rerolled the patch.
Corrected the indentation and changed "type" with "enter".
Before screenshot


After screenshot
And patch attached
Comment #5
xjmThanks! Setting NR so testbot picks it up.
Comment #6
mallezieIf im not mistaken test bots has troubles with dots, so renaming patch and rerolling.
and i forget to set to needs review.
Comment #7
kathyh commentedVerified patch applied cleanly and the new text is seen when new content is added.
Comment #8
xjm#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.
Comment #9
kathyh commentedComment #10
xjmNah, 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.)
Comment #11
catchLooks good. Committed/pushed to 8.x. Thanks!
Comment #13
ro-29 commentedComment #14
srikanthk16 commented#14 rerolled url alias description
Comment #15
Lord_of_Codes commentedSubmission of patch for Google-Code-In.
Comment #16
Lord_of_Codes commentedComment #17
Archisman commentedre-rolled the patch for gsoc ladder
Comment #18
Percy101 commented