Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Comment | File | Size | Author |
---|---|---|---|
#5 | fix_grammar_URL_UI-2585989-4.patch | 20.99 KB | imalabya |
#2 | fix_grammar_URL_UI-2585989-2.patch | 2.14 KB | pguillard |
Comments
Comment #1
pguillard CreditAttribution: pguillard commentedpguillard created an issue.
See #2574981 Fix grammar ("an URL" should be "a URL") and consistent use of URL (not "url" / "Url") for details
Comment #2
pguillard CreditAttribution: pguillard commentedPatch from 2574981 # 16
Comment #3
jhodgdonun-postponing. I don't think this completely covers the UI text though? Let's see. I grepped for " an URL" (case insensitive) and also for " url" and " Url" and found UI text that should be fixed in:
(and yes, I know many of these are in tests, but they are still text that is displayed in the UI if you run the tests)
(and I'm not sure if config schema descriptions are considered UI text or not? I think they may be shown on config translate forms, but not sure)
Comment #4
imalabyaWorking on it
Comment #5
imalabya@jhodgdon according to your comment have the changes.
Url -> URL
url -> URL
Comment #6
imalabyaComment #7
jhodgdonThanks! This patch looks perfect to me. All of the capitalization changes are correct as far as I can tell, and in addition, you've covered all of the spots I listed in #3. Great job!
Comment #8
catchCommitted/pushed to 8.1.x, thanks!