I am reviewing some modules and noticed that the name for this module could use some title-casing.
For Reference: https://www.drupal.org/docs/develop/documenting-your-project/help-text-standards
Capitalize the name of the module, because module names are proper nouns.
This issue affects a few things: Drupal 7 version, Drupal 8 version, and project page. I'll submit a patch to this issue for 7.x, and create a new issue for the 8.x patch.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | after.png | 12.02 KB | volkswagenchick |
| #3 | before.png | 14.48 KB | volkswagenchick |
| #2 | 2984262-2-noreferrer-titlecase-module-name.patch | 1.27 KB | daggerhart |
Comments
Comment #2
daggerhart commentedAttached is the patch for the changes to the 7.x module.
Comment #3
volkswagenchickCode looks good. Patch applied using simplytest.me . The name of the project is now in title case on the modules page.
Screenshots attaches and marking RTBC. Thanks!
Comment #4
mfbBut I so carefully made this module improper :p
Comment #5
daggerhart commentedMuch appreciation for the module and that sic project logo :D
If I had my druthers I'd mispell every instance of referrer on the internet, consistancy is important ¯\_(ツ)_/¯
Comment #7
mfbThanks for the patch. I also updated some other strings (at one point this module also allowed for configuring the referrer policy, but that functionality is now provided by Metatag module).