Somone have tested the last scriptaculous version that ships with last prototype? I you make an Effect.Pulsate, the element stays hidden after the effect finish if the element don''t have opacity stablished. This is for the changes on the setStyle method on prototype. The original code is: --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- setStyle: function(element, style) { element = $(element); for (var name in style) { var value = style[name]; if(name == ''opacity'') { if (value == 1) { value = (/Gecko/.test(navigator.userAgent) && !/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ? 0.999999: 1.0; if(/MSIE/.test(navigator.userAgent) && !window.opera) element.style.filter = element.getStyle (''filter'').replace(/alpha\([^\)]*\)/gi,''''); } else { if(value < 0.00001) value = 0; if(/MSIE/.test(navigator.userAgent) && !window.opera) element.style.filter element.getStyle(''filter'').replace(/alpha\([^\)]*\)/gi,'''') + ''alpha(opacity=''+value*100+'')''; } } else if(name == ''float'') name = (typeof element.style.styleFloat !''undefined'') ? ''styleFloat'' : ''cssFloat''; element.style[name.camelize()] = value; } return element; }, --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- Maybe this change on line 6, solve the problem: original --> if (value == 1) { new (?) --> if (value == 1 || value == '''') { Hope somebody says something about this. Thanks. --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Spinoffs" group. To post to this group, send email to rubyonrails-spinoffs-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-spinoffs-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-spinoffs?hl=en -~----------~----~----~----~------~----~------~--~---
Javier Martinez ha scritto:> Somone have tested the last scriptaculous version that ships with last > prototype? > I you make an Effect.Pulsate, the element stays hidden after the > effect finish if the element don''t have opacity stablished. This is > for the changes on the setStyle method on prototype. > The original code is: > --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > > setStyle: function(element, style) { > element = $(element); > for (var name in style) { > var value = style[name]; > if(name == ''opacity'') { > if (value == 1) {[cuts]> --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > > Maybe this change on line 6, solve the problem: > > original --> if (value == 1) { > new (?) --> if (value == 1 || value == '''') { > > Hope somebody says something about this. > > Thanks.Hi Javier, this seems to fix the problem in script.aculo.us functional tests too, running with FF 2.0 or IE7. Alessandro --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Spinoffs" group. To post to this group, send email to rubyonrails-spinoffs-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-spinoffs-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-spinoffs?hl=en -~----------~----~----~----~------~----~------~--~---
It''s not completely correct, because if you want to change the css to 0, it''s evaluated as ''''. Then, maybe a better fix will be: ----------------------------------------------- if (value == 1 || value === '''') { ----------------------------------------------- Hope helps someone. 2006/12/23, Alessandro Curci <a.curci-cMKc6YDe3ZConA0d6jMUrA@public.gmane.org>:> > > Javier Martinez ha scritto: > > Somone have tested the last scriptaculous version that ships with last > > prototype? > > I you make an Effect.Pulsate, the element stays hidden after the > > effect finish if the element don''t have opacity stablished. This is > > for the changes on the setStyle method on prototype. > > The original code is: > > > --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > > > > setStyle: function(element, style) { > > element = $(element); > > for (var name in style) { > > var value = style[name]; > > if(name == ''opacity'') { > > if (value == 1) { > [cuts] > > > --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > > > > Maybe this change on line 6, solve the problem: > > > > original --> if (value == 1) { > > new (?) --> if (value == 1 || value == '''') { > > > > Hope somebody says something about this. > > > > Thanks. > Hi Javier, > this seems to fix the problem in script.aculo.us functional tests too, > running with FF 2.0 or IE7. > > Alessandro > > > >--~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Spinoffs" group. To post to this group, send email to rubyonrails-spinoffs-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org To unsubscribe from this group, send email to rubyonrails-spinoffs-unsubscribe-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org For more options, visit this group at http://groups.google.com/group/rubyonrails-spinoffs?hl=en -~----------~----~----~----~------~----~------~--~---