Bug Tracker

Changes between Initial Version and Version 1 of Ticket #5145, comment 8


Ignore:
Timestamp:
Sep 29, 2011, 9:06:52 PM (8 years ago)
Author:
Rick Waldron
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5145, comment 8

    initial v1  
    33> I know this is a dublicate of 'Ticket #5145'. I can't reopen it :-(
    44
    5 My phone (a webkit clone) throws an error on FadeIn() and fadeOut().
    6 That's because it returns 'false' for '$.support.opacity'.
    7 Then it triggers the IE only 'cssHooks.opacity' and chunks on
    8 'style.removeAttribute'.
    9 The real problem is that when setting style.opacity to '.55' and then read it, it says: '0,550000011920929' (note the comma and the rounding error).
     5> My phone (a webkit clone) throws an error on FadeIn() and fadeOut(). That's because it returns 'false' for '$.support.opacity'. Then it triggers the IE only 'cssHooks.opacity' and chunks on 'style.removeAttribute'. The real problem is that when setting style.opacity to '.55' and then read it, it says: '0,550000011920929' (note the comma and the rounding error).
    106
    11 Now if I simply add this to my code:
     7> Now if I simply add this to my code:
    128
    139{{{
     
    1713}}}
    1814
    19 everything is fine, fadeIn() works like show() and fadeOut() works like hide() - a lot better than an error.
     15> everything is fine, fadeIn() works like show() and fadeOut() works like hide() - a lot better than an error.
    2016
    21 So, basically we need to change the test for 'support.opacity', so it includes the above value, but still excludes old IE.
    22 Line 1221 currently says:
     17> So, basically we need to change the test for 'support.opacity', so it includes the above value, but still excludes old IE.
     18> Line 1221 currently says:
    2319
    2420
     
    2824}}}
    2925
    30 By removing the '$' sign it should work:
     26> By removing the '$' sign it should work:
    3127
    3228