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
#2 image_field_tokens_d8-info-file-3004344-2.patch384 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 image_field_tokens_d8-info-file-3004344-2.patch384 bytes

I uploaded a patch the capitalizes the name of the module on the Extend page. Thanks!

jqtnguyen’s picture

jqtnguyen commented
Status: Needs review » Reviewed & tested by the community

Changes look good in the UI, name capitalization aligns with Drupal Standards, and patch applied without errors using Simplytest.Me. Marking RTBC.

ysamoylenko’s picture

ysamoylenko commented

Thank you for the report and reviewing, the patch will be added to the next module release.

ysamoylenko’s picture

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

The patch was commitеed. Thank you.

Status: Fixed » Closed (fixed)

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