[ 
https://issues.apache.org/jira/browse/TOMAHAWK-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Leonardo Uribe resolved TOMAHAWK-1448.
--------------------------------------

       Resolution: Fixed
    Fix Version/s: 1.1.10-SNAPSHOT
         Assignee: Leonardo Uribe

Will be fixed on TOMAHAWK-1498

> InputHtmlRenderer: Missing xmlns:i18n declarations in various HTML elements
> ---------------------------------------------------------------------------
>
>                 Key: TOMAHAWK-1448
>                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1448
>             Project: MyFaces Tomahawk
>          Issue Type: Bug
>          Components: Html Editor
>    Affects Versions: 1.1.9
>            Reporter: Lutz Ulruch
>            Assignee: Leonardo Uribe
>            Priority: Minor
>             Fix For: 1.1.10-SNAPSHOT
>
>
> In encodeEndNormalMode() various HTML elements are written with attributes 
> prefixed with "i18n".
> Not all those HTML elements have a namespace definition.
> That is:
> writer.writeAttribute("xmlns:i18n", "http://xml.zope.org/namespaces/i18n";, 
> null);
> is missing for various elements (whereas for most elements the namespace 
> attribute is set already).
> This results in invalid XHTML. This in turn results in errors with DOM 
> parsing throug browser-side scripts, like with RichFaces' PPR.
> Please, just add the missing xmlns:i18n attributes.
> Suggestion: Put the xmlns attributes in parent HTML elements instead of 
> reproducing them all the child elements. This would save HTML code.
> For example:
> <div xmlns:i18n="http://xml.zope.org/namespaces/i18n"; id="toolbar" 
> class="kupu-tb">
> ...
>   <button i18n:attributes="title"/> // no need to repeat xmlns-attribute 
> because it is now in the parent div
>   
> There are just 2 div elements which need the xmlns:i18n attribute:
> <div id="toolbar" > and <div id="kupu-toolboxes">

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to