WordPress.org

Making WordPress.org

Opened 3 years ago

Last modified 18 months ago

#1773 assigned enhancement

Reduce Tagregator JavaScript size

Reported by: walterebert Owned by:
Milestone: Priority: low
Component: WordCamp Site & Plugins Keywords: good-first-bug has-patch
Cc:

Description

The Tagregator plugin is active by default on WordCamp sites. It currently loads an 1.5 MB JavaScript file, even if the functionality is not used. This seems excessive.

Example:
https://2016.frankfurt.wordcamp.org/wp-content/plugins/tagregator/javascript/front-end.js?ver=1

Attachments (2)

1773.diff (539 bytes) - added by xkon 22 months ago.
Load css / js only on tagregator pages.
1773.2.diff (520 bytes) - added by xkon 21 months ago.
Fixed Coding Standards.

Download all attachments as: .zip

Change History (7)

#1 @iandunn
3 years ago

  • Keywords needs-patch good-first-bug added
  • Owner set to iandunn
  • Status changed from new to accepted

Doh, I accidentally committed the development version instead of the build version :|

I fixed that in [plugins 1438841], and it now comes out as 131k gzipped, or 556k uncompressed.

It'd still be good to make sure the script is only enqueued on pages where the shortcode it used, though.

@xkon
22 months ago

Load css / js only on tagregator pages.

#2 @xkon
22 months ago

  • Keywords has-patch added; needs-patch removed

1773.diff addresses the issue. The css & js files will only be loaded if Tagregator is present in the page.

Best regards,
Konstantinos

@xkon
21 months ago

Fixed Coding Standards.

#3 @xkon
21 months ago

There was an issue with my editor so I've updated the diff ( 1773.2.diff ) to meet the coding standards.

Sorry for the inconvenience.

Best regards,
Konstantinos

#4 @iandunn
18 months ago

  • Owner iandunn deleted
  • Status changed from accepted to assigned

#5 @iandunn
18 months ago

  • Priority changed from normal to low
Note: See TracTickets for help on using tickets.