Problem/Motivation

  • Make configurable & flexible
  • Drupal 9 support

Proposed resolution

Attached Zip contains Drupal 9 & settings form (/admin/config/user-interface/regionincontent) for region in templates

CommentFileSizeAuthor
#5 regionincontent-3204260-05.patch5.28 KBthirstysix
#3 RegioninContentTemplatesSettings.png13.36 KBthirstysix
regionincontent.zip10.25 KBthirstysix

Comments

ThirstySix created an issue. See original summary.

thirstysix’s picture

thirstysix commented
Status: Active » Needs review
thirstysix’s picture

thirstysix commented
StatusFileSize
new RegioninContentTemplatesSettings.png13.36 KB
thirstysix’s picture

thirstysix commented
Issue summary: View changes
thirstysix’s picture

thirstysix commented
StatusFileSize
new regionincontent-3204260-05.patch5.28 KB
mlncn’s picture

mlncn commented
Assigned: Unassigned » mlncn

Wow, thank you!

Apologies for missing this contribution.

  • mlncn committed c21fc82 on 8.x-1.x authored by ThirstySix 
    Issue #3204260 by ThirstySix, mlncn: Make configurable, flexible...
mlncn’s picture

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

This is good! Made a couple edits to the patch when committing it:

- removed the Drupal.org-added lines from the .info part of the patch, those are not meant to be in the repository
- restored the function to have the module name in it— for functional programming (as opposed to object-oriented in classes) all modules in Drupal must keep their functions namespaced to avoid collisions with function names in other modules.

mlncn’s picture

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

Thanks again!

Status: Fixed » Closed (fixed)

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