Changes between Initial Version and Version 1 of Ticket #5145, comment 8
- Timestamp:
- Sep 29, 2011, 9:06:52 PM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #5145, comment 8
initial v1 3 3 > I know this is a dublicate of 'Ticket #5145'. I can't reopen it :-( 4 4 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). 10 6 11 Now if I simply add this to my code:7 > Now if I simply add this to my code: 12 8 13 9 {{{ … … 17 13 }}} 18 14 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. 20 16 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: 23 19 24 20 … … 28 24 }}} 29 25 30 By removing the '$' sign it should work:26 > By removing the '$' sign it should work: 31 27 32 28