This issue is for efforts around porting this theme to 5.x. Patch forthcoming.

CommentFileSizeAuthor
#1 internet_services_5x.patch3.28 KBwebchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Active » Needs review
FileSize
internet_services_5x.patch3.28 KB

This solves the immediate issues; namely the fatal call to module_exist, and the raw HTML being printed instead of menu items. There may be some other stuff, too.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Btw, I removed internet_services_primary because:

a) the format of links has changed in 5.x.
b) 5.x is correctly picking up active links as active, which looks like all that snippet was intended to solve.

hswong3i’s picture

hswong3i CreditAttribution: hswong3i commented

BTW, i have release the 5.0 implementation. so this thread will now close. please report for any bug in new version

hswong3i’s picture

hswong3i CreditAttribution: hswong3i commented
Status: Needs review » Closed (fixed)

as have release the 5.0 implementation, this thread will now close. please report for any bug in new version

hswong3i’s picture

hswong3i CreditAttribution: hswong3i commented

as i have release the 5.0 implementation, this thread will now close. please report for any bug in new version