When it comes to inserting an image, we need a way to assign a style. Perhaps a simple text field under the 'insert title and description' option that will allow us to use a custom CSS class for image styling.

Comments

hutch’s picture

hutch commented
Version: 6.x-2.1 » 6.x-2.x-dev
Status: Active » Needs review

Simple enough, I have made it an option which can be enabled under the advanced features on the configuration
Added to CVS, will appear in the 2.x dev version later today.
Please test and let me know ;-)

rewted’s picture

rewted commented

Will do. *Awaits dev version commit*

hutch’s picture

hutch commented

there now

rewted’s picture

rewted commented

It doesn't add the css class properly. Seems to just insert the class name you give it, but no class=" ".

hutch’s picture

hutch commented

Oh, it did for me, would you be using a wysiwyg editor?
I notice that non 'Full HTML' strips style="......" but it should not strip class="...."

justed tested it it a plain body box in a zen theme, here is what it pasted

<img src="/drupal6-2/sites/default/files/imagepicker/1/thumbs/ATT00014.jpg" alt="catnmouse"  class="thingy" />

So I don't know what is stripping it, I tried it with single quotes too and that pasted OK too.

rewted’s picture

rewted commented

My node was full html too. Noticed mixed results when using the align left defaults in conjunction with this new custom class too.

hutch’s picture

hutch commented
Status: Needs review » Reviewed & tested by the community

Please remember that imagepicker has *no* control over how a node renders, it only supplies the means to paste a piece of html into the node body, what happens to it after that is completely beyond the control of this module.
As far as I can see this (optional) feature does what it says, it adds css to the image tag. If the result is different than you expected than that is an issue with the filtering or with css, not imagepicker.

hutch’s picture

hutch commented
Status: Reviewed & tested by the community » Closed (fixed)