Needs review
Project:
Image Assist
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
21 Jul 2008 at 15:37 UTC
Updated:
9 Jan 2011 at 08:45 UTC
Jump to comment: Most recent file
Comments
Comment #1
zoo33 commentedThe 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!
Comment #2
sunComment #3
sunWell, 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.
Comment #4
zoo33 commentedWell, 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...
Comment #5
rahim123 commentedThank you all for looking into this! Nice work.
Comment #6
benwei commentedThis 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?
Comment #7
sun