Posted by sb56637 on July 21, 2008 at 3:37pm
| Project: | Image Assist |
| Version: | 6.x-3.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
Hello,
As I have a slow internet connection, is there any way to prevent img_assist from show "My images" as the default filter selection? It would be nice if it displayed nothing until I select the gallery that I want. Thanks!
Comments
#1
The way it's designed, that's not possible at the moment (more info here). If you want to hack your copy, you could comment out the command in
onClickStartOver()in img_assist.js. You could also match this behaviour with a default "empty" option in the select list in the header.This i a feature request I suppose – we could make a setting for this. If someone has any clever ideas and want to write a patch, go ahead!
#2
#3
Well, that's it - basically.
The major caveat is that the header frame in the popup is controlled by the main (lower) frame currently. That is (an also) wanted behavior and means, if IA is in wysiwyg mode and is invoked with an existing inline tag, then IA loads the image properties of the already embedded image into the lower frame and updates the header frame accordingly (-> "Cancel" only). Also, when browsing through images and a user clicks on one of the images, the main frame loads the image property page and updates the header frame accordingly afterwards.
So this tiny but definitely needed feature requires a major redesign of IA's popup handling.
Thus, bumping from minor to critical.
#4
Well, I read though it and it looks good. That's about all I can say right now, no time to test or in-depth reviewing...
#5
Thank you all for looking into this! Nice work.
#6
This is the number one request I get from users regarding image assist on my site - thanks for working on it.
I'm confused about the patch, however. sun, you say this "requires a major redesign of IA's popup handling", but the patch looks pretty small. Does this patch actually implement the requested functionality (that is, being able to launch the IA popup directly to upload mode rather than browse mode when inserting a new image)? So far as I can tell by reading it and attempting to merge it up to a recent img_assist release, the answer is no.
If not, what exactly is its intended purpose?
#7