Problem/Motivation

Special Menu Items 7.x-2.x can cause a significant performance impact on link-heavy pages.

Proposed resolution

Document the performance implications in the README.txt and on the project homepage.

CommentFileSizeAuthor
#2 special_menu_items-document_theme_link_performance-2644768-1.patch960 bytestedfordgif
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tedfordgif created an issue. See original summary.

tedfordgif’s picture

tedfordgif CreditAttribution: tedfordgif at WebFirst, Inc. commented
FileSize
special_menu_items-document_theme_link_performance-2644768-1.patch960 bytes

Patch with sample text for README.txt.

tedfordgif’s picture

tedfordgif CreditAttribution: tedfordgif at WebFirst, Inc. commented
Status: Active » Needs review
gagarine’s picture

gagarine CreditAttribution: gagarine as a volunteer commented
Status: Needs review » Patch (to be ported)

Thanks

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Patch (to be ported) » Reviewed & tested by the community

FYI the "patch (to be ported)" status is for when a patch exists for one branch and it needs to be ported to another branch. I think RTBC is the status you're looking for.

  • gagarine committed 6456ede on 7.x-2.x 
    Fix #2644768 by tedfordgif: Document performance issue with implementing...
gagarine’s picture

gagarine CreditAttribution: gagarine as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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