While evaluating this module I noticed that on the Extend page, the project name is in lower case and should be in title case to be consistent with the rest of the project names on the page.
"Capitalize the name of the module, because module names are proper nouns."
https://www.drupal.org/docs/develop/documenting-your-project/help-text-s...
Patch to follow, thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | Screen Shot 2018-09-17 at 11.06.39 AM.png | 23.81 KB | volkswagenchick |
| #4 | media_entity_twitter-info-2999187-4.patch | 383 bytes | alonaoneill |
| #3 | Capitalize module_Media entity Twitter.png | 16.09 KB | akash.addweb |
| #2 | media_entity_twitter-capitalize-2999187-2.patch | 384 bytes | volkswagenchick |
Comments
Comment #2
volkswagenchickI uploaded a patch that capitalizes the name of the modules on the extend page as per Drupal documentation standards.
Thanks
Comment #3
akash.addweb commented@volkswagenchick, Thanks for the patch, your patch works properly for me only get 1 typo in it. Can you please change "Media eEtity Twitter" to "Media entity Twitter" PFA
Comment #4
alonaoneill commentedI uploaded a patch that corrects typo in "Media Entity Twitter".
Thanks!
Comment #5
volkswagenchickOoops. Thanks @alonaoneill for fixing this.
Patch Applied vis simplytest.me and the project name is now capitalized on the Extend page. Screenshot attached.
Marking RTBC
Comment #6
chr.fritsch