Effect.Appear Safari issue

I've been putting this off for several days now, waiting to
check
dev.rubyonrails.org before I posted here but it is still
down.

I am using Effect.Appear and it is working great except in
Safari
where it fades in and then disappears. I traced it back to
the
Element.setOpacity function. Is there a reason why Safari is
singled
out and given a null value for opacity instead of 1 or
0.999999? This
is where the problem is happening and can be seen directly
from the
demo page itself
(http://wiki.script.aculo.us/scriptaculous/s
how/CombinationEffectsDemo).
I simply changed this line:

Element.setStyle(element, { opacity:
(/Gecko/.test(navigator.userAgent) &&
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
0.999999 : null });

to this:

Element.setStyle(element, { opacity: 0.999999 });

And this working perfectly for me in Safari 2 and 1.3 and it
didn't
hurt any of the other supported browsers. I don't have 1.2
to test it
in.

However, I believe that Element.setStyle should have the
responsiblity
of normalizing special cases like opacity. More details
here:
http://www.brandonaaron.net/articles/200
6/08/12/effect-appear-safari-fix

Brandon
_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

Hi Brandon,

I,ve had the same issue with opera.
I changed the line like this without any problems so far:

Element.setStyle(element, { opacity:
(/Gecko/.test(navigator.userAgent) &&
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
0.999999 : null });

to

Element.setStyle(element, { opacity:
(/Gecko/.test(navigator.userAgent) &&
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
0.999999 : 1 });

Perhaps this works for Safari too.

Lorenz

On 16.08.2006, at 15:40, Brandon Aaron wrote:

> I've been putting this off for several days now,
waiting to check
> dev.rubyonrails.org before I posted here but it is
still down.
>
> I am using Effect.Appear and it is working great except
in Safari
> where it fades in and then disappears. I traced it back
to the
> Element.setOpacity function. Is there a reason why
Safari is singled
> out and given a null value for opacity instead of 1 or
0.999999? This
> is where the problem is happening and can be seen
directly from the
> demo page itself
> (http:
//wiki.script.aculo.us/scriptaculous/show/ 
> CombinationEffectsDemo).
> I simply changed this line:
>
> Element.setStyle(element, { opacity:
> (/Gecko/.test(navigator.userAgent) &&
> !/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> 0.999999 : null });
>
> to this:
>
> Element.setStyle(element, { opacity: 0.999999 });
>
> And this working perfectly for me in Safari 2 and 1.3
and it didn't
> hurt any of the other supported browsers. I don't have
1.2 to test it
> in.
>
> However, I believe that Element.setStyle should have
the responsiblity
> of normalizing special cases like opacity. More details
here:
> http://www.brandonaaron.net/articles/2006/08/12/ef
fect-appear- 
> safari-fix
>
> Brandon
> _______________________________________________
> Rails-spinoffs mailing list
> Rails-spinoffslists.rubyonrails.org
> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

I think we should make a test case page for this, that has
all sorts  
of opacity enabled/disabled elements
and see if everything works fine. Any volunteers? 

-Thomas

Am 18.08.2006 um 09:42 schrieb Lorenz Knies:

> Hi Brandon,
>
> I,ve had the same issue with opera.
> I changed the line like this without any problems so
far:
>
> Element.setStyle(element, { opacity:
> (/Gecko/.test(navigator.userAgent) &&
> !/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> 0.999999 : null });
>
> to
>
> Element.setStyle(element, { opacity:
> (/Gecko/.test(navigator.userAgent) &&
> !/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> 0.999999 : 1 });
>
> Perhaps this works for Safari too.
>
> Lorenz
>
> On 16.08.2006, at 15:40, Brandon Aaron wrote:
>
>> I've been putting this off for several days now,
waiting to check
>> dev.rubyonrails.org before I posted here but it is
still down.
>>
>> I am using Effect.Appear and it is working great
except in Safari
>> where it fades in and then disappears. I traced it
back to the
>> Element.setOpacity function. Is there a reason why
Safari is singled
>> out and given a null value for opacity instead of 1
or 0.999999? This
>> is where the problem is happening and can be seen
directly from the
>> demo page itself
>> (http:
//wiki.script.aculo.us/scriptaculous/show/ 
>> CombinationEffectsDemo).
>> I simply changed this line:
>>
>> Element.setStyle(element, { opacity:
>> (/Gecko/.test(navigator.userAgent) &&
>>
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> 0.999999 : null });
>>
>> to this:
>>
>> Element.setStyle(element, { opacity: 0.999999 });
>>
>> And this working perfectly for me in Safari 2 and
1.3 and it didn't
>> hurt any of the other supported browsers. I don't
have 1.2 to test it
>> in.
>>
>> However, I believe that Element.setStyle should
have the  
>> responsiblity
>> of normalizing special cases like opacity. More
details here:
>> http://www.brandonaaron.net/articles/2006/08/12/ef
fect-appear- 
>> safari-fix
>>
>> Brandon
>> _______________________________________________
>> Rails-spinoffs mailing list
>> Rails-spinoffslists.rubyonrails.org
>> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>
> _______________________________________________
> Rails-spinoffs mailing list
> Rails-spinoffslists.rubyonrails.org
> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

--
Thomas Fuchs
wollzelle

http://www.wollzelle.com

questentier on AIM
madrobby on irc.freenode.net

http://www.fluxiom.com
:: online digital asset management
http://script.aculo.us
:: Web 2.0 JavaScript
http://mir.aculo.us ::
Where no web developer has gone before




_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

Sure I'll create some unit tests.

Brandon

On 8/18/06, Thomas Fuchs <t.fuchswollzelle.com> wrote:
> I think we should make a test case page for this, that
has all sorts
> of opacity enabled/disabled elements
> and see if everything works fine. Any volunteers? 
>
> -Thomas
>
> Am 18.08.2006 um 09:42 schrieb Lorenz Knies:
>
> > Hi Brandon,
> >
> > I,ve had the same issue with opera.
> > I changed the line like this without any problems
so far:
> >
> > Element.setStyle(element, { opacity:
> > (/Gecko/.test(navigator.userAgent) &&
> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> > 0.999999 : null });
> >
> > to
> >
> > Element.setStyle(element, { opacity:
> > (/Gecko/.test(navigator.userAgent) &&
> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> > 0.999999 : 1 });
> >
> > Perhaps this works for Safari too.
> >
> > Lorenz
> >
> > On 16.08.2006, at 15:40, Brandon Aaron wrote:
> >
> >> I've been putting this off for several days
now, waiting to check
> >> dev.rubyonrails.org before I posted here but
it is still down.
> >>
> >> I am using Effect.Appear and it is working
great except in Safari
> >> where it fades in and then disappears. I
traced it back to the
> >> Element.setOpacity function. Is there a reason
why Safari is singled
> >> out and given a null value for opacity instead
of 1 or 0.999999? This
> >> is where the problem is happening and can be
seen directly from the
> >> demo page itself
> >> (http:
//wiki.script.aculo.us/scriptaculous/show/
> >> CombinationEffectsDemo).
> >> I simply changed this line:
> >>
> >> Element.setStyle(element, { opacity:
> >> (/Gecko/.test(navigator.userAgent) &&
> >>
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
> >> 0.999999 : null });
> >>
> >> to this:
> >>
> >> Element.setStyle(element, { opacity: 0.999999
});
> >>
> >> And this working perfectly for me in Safari 2
and 1.3 and it didn't
> >> hurt any of the other supported browsers. I
don't have 1.2 to test it
> >> in.
> >>
> >> However, I believe that Element.setStyle
should have the
> >> responsiblity
> >> of normalizing special cases like opacity.
More details here:
> >> http://www.brandonaaron.net/articles/2006/08/12/ef
fect-appear-
> >> safari-fix
> >>
> >> Brandon
> >>
_______________________________________________
> >> Rails-spinoffs mailing list
> >> Rails-spinoffslists.rubyonrails.org
> >> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
> >
> > _______________________________________________
> > Rails-spinoffs mailing list
> > Rails-spinoffslists.rubyonrails.org
> > http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>
> --
> Thomas Fuchs
> wollzelle
>
> http://www.wollzelle.com
>
> questentier on AIM
> madrobby on irc.freenode.net
>
> http://www.fluxiom.com ::
online digital asset management
> http://script.aculo.us ::
Web 2.0 JavaScript
> http://mir.aculo.us
:: Where no web developer has gone before
>
>
>
>
> _______________________________________________
> Rails-spinoffs mailing list
> Rails-spinoffslists.rubyonrails.org
> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>
_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

I've done some further testing.
It is a different problem than that with Opera.

The problem is related to the style 'float: left;' on the
div that  
appears.

But since Opera 9, the setOpacity function in effects.js
isn't  
correct anymore.
Opera simply does nothing, when setting the style.opacity
property to  
null.

Lorenz


On 18.08.2006, at 15:08, Brandon Aaron wrote:

> Sure I'll create some unit tests.
>
> Brandon
>
> On 8/18/06, Thomas Fuchs <t.fuchswollzelle.com> wrote:
>> I think we should make a test case page for this,
that has all sorts
>> of opacity enabled/disabled elements
>> and see if everything works fine. Any volunteers?

>>
>> -Thomas
>>
>> Am 18.08.2006 um 09:42 schrieb Lorenz Knies:
>>
>> > Hi Brandon,
>> >
>> > I,ve had the same issue with opera.
>> > I changed the line like this without any
problems so far:
>> >
>> > Element.setStyle(element, { opacity:
>> > (/Gecko/.test(navigator.userAgent) &&
>> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> > 0.999999 : null });
>> >
>> > to
>> >
>> > Element.setStyle(element, { opacity:
>> > (/Gecko/.test(navigator.userAgent) &&
>> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> > 0.999999 : 1 });
>> >
>> > Perhaps this works for Safari too.
>> >
>> > Lorenz
>> >
>> > On 16.08.2006, at 15:40, Brandon Aaron wrote:
>> >
>> >> I've been putting this off for several
days now, waiting to check
>> >> dev.rubyonrails.org before I posted here
but it is still down.
>> >>
>> >> I am using Effect.Appear and it is working
great except in Safari
>> >> where it fades in and then disappears. I
traced it back to the
>> >> Element.setOpacity function. Is there a
reason why Safari is  
>> singled
>> >> out and given a null value for opacity
instead of 1 or  
>> 0.999999? This
>> >> is where the problem is happening and can
be seen directly from  
>> the
>> >> demo page itself
>> >> (http:
//wiki.script.aculo.us/scriptaculous/show/
>> >> CombinationEffectsDemo).
>> >> I simply changed this line:
>> >>
>> >> Element.setStyle(element, { opacity:
>> >> (/Gecko/.test(navigator.userAgent)
&&
>> >>
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> >> 0.999999 : null });
>> >>
>> >> to this:
>> >>
>> >> Element.setStyle(element, { opacity:
0.999999 });
>> >>
>> >> And this working perfectly for me in
Safari 2 and 1.3 and it  
>> didn't
>> >> hurt any of the other supported browsers.
I don't have 1.2 to  
>> test it
>> >> in.
>> >>
>> >> However, I believe that Element.setStyle
should have the
>> >> responsiblity
>> >> of normalizing special cases like opacity.
More details here:
>> >> http://www.brandonaaron.net/articles/2006/08/12/ef
fect-appear-
>> >> safari-fix
>> >>
>> >> Brandon
>> >>
_______________________________________________
>> >> Rails-spinoffs mailing list
>> >> Rails-spinoffslists.rubyonrails.org
>> >> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>> >
>> >
_______________________________________________
>> > Rails-spinoffs mailing list
>> > Rails-spinoffslists.rubyonrails.org
>> > http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>>
>> --
>> Thomas Fuchs
>> wollzelle
>>
>> http://www.wollzelle.com
>>
>> questentier on AIM
>> madrobby on irc.freenode.net
>>
>> http://www.fluxiom.com ::
online digital asset management
>> http://script.aculo.us ::
Web 2.0 JavaScript
>> http://mir.aculo.us :: Where
no web developer has gone before
>>
>>
>>
>>
>> _______________________________________________
>> Rails-spinoffs mailing list
>> Rails-spinoffslists.rubyonrails.org
>> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>>
> _______________________________________________
> Rails-spinoffs mailing list
> Rails-spinoffslists.rubyonrails.org
> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

It seems to be a rendering bug in Safari related to the
float:left  
style.
Try Effect.Appear in the script.aculo.us demo. The DIV
appears  
correctly, then 'disappears'. Now just do a tiny resize on
the window  
and the DIV is there again.
I would say this is a bug.

Lorenz


On 18.08.2006, at 15:08, Brandon Aaron wrote:

> Sure I'll create some unit tests.
>
> Brandon
>
> On 8/18/06, Thomas Fuchs <t.fuchswollzelle.com> wrote:
>> I think we should make a test case page for this,
that has all sorts
>> of opacity enabled/disabled elements
>> and see if everything works fine. Any volunteers?

>>
>> -Thomas
>>
>> Am 18.08.2006 um 09:42 schrieb Lorenz Knies:
>>
>> > Hi Brandon,
>> >
>> > I,ve had the same issue with opera.
>> > I changed the line like this without any
problems so far:
>> >
>> > Element.setStyle(element, { opacity:
>> > (/Gecko/.test(navigator.userAgent) &&
>> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> > 0.999999 : null });
>> >
>> > to
>> >
>> > Element.setStyle(element, { opacity:
>> > (/Gecko/.test(navigator.userAgent) &&
>> >
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> > 0.999999 : 1 });
>> >
>> > Perhaps this works for Safari too.
>> >
>> > Lorenz
>> >
>> > On 16.08.2006, at 15:40, Brandon Aaron wrote:
>> >
>> >> I've been putting this off for several
days now, waiting to check
>> >> dev.rubyonrails.org before I posted here
but it is still down.
>> >>
>> >> I am using Effect.Appear and it is working
great except in Safari
>> >> where it fades in and then disappears. I
traced it back to the
>> >> Element.setOpacity function. Is there a
reason why Safari is  
>> singled
>> >> out and given a null value for opacity
instead of 1 or  
>> 0.999999? This
>> >> is where the problem is happening and can
be seen directly from  
>> the
>> >> demo page itself
>> >> (http:
//wiki.script.aculo.us/scriptaculous/show/
>> >> CombinationEffectsDemo).
>> >> I simply changed this line:
>> >>
>> >> Element.setStyle(element, { opacity:
>> >> (/Gecko/.test(navigator.userAgent)
&&
>> >>
!/Konqueror|Safari|KHTML/.test(navigator.userAgent)) ?
>> >> 0.999999 : null });
>> >>
>> >> to this:
>> >>
>> >> Element.setStyle(element, { opacity:
0.999999 });
>> >>
>> >> And this working perfectly for me in
Safari 2 and 1.3 and it  
>> didn't
>> >> hurt any of the other supported browsers.
I don't have 1.2 to  
>> test it
>> >> in.
>> >>
>> >> However, I believe that Element.setStyle
should have the
>> >> responsiblity
>> >> of normalizing special cases like opacity.
More details here:
>> >> http://www.brandonaaron.net/articles/2006/08/12/ef
fect-appear-
>> >> safari-fix
>> >>
>> >> Brandon
>> >>
_______________________________________________
>> >> Rails-spinoffs mailing list
>> >> Rails-spinoffslists.rubyonrails.org
>> >> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>> >
>> >
_______________________________________________
>> > Rails-spinoffs mailing list
>> > Rails-spinoffslists.rubyonrails.org
>> > http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>>
>> --
>> Thomas Fuchs
>> wollzelle
>>
>> http://www.wollzelle.com
>>
>> questentier on AIM
>> madrobby on irc.freenode.net
>>
>> http://www.fluxiom.com ::
online digital asset management
>> http://script.aculo.us ::
Web 2.0 JavaScript
>> http://mir.aculo.us :: Where
no web developer has gone before
>>
>>
>>
>>
>> _______________________________________________
>> Rails-spinoffs mailing list
>> Rails-spinoffslists.rubyonrails.org
>> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs
>>
> _______________________________________________
> Rails-spinoffs mailing list
> Rails-spinoffslists.rubyonrails.org
> http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs

_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffslists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-
spinoffs