Hi Andreas,

 > Hi Rainer,
 > 
 > Rainer Schöpf schrieb:
 > >  > Thanks Rainer, now it's working. I've applied the patch to
 > > */tiny_config.js/*
 > >  >  > Now it doesn't throw any exception when I add border, vspace,hspace
 > >  > attributes. But these attributes are not getting reflected in the image
 > > tag
 > >  > (after saving).
 > >  >  > Should they or not as these are deprecated?
 > > 
 > > These attributes deprecated in HTML 4.01, and not allowed in XHTML.
 > > 
 > > By default, lenya uses the XHTML basic schema (see
 > > 
 > >   http://www.w3.org/2007/09/dtd-comparison.html
 > > 
 > > for a comparison to other XHTML schemas.) The problem with the default
 > > setup for TinyMCE is that is can produce XHTML tags and attributes that are
 > > not allowed in XHTML basic. With the changes, TinyMCE removes all these
 > > invalid elements and attributes.
 > 
 > I have to admit that I didn't test the patch yet, but will do so now. Will
 > the user be aware that these elements aren't supported, or are they just
 > silently ignored? I'm fine with both cases :)

Mostly the latter: unless the option "cleanup" is set to false (in 
tiny_config.js), all invalid elements and attributes are removed when you click 
on the save button.  The popup windows have a fixed list of editable 
attributes, 
independent of the list of allowed ones.

For example, the "Insert/edit image" popup is defined in

  modules/tinymce/resources/jscripts/themes/advanced/image.htm

It might be worthwhile to create a special theme for lenya, for example by 
copying the themes/advanced directory and changing all .htm files as necessary.

For TinyMCE 3.x, perhaps?

 > Did any of the other TinyMCE users test the patch?

I've just come across another small problem when your current language is not 
english: when you click on the TinyMCE Help button (last one in the second 
row), 
then select the help tab in the middle, you get an error: It tries to load a 
language specific help text, but only english is available.  A simple 
workaround: set

  docs_language : "en"

in tiny_config.js. 

  Rainer 
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to