RESOLVED CONFIGURATION CHANGED 82596
Remove document.width / document.height 
https://bugs.webkit.org/show_bug.cgi?id=82596
Summary Remove document.width / document.height
Ms2ger (he/him; ⌚ UTC+1/+2)
Reported 2012-03-29 04:11:38 PDT
+++ This bug was initially created as a clone of Bug #72591 +++ Bug 72591 was regressed in bug 80494, despite the fact that the patch there didn't have review.
Attachments
Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2012-03-29 09:17:05 PDT
> despite the fact that the patch there didn't have review. This is factually incorrect, the patch had an r+. Anyway, we can certainly discuss this again. However, we'll need a strong reason to consider removing this long-standing feature. In addition to compatibility constraints, there is evidence that developers want it - e.g. jQuery implements document.width/height for engines that don't have it. As an aside, I'm puzzled by the influx of bugs asking us to remove things that are in latest official specs or used to be more widely implemented, while not seeing any bugs asking us to remove truly WebKit-specific extensions to DOM.
Ms2ger (he/him; ⌚ UTC+1/+2)
Comment 2 2012-03-29 10:17:44 PDT
(In reply to comment #1) > As an aside, I'm puzzled by the influx of bugs asking us to remove things that are in latest official specs or used to be more widely implemented, while not seeing any bugs asking us to remove truly WebKit-specific extensions to DOM. Would you accept patches for such bugs? If so, I'd be very happy to file them.
Alexey Proskuryakov
Comment 3 2012-03-29 11:04:55 PDT
Removing purely WebKit extensions is less controversial than removing things that are in current W3C specs. Each suggestion would need to be judged on its own merits, and I obviously cannot guarantee that all or any of these would result in code changes.
Ms2ger (he/him; ⌚ UTC+1/+2)
Comment 4 2012-03-29 11:07:00 PDT
Note that document.width/height aren't in any spec, btw.
Alexey Proskuryakov
Comment 5 2012-03-29 11:27:45 PDT
Correct. This has been extensively discussed in bug 72591.
Anne van Kesteren
Comment 6 2023-12-11 07:37:48 PST
Both of these return undefined for me. This must have gotten fixed at some point. (We still have Document::width() and Document::height(), but presumably those are for app compatibility as discussed in the various bugs.)
Note You need to log in before you can comment on or make changes to this bug.