some patches for EFM (Trac #852)

[sleemanj]

Hi!

1. If you dont't specify any margin, padding, or border the respective styles are inserted anyway with a value of 0px, which, first, blows up the code unnecessaryly and, second, may interfere with existing stylesheets

2. Like the two new config variables for alt and title, I'd like to introduce even more options to grant the administrator more control over what a user can change

• $IMConfig['images_enable_margin'] = true;
• $IMConfig['images_enable_padding'] = true;
• $IMConfig['images_enable_align'] = true;
• $IMConfig['images_enable_border'] = true;
• $IMConfig['images_enable_background_color'] = true;
• $IMConfig['link_enable_target'] = true;

I think especially the last one is important regarding XHTML strict

3. some UI tweaks
   • show max upload size (imo very important to let the user not run into the limit unknowingly)
   • display the mode in the title bar of the popup
   • made a new "constrain proportions unlocked" icon (ok, that a matter of taste :))

looks like this: http://raimundmeyer.de/xinha/efm_screenshot.png

What do you think, shall I commit?

Reported by ray, migrated from http://trac.xinha.org/ticket/852

[sleemanj]

koto commented:

Of course (these are great features, thanks!), just one note: make sure that you update the col/rowspans for the table when changing the number of rows to hide/show appropriate fields in the popup. The code is quite trick about that :(

[sleemanj]

• ray commented:

Yes, quite tricky indeed. Cut it down to $IMConfig['images_enable_styling'] to toggle them altogether. That's what I was about anyway... Next is, I'd like to change alignment to css, because, you know, deprecated etc. :)

• ray changed resolution to fixed
• ray changed status from new to closed

[sleemanj]

• koto commented:

Commited a small color-pickers related bugfix - see changeset:576

• koto changed component from Documentation to Plugin_ImageManager