Download & Extend
Image Assist » Issues

Only load img_assist.js when necessary

Posted by casey on March 21, 2006 at 1:33pm
Project:Image Assist
Version:6.x-1.0-beta1
Component:Code
Category:task
Priority:normal
Assigned:sun
Status:closed (duplicate)

Issue Summary

Move code which loads img_assist.js and other js from img_assist_menu() to img_assist_textarea() so it only will be loaded when neccesary.

Login or register to post comments

Comments

#1

Posted by Darren Oh on November 28, 2006 at 9:11pm
Version:6.x-3.x-dev» 4.7.x-1.x-dev

#2

Posted by sun on September 7, 2007 at 11:13pm
Title:only load img_assist.js when neccesary» Only load img_assist.js when necessary
Version:4.7.x-1.x-dev» 5.x-1.4
Priority:minor» normal
Status:active» needs review

I second this, that script is quite big and completely useless for most visitors of a site.

AttachmentSize
img_assist-DRUPAL-5_0.patch 3.12 KB

#3

Posted by zoo33 on September 16, 2007 at 11:47pm

Looks very reasonable. I'd just like to get some confirmation that everything's working alright quickly before committing. I'll try to test it myself.

#4

Posted by mr.j on October 13, 2007 at 6:15am

Just subscribing

#5

Posted by sun on October 13, 2007 at 5:00pm
Category:bug report» feature request

#6

Posted by TheClue on November 16, 2007 at 9:08pm

+1. It's definitively a good feature to add!

#7

Posted by sun on December 6, 2007 at 9:39pm
Version:5.x-1.4» 5.x-1.x-dev
Category:feature request» task
Assigned to:Anonymous» sun

#8

Posted by sun on April 12, 2008 at 4:22pm
Version:5.x-1.x-dev» 6.x-1.0-beta1

Re-classifying for 6.x.

#9

Posted by zoo33 on June 29, 2008 at 8:19pm
Status:needs review» closed (duplicate)

This is also implemented by #246385: Handle teaser splitter, separate JS/CSS which I think is getting close to RTBC.

nobody click here