Bug 26673 - Dynamically changing .disabled doesn't work
Summary: Dynamically changing .disabled doesn't work
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://junkyard.damowmow.com/393
Keywords: NeedsReduction
Depends on:
Blocks:
 
Reported: 2009-06-24 02:19 PDT by sideshowbarker
Modified: 2013-01-25 20:31 PST (History)
6 users (show)

See Also:


Attachments
Testcase working as normal (450 bytes, text/html)
2009-06-26 06:30 PDT, Anthony Ricaud
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description sideshowbarker 2009-06-24 02:19:00 PDT
<MikeSmith> Hixie set up an "author view" of the HTML5 spec that hides some parts of the spec by disabling styles on them
<MikeSmith> but hiding doesn't seem to work in any version of Webkit I tried
<MikeSmith> http://www.whatwg.org/specs/web-apps/current-work/?style=author
<bdash> MikeSmith: hiding them how?
<Hixie> bdash: looks like dynamically changing .disabled isn't working if it's done onload (or near onload)
<Hixie> bdash: (.disabled on <link>)
<Hixie> bdash: the .disabled gets set right, but the stylesheets that apply don't update
<Hixie> according to the inspector
<bdash> ugh :(
Comment 1 Ian 'Hixie' Hickson 2009-06-26 02:01:59 PDT
(note that the spec's script has changed to work around this bug)
Comment 2 Anthony Ricaud 2009-06-26 06:30:11 PDT
Created attachment 31931 [details]
Testcase working as normal

Can't reproduce with the attached testcase.

Do you have a static copy of the spec with the bug ?
Comment 3 Ian 'Hixie' Hickson 2009-06-28 02:34:40 PDT
Here's a testcase showing the bug: http://junkyard.damowmow.com/393
Comment 4 sideshowbarker 2009-06-28 02:43:44 PDT
[updated URL field to point to Hixie's reduced test case]

fwiw, I note the test fails with KHTML as well (which I guess would suggest it's a bug that's been in there a long time, not a regression)
Comment 5 Anthony Ricaud 2009-06-28 07:56:52 PDT
From what I understand, this is not a bug with the disabled attribute. I think the stylesheet is never applied because it's linked as an alternate stylesheet.
Comment 6 Simon Fraser (smfr) 2011-01-08 21:33:32 PST
Dup of 25287?
Comment 7 Alexey Proskuryakov 2011-01-09 01:48:12 PST
Not a dupe of bug 25287-  this is about <link>, not <style>. But I'm wondering if <style> remains broken in this particular case, too.
Comment 8 Alexey Proskuryakov 2011-01-09 02:07:09 PST
A modification of Hixie's test that uses setTimeout(..., 1000) instead of window.onload still fails.

It seems wrong that .disabled is implemented via [Reflect] to me.