While evaluating this module we noticed it could use a hook_help implementation to make it easier for users to find helpful information about the module. Patch to follow. :)

CommentFileSizeAuthor
#3 patch - help page after.png18.86 KBvolkswagenchick
#3 patch after.png17.38 KBvolkswagenchick
#2 colorbox_inline_hook_help-2966894-2.patch987 bytesdani3lr0se
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

daniel_rose created an issue. See original summary.

dani3lr0se’s picture

dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commented
Assigned: dani3lr0se » Unassigned
Status: Active » Needs review
FileSize
colorbox_inline_hook_help-2966894-2.patch987 bytes

Here is a patch that adds a hook_help function so users have a place to go for a little extra help. Thanks for your work on this module. :)

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
CreditAttribution: volkswagenchick at Hook 42 commented
Status: Needs review » Reviewed & tested by the community
FileSize
patch after.png17.38 KB
patch - help page after.png18.86 KB

Thanks for extending documentation. Now when the module is enabled a hook help implemented and there is a link to the project page.

Screenshoot attached, marking RTBC.

  • daniel_rose authored 1ffca48 on 8.x-1.x 
    Issue #2966894 by daniel_rose, volkswagenchick: Missing hook_help...
NickDickinsonWilde’s picture

NickDickinsonWilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
CreditAttribution: NickDickinsonWilde at BriarMoon Design commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch!

Status: Fixed » Closed (fixed)

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