Closed (fixed)
Project:
Image Picker
Version:
5.x-1.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jul 2008 at 01:52 UTC
Updated:
29 Aug 2008 at 09:49 UTC
It would be great if you could indicate a class when inserting an image. It would save some repetitive work. This would be even more useful if there was a default class in the configuration options.
I use this a great deal to insert pics into news items. I use CSS to provide float, margins and a uniform size.
Comments
Comment #1
hutch commentedTry the development version, you can set classes instead of the default float:left or float:right.
You also get a 'My imagepicker' tab in My account, and there is support for lightbox2 as well as paged browsing
Comment #2
eyecon-1 commentedOK but there is not enough room in the text box (maxlength="50"). An average string, allowing for float, margins and size is about 72 characters. Also, it might be cleaner and DRYer to provide a default CSS class. This also provides greater flexibility for mass changes which then only require one edit to the style sheet. Maybe worth thinking about . . .
Comment #3
hutch commentedWhat I put into mine is
and put all the styling for those classes in style.css, better than storing styling info in the variable table.
I can't really add these two classes to imagepicker.css as this is only enabled when an imagepicker 'window' is on the page and is not loaded otherwise as it's not needed.
Perhaps a note in the readme.....
Comment #4
eyecon-1 commentedYup that works. big "duh" on my part. Thank you for thawing my brain freeze.
Comment #5
hutch commentedYou're not the first, a note in the readme is a must ;-)
Comment #6
hutch commented