I've been trying out the 2.x-dev version and love it. I have just a few suggestions that I think would really make this something I would use on all my client sites:

  1. imagecache preset selection - That's probably not the right way to title it, but here's what I'm thinking would be really helpful. I'd love it if there was an option at admin/settings/imagebrowser (perhaps http://sprocketcreative/admin/settings/imagebrowser/imagecache) that allowed me to select WHICH imagecache presets are revealed to image_browser. This would be very helpful on sites with lots of presets to simplify it down for content creators/editors so there's only a few options there. It would also be really nice if you had the option to assign a human readable name for each preset.
  2. styling classes - I haven't figured out exactly how the styling thing is supposed to work in the 2.x-dev version since even if you add some css with your style it doesn't seem to be loaded on the page. That being said, I think it would be a nicer approach anyway to simply have a "class" field where a user can simply select from available classes to add, and for it to add them as an additional class on the image as opposed to a hyphened addition to an already existing class (ex. class="ibimage left" instead of class="ibimage-left").
  3. allow editing in WYSIWYG - Not sure how complicated this might be, but it would be really nice if you could click on an image and then click the image_browser WYSIWYG button and have it pop open the image_browser settings for you image again for adjustment.

Love this module. Great work!

Comments

q0rban’s picture

q0rban commented

Subscribe

jdelaune’s picture

jdelaune commented

Thanks sprocketjared.

1. This can be done via permissions, simply give a role access to the presets you want them to see and use. If this doesn't work get back to me. As for giving them more friendly names, I think that's something you would have to ask the ImageCache module for.

2. Available classes from where? You need to clear the cache to see the styles take effect. Maybe you could explain this a bit better.

3. Is coming, ran into some issues within WYSIWYG API, submitted fixes and they are looking into it. Plus I need to make lots of changes, I'm just short of time at the moment.

mwoodwar’s picture

mwoodwar commented

I'm also struggling with the style issue. After reading various issues here, I put the img.ibimage left and right in my style.css, but wasn't sure how to add the style in the admin of the image browser, and it didn't show up in the Styles box in the editor (cleared all caches)

For myriad reasons I'd prefer to add the style classes in the styles section of image browser. I tried that to no avail, is there a specific syntax that needs to be put in the box? ie

Name : float-left

CSS : do I need to include the { } braces like a regular style sheet? or just the actual rules

Thanks in advance, this is a VAST IMPROVEMENT over the other image browsers...

Mark

jdelaune’s picture

jdelaune commented

This is a screencast I created when the feature was first introduced. Interface is a little different now but it should still work the same.

http://blip.tv/file/2370282

pyrollo’s picture

pyrollo commented

I had problems with "Image on right" and "Image on left" styles in FCKEditor. They does not work at all with image inserted by Image Browser. This screencast is what I was looking for! Thanks!

Moreover, that could be very nice to be able to assign a css class to an image browser style rather that styling directions.

peter törnstrand’s picture

peter törnstrand commented
Assigned: Unassigned » peter törnstrand

I'm on it. Starnox has made some progress on number 3, will see how much work is required to get this in. A setting for available imagecache presets is a good ide. Will implement this as well.

The styling works, just empty your cache after you have created your new styles.

peter törnstrand’s picture

peter törnstrand commented

Now I see what Starnox ment in comment #2 ... the imagecache presets can be controlled via the premissions page. Just check/uncheck the presets you want available. So that will not be implemented. Remains number 3.

jdelaune’s picture

jdelaune commented

#3 is still a long way off because we are waiting for the WYSIWYG API to move forward still. I've patched the issue but it doesn't seem to a solution which they want to implement. So we may have to wait a bit longer even if I finish off the work I had started on it.

http://drupal.org/node/613944

We might be able to get something working for CKEditor however without WYSIWYG API integration supporting this.

peter törnstrand’s picture

peter törnstrand commented

Ok. I'll keep any eye on Wysiwyg module and try out the changes you made earlier Starnox and see if we can do some magic för CKEditor.

peter törnstrand’s picture

peter törnstrand commented
Status: Active » Postponed
peter törnstrand’s picture

peter törnstrand commented
Title: imagecache preset selection options & styling classes » Allow editing of existing images
jdelaune’s picture

jdelaune commented

Thanks I'm waiting to hear back from wwalc on how we can implement these features for CKeditor as there isn't any developer docs out there which I can find. Will let you know if I hear back from him.

karens’s picture

karens commented

There's a patch now on the WYSIWYG issue so it might be time to follow up and see if that works to solve the problem. Being able to edit the image is pretty critical, IMO.

echoz’s picture

echoz commented

subscribe - +1 on number 3 of original post, for wysiwyg with tinymce.

webkomplize’s picture

webkomplize commented

Hi, is there any progress for point 3 (allow editing in WYSIWYG) of the original post for wysiwyg with ckeditor?