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!

CommentFileSizeAuthor
#3 private-content.png10.87 KBdani3lr0se
#2 private_content-capitalize-name-2989992-2.patch347 bytesvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new private_content-capitalize-name-2989992-2.patch347 bytes

I uploaded a patch that capitalizes the name of the module in the .info.yml file to align with Drupal Code Standards.
Thanks!

dani3lr0se’s picture

dani3lr0se commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new private-content.png10.87 KB

Patch applies successfully locally. I now see the whole title capitalized. See attached screenshot. Thanks for the patch @volkswagenchick.

adamps’s picture

adamps commented
Priority: Normal » Minor

Thanks, it's minor but valid, I will check this in at some stage.

adamps’s picture

adamps commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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