15559 0 true 2008-09-30T17:13:18+02:00 17579 2009-07-14T00:00:00+02:00 20238 364 ie8-doesnt-like-readattributeclass 3 8886 resolved needs:patch section:dom 2009-07-24T03:37:19+02:00 14168 Tobie Langel Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 It appears that IE8 doesn't need the renaming of class to className when calling readAttribute. If you create a standards compliant page (to force it to use IE8 code paths) and then call $('something').readAttribute('class'), you'll get null. But if you do $('something').getAttribute('class'), you'll get the actual class name. There might be other renamings that are done for IE that are break IE8 but I only discovered the 'class' as it broke my site. It appears that IE8 doesn't need the renaming of class to className when calling readAttribute. If you create a standards compliant page (to force it to use IE8 code paths) and then call $('something').readAttribute('class'), you'll get null. But if you do $('something').getAttribute('class'), you'll get the actual class name. There might be other renamings that are done for IE that are break IE8 but I only discovered the 'class' as it broke my site. <div><p>It appears that IE8 doesn't need the renaming of class to className when calling readAttribute. If you create a standards compliant page (to force it to use IE8 code paths) and then call $('something').readAttribute('class'), you'll get null. But if you do $('something').getAttribute('class'), you'll get the actual class name. There might be other renamings that are done for IE that are break IE8 but I only discovered the 'class' as it broke my site.</p></div> 0 It appears that IE8 doesn't need the renaming of class to className when calling readAttribute. If you create a standards compliant page (to force it to use IE8 code paths) and then call $('something').readAttribute('class'), you'll get null. But if you do $('something').getAttribute('class'), you'll get the actual class name. There might be other renamings that are done for IE that are break IE8 but I only discovered the 'class' as it broke my site. <div><p>It appears that IE8 doesn't need the renaming of class to className when calling readAttribute. If you create a standards compliant page (to force it to use IE8 code paths) and then call $('something').readAttribute('class'), you'll get null. But if you do $('something').getAttribute('class'), you'll get the actual class name. There might be other renamings that are done for IE that are break IE8 but I only discovered the 'class' as it broke my site.</p></div> false 2008-09-30T17:13:19+02:00 17579 --- {} 364 ie8-doesnt-like-readattributeclass 0 8886 new 2008-09-30T17:13:21+02:00 17579 Webbles Webbles http://prototype.lighthouseapp.com/projects/8886/tickets/364 0 There are probably quicker and more sophicated ways to do this, but here's something I came up with: @@@ javascript Element._attributeTranslations = { read: { names: { 'class': navigator.userAgent.indexOf('MSIE 8') > -1 ? 'class' : 'className', 'for': 'htmlFor' }, @@@ `navigator.userAgent.indexOf('MSIE 8')` obviously checks the browser version, but I think feature detection (something like `if (Element.className)`) would be better. Of course pre-checking it would improve performance... (I have not tested the example code, since I have no IE8 on my computer!) <div><p>There are probably quicker and more sophicated ways to do this, but here's something I came up with:</p> <pre><code class="javascript"> Element._attributeTranslations = { read: { names: { 'class': navigator.userAgent.indexOf('MSIE 8') &gt; -1 ? 'class' : 'className', 'for': 'htmlFor' }, </code></pre> <p><code>navigator.userAgent.indexOf('MSIE 8')</code> obviously checks the browser version, but I think feature detection (something like <code>if (Element.className)</code>) would be better. Of course pre-checking it would improve performance...</p> <p>(I have not tested the example code, since I have no IE8 on my computer!)</p></div> false 2008-10-01T02:59:59+02:00 17579 --- :tag: 364 ie8-doesnt-like-readattributeclass 0 8886 new ie8 2008-10-01T03:00:03+02:00 33079 Xanadu Webbles http://prototype.lighthouseapp.com/projects/8886/tickets/364 15559 0 false 2008-10-01T18:56:47+02:00 17579 --- :tag: ie8 :state: new :milestone: :title: IE8 doesn't like readAttribute('class') :assigned_user: 9616 364 ie8-doesnt-like-readattributeclass 0 8886 bug 2008-10-01T18:56:52+02:00 15559 Juriy Zaytsev Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.7 15559 0 Could someone test if IE8 behaves the same way with for/htmlFor? Thanks. <div><p>Could someone test if IE8 behaves the same way with for/htmlFor?</p> <p>Thanks.</p></div> false 2008-10-01T19:00:00+02:00 17579 --- {} 9616 364 ie8-doesnt-like-readattributeclass 0 8886 bug 2008-10-01T19:00:04+02:00 15559 Juriy Zaytsev Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.7 15559 0 Did a little google search on the subject, I found this [page on the dojo trac](http://trac.dojotoolkit.org/ticket/6665) stating that IE8 uses `htmlFor`. <div><p>Did a little google search on the subject, I found this <a href="http://trac.dojotoolkit.org/ticket/6665">page on the dojo trac</a> stating that IE8 uses <code>htmlFor</code>.</p></div> false 2008-10-01T19:56:09+02:00 17579 --- {} 9616 364 ie8-doesnt-like-readattributeclass 0 8886 bug 2008-10-01T19:56:13+02:00 33079 Xanadu Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.7 15559 0 A better resource at [MSDN](http://msdn.microsoft.com/en-us/library/ms536739(VS.85).aspx) > When setting the CLASS attribute using this method, set the sName to be "className", which is the corresponding Dynamic HTML (DHTML) property. <div><p>A better resource at <a href="http://msdn.microsoft.com/en-us/library/ms536739%28VS.85">MSDN</a>.aspx)</p> <blockquote> <p>When setting the CLASS attribute using this method, set the sName to be "className", which is the corresponding Dynamic HTML (DHTML) property.</p> </blockquote></div> false 2008-10-02T15:13:18+02:00 17579 --- {} 9616 364 ie8-doesnt-like-readattributeclass 0 8886 bug 2008-10-02T15:13:21+02:00 33079 Xanadu Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.7 15559 0 The right URL: http://msdn.microsoft.com/en-us/library/ms536739(VS.85).aspx (The bracket in the URL breaks the URL formatting - not really bug in Lighthouse, I guess) <div><p>The right URL: <a href="http://msdn.microsoft.com/en-us/library/ms536739(VS.85).aspx">http://msdn.microsoft.com/en-us/...</a></p> <p>(The bracket in the URL breaks the URL formatting - not really bug in Lighthouse, I guess)</p></div> false 2008-10-02T15:16:42+02:00 17579 --- {} 9616 364 ie8-doesnt-like-readattributeclass 0 8886 bug 2008-10-02T15:16:47+02:00 33079 Xanadu Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.7 15559 0 false 2008-10-03T18:45:21+02:00 17579 --- :tag: :milestone: 9616 20238 364 ie8-doesnt-like-readattributeclass 0 8886 bug attributes dom needs_patch 2008-10-03T18:45:23+02:00 15556 Andrew Dupont Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 I feel hesitant to mark it as 1.6.0.4. It's not clear what IE is up to and how later revisions will behave. Can we move it to 1.6.1? <div><p>I feel hesitant to mark it as 1.6.0.4. It's not clear what IE is up to and how later revisions will behave.</p> <p>Can we move it to 1.6.1?</p></div> false 2008-10-07T06:28:15+02:00 17579 --- {} 20238 364 ie8-doesnt-like-readattributeclass 0 8886 bug attributes dom needs_patch 2008-10-07T06:28:18+02:00 15559 Juriy Zaytsev Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 IE8 is still beta, right? As long as it is still in development, there's hurry to fix this bug, like Juriy said. <div><p>IE8 is still beta, right? As long as it is still in development, there's hurry to fix this bug, like Juriy said.</p></div> false 2008-10-07T13:55:38+02:00 17579 --- {} 20238 364 ie8-doesnt-like-readattributeclass 0 8886 bug attributes dom needs_patch 2008-10-07T13:55:42+02:00 33079 Xanadu Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 Proper testing should allow us to just avoid sniffing. <div><p>Proper testing should allow us to just avoid sniffing.</p></div> false 2008-10-07T18:39:51+02:00 17579 --- {} 20238 364 ie8-doesnt-like-readattributeclass 0 8886 bug attributes dom needs_patch 2008-10-07T18:39:54+02:00 14168 Tobie Langel Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 This was fixed a while back when I made all the changes necessary to get the DOM tests passing in IE8. <div><p>This was fixed a while back when I made all the changes necessary to get the DOM tests passing in IE8.</p></div> true 2009-02-28T21:33:16+01:00 17579 --- :state: bug 20238 364 ie8-doesnt-like-readattributeclass 0 8886 resolved attributes dom needs_patch 2009-02-28T21:33:21+01:00 15556 Andrew Dupont Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 [not-tagged:"dom" tagged:"section:dom" bulk edit command] <div><p>[not-tagged:"dom" tagged:"section:dom" bulk edit command]</p></div> true 2009-07-24T02:10:43+02:00 17579 --- :tag: attributes dom needs_patch 20238 364 ie8-doesnt-like-readattributeclass 0 8886 resolved needs_patch section:dom 2009-07-24T02:10:43+02:00 14168 Tobie Langel Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 [not-tagged:"needs_patch" tagged:"missing:patch" bulk edit command] <div><p>[not-tagged:"needs_patch" tagged:"missing:patch" bulk edit command]</p></div> true 2009-07-24T02:28:20+02:00 17579 --- :tag: needs_patch section:dom 20238 364 ie8-doesnt-like-readattributeclass 0 8886 resolved missing:patch section:dom 2009-07-24T02:28:20+02:00 14168 Tobie Langel Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1 15559 0 [not-tagged:"missing:patch" tagged:"needs:patch" bulk edit command] <div><p>[not-tagged:"missing:patch" tagged:"needs:patch" bulk edit command]</p></div> true 2009-07-24T03:37:19+02:00 17579 --- :tag: missing:patch section:dom 20238 364 ie8-doesnt-like-readattributeclass 0 8886 resolved needs:patch section:dom 2009-07-24T03:37:19+02:00 14168 Tobie Langel Webbles Juriy Zaytsev http://prototype.lighthouseapp.com/projects/8886/tickets/364 1.6.1