[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Ilya Shaikovsky commented on RF-8608: ------------------------------------- <div id="contentDiv"> ..... div content ...... </div> <input type="button" onclick="hideDiv({duration:0.7})" value="Hide" /> <input type="button" onclick="showDiv()" value="Show" /> <rich:effect name="hideDiv" for="contentDiv" type="Fade" /> <rich:effect name="showDiv" for="contentDiv" type="Appear" /> used for testing. Works fine at FF 3.6, IE8 and Opera. Not works in Chrome but it's duplicate of https://jira.jboss.org/jira/browse/RF-8088 So will close as can't reproduce. Please rise forum thread as you seems have some configuration problems. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Tomas Cerny reopened RF-8608: ----------------------------- -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Tomas Cerny updated RF-8608: ---------------------------- code generated on the new version 3.3.3: (broken) <script type="text/javascript">Richfaces.effect.create({'name':'showOpenDiv','params':{},'type':'BlindDown','attachId':'register:openDiv'} )</script> <script type="text/javascript">Richfaces.effect.create({'name':'hideOpenLink','params':{duration:0.0,from:0.5,to:0},'type':'Fade','attachId':'register:openLink'} )</script> <script type="text/javascript">Richfaces.effect.create({'name':'showOpenLink','params':{delay:0.8,duration:0.9,from:0.5,to:1.0},'type':'Appear','attachId':'register:openLink'} )</script> <script type="text/javascript">Richfaces.effect.create({'name':'hideOpenDiv','params':{},'type':'Squish','attachId':'register:openDiv'} )</script> code generated on the new version 3.3.1: (works) <script type="text/javascript">//<![CDATA[ showOpenDiv = function () { return Richfaces.processEffect(Object.extend(Object.extend( Object.extend({type:'BlindDown'},{targetId:'register:openDiv',attachId:'register:openDiv'}), Object.extend ({},{})), arguments[0]||{})); }; //]]></script> <script type="text/javascript">//<![CDATA[ hideOpenLink = function () { return Richfaces.processEffect(Object.extend(Object.extend( Object.extend({type:'Fade'},{targetId:'register:openLink',attachId:'register:openLink'}), Object.extend ({},{duration:0.0,from:0.5,to:0})), arguments[0]||{})); }; //]]></script> <script type="text/javascript">//<![CDATA[ showOpenLink = function () { return Richfaces.processEffect(Object.extend(Object.extend( Object.extend({type:'Appear'},{targetId:'register:openLink',attachId:'register:openLink'}), Object.extend ({},{delay:0.8,duration:0.9,from:0.5,to:1.0})), arguments[0]||{})); }; //]]></script> <script type="text/javascript">//<![CDATA[ hideOpenDiv = function () { return Richfaces.processEffect(Object.extend(Object.extend( Object.extend({type:'Squish'},{targetId:'register:openDiv',attachId:'register:openDiv'}), Object.extend ({},{})), arguments[0]||{})); }; //]]></script> Hope I am more convincing now :) -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Nick Belaevski updated RF-8608: ------------------------------- Component/s: component-misc regression Priority: Critical (was: Blocker) -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Tomas Cerny commented on RF-8608: --------------------------------- Hi Nick, I have been looking at your suggestion and got down to the attached fragment. It seems that rich:panelMenuGroup causes the issue. If I comment out the code with rich:panelMenuGroup in my posted file it would work, but I guess I will need to use both components on one page :).. Let me know if you need some more feedback. Tom -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/jira/browse/RF-8608?page=com.atlassian.jira.plugin... ] Nick Belaevski commented on RF-8608: ------------------------------------ Introduced by http://fisheye.jboss.org/viewrep/RichFaces/branches/community/3.3.X/ui/ef... in processEffect.js file. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/RF-8608?page=com.atlassian.jira.plugin.sy... ] Jay Balunas updated RF-8608: ---------------------------- Priority: Major (was: Critical) -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/RF-8608?page=com.atlassian.jira.plugin.sy... ] Tomas Cerny edited comment on RF-8608 at 11/14/11 7:44 PM: ----------------------------------------------------------- Here is a workaround: go to file ui.pack.js and replace variable "params" for "myparams" in the part related to processEffect.js ---- if(!Richfaces.effect){Richfaces.effect={} }Richfaces.effect.create=function(options){if(!options){options={} }var myparams=options.params||{}; options.params=null; var attachObj; var targetObj; var targetId=options.targetId; var attachId=options.attachId; if(options.attachObj){try{attachObj=eval(options.attachObj) }catch(e){}if(typeof attachObj=="object"){attachId=attachObj }}if(options.targetObj){try{targetObj=eval(options.targetObj) }catch(e){}if(typeof targetObj=="object"){targetId=targetObj }}if(!targetId){targetId=attachId }if(!myparams.targetId){myparams.targetId=targetId }myparams.type=options.type; if(!options.event){with(window){eval(options.name+"=function(){return Richfaces.processEffect(Object.extend(this,arguments[0]||{}));}.bind(myparams)") }}else{var ename=Richfaces.effectEventOnOut(options.event||""); if(ename){var bindedFunction=function(event){return Richfaces.processEffect(this) }.bindAsEventListener(myparams); Event.observe(attachId,ename,bindedFunction,myparams.useCapture||false) }}} ; ----- if you do not use bulk js. then do the same in the file processEffect.js was (Author: enda): Here is a workaround: go to file ui.pack.js and replace variable "params" for "myparams" ---- if(!Richfaces.effect){Richfaces.effect={} }Richfaces.effect.create=function(options){if(!options){options={} }var myparams=options.params||{}; options.params=null; var attachObj; var targetObj; var targetId=options.targetId; var attachId=options.attachId; if(options.attachObj){try{attachObj=eval(options.attachObj) }catch(e){}if(typeof attachObj=="object"){attachId=attachObj }}if(options.targetObj){try{targetObj=eval(options.targetObj) }catch(e){}if(typeof targetObj=="object"){targetId=targetObj }}if(!targetId){targetId=attachId }if(!myparams.targetId){myparams.targetId=targetId }myparams.type=options.type; if(!options.event){with(window){eval(options.name+"=function(){return Richfaces.processEffect(Object.extend(this,arguments[0]||{}));}.bind(myparams)") }}else{var ename=Richfaces.effectEventOnOut(options.event||""); if(ename){var bindedFunction=function(event){return Richfaces.processEffect(this) }.bindAsEventListener(myparams); Event.observe(attachId,ename,bindedFunction,myparams.useCapture||false) }}} ; ----- if you do not use bulk js. then do the same in the file processEffect.js -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/RF-8608?page=com.atlassian.jira.plugin.sy... ] Marek Będkowski edited comment on RF-8608 at 3/8/12 4:17 PM: ------------------------------------------------------------- If you experience this problem and still using richfaces 3.3.x - please ignore comment above by Thomas Cerny, swapping local variable names seemed to make no difference in my environment. To make it quick - just apply the supplied patch (processEffect.js.patch) to the formentioned processEffect.js file - I got rid of 'with' and 'eval' evils and seems to work :) Enjoy! was (Author: bendi): If you experience this problem and still using richfaces 3.3.x - please ignore comment above by Thomas Cerny, it doesn't change anything (btw I've no idea how it should since it's just swapping variable name in the local context). To make it quick - just apply the supplied patch (processEffect.js.patch) to the formentioned processEffect.js file - current solution there uses two of the top 10 JS worst practices: with and eval :) Enjoy! -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/RF-8608?page=com.atlassian.jira.plugin.sy... ] Marek Będkowski edited comment on RF-8608 at 3/8/12 4:19 PM: ------------------------------------------------------------- Hi Since the problem still exists in 3.3.3.Final I decided to prepare a patch (processEffect.js.patch) to the formentioned processEffect.js file - I got rid of 'with' and 'eval' evils and seems to do the trick :) @Thomas - I tried swapping variable name from params to myparams but it made no difference. was (Author: bendi): If you experience this problem and still using richfaces 3.3.x - please ignore comment above by Thomas Cerny, swapping local variable names seemed to make no difference in my environment. To make it quick - just apply the supplied patch (processEffect.js.patch) to the formentioned processEffect.js file - I got rid of 'with' and 'eval' evils and seems to work :) Enjoy! -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira