On Wed, Jul 30, 2008 at 7:20 AM, Gertjan van Oosten <[EMAIL PROTECTED]> wrote:
> As quoted from Gertjan van Oosten <[EMAIL PROTECTED]>: > > To follow-up on my own message: the classes from r645741 seem to work, > > the classes from r647237 fail under MSIE. So the incompatible change > > seems to appear first after: > > > > 2008-04-11 17:39 lu4242 > > * [r647237] > > > core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlButtonRendererBase.java, > > > core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlLinkRendererBase.java, > > > core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlRendererUtils.java: > > fix MYFACES-1855 hidden fields created when using h:commandLink > > and f:param should be created and removed using javascript > > > > > > [All this is in myfaces/shared/trunk_3.0.x] > > Then if I re-enable the lines on trunk commented out in r647237 in > HtmlButtonRendererBase and HtmlLinkRendererBase, it works again. > > I'll attach a patch. > Hi The change done before was instead create input hidden fields rendering using html for each param on h:commandLink, use javascript for create and remove this fields (see the arguments on MYFACES-1855). I can see the problem when id is used as param name on IE (on firefox works well). But I think that a better solution is do some javascript modification instead go back to old and "source of incompatibilities" code. I'll try to solve it. regards Leonardo Uribe > > Kind regards, > -- > -- Gertjan van Oosten, [EMAIL PROTECTED], West Consulting B.V., +31 15 2191 > 600 >
