TYPO3 Core

the patch is for the file class.t3lib_tceforms.php

i used version 4.1.1 as working base

i slightly modified the patch for better rendering of the forms in safari.

I didn't look to your code in detail, but e.g. the JS function "toggleDisplayMode" doesn't need to be set in PHP and could go to a library like jsfunc.tbe_editor.js.

you are right. thats a more clean solution. i uploaded 2 new patches for jsfunc.tbe_editor and t3lib_tceforms. i sucessfully tested this version witch firefox and safari. ie should also work.

regards Martin

did anyone look at the new code is suggested (last two patches)?

regards Martin

i modified the patches because it turned out that my first solution was'nt compatble with irre. the current state works for me but needs some testing in explorer. i tested sucessfully with safari and firefox.

regards Martin

I will send an RFC to the core list.

i added some js to alt_doc_php to show the palettes immediately after checking "show secondary options" as requested. the same code could call an ajax function to save the state of the secondary options but i have no clue how to do that correctly.

see new patch ... regards Martin

Hey,

this topic just came up in a small HCI brainstorming session last night.
We also think that the second options palette as it is is painful (who suspects something in that area showing up?). In addition to the feature you provided we though of having a different icon there since the existing button is not quite meaningful.
Anyway, I quickly read over your latest patch. It looks quite good. I think it's a good start and mabye we can do a little bit more for 4.2 -- so stay tuned :-).

Hi Benjamin, Hi Martin,

nice to se things going.

@Benjamin Fischer: Can you please contact me "off-tracker" to speak about the next steps? (thomas (at) typo3-unleashed (dot) net.

Greets,
Thomas

P.S: I deleted the old patch versions in agreement with Martin.

Commited to trunk.

Benni might add some improvements in a separate patch.

Greets and thanks,
Thomas