Closed
Bug 651750
Opened 14 years ago
Closed 14 years ago
Fix emailWizard.js now that document.height and document.width have gone
Categories
(MailNews Core :: Account Manager, defect)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
standard8
:
review-
|
Details | Diff | Splinter Review |
Bug 585877 removed document.height and document.width, so need to fix emailWizard.js to use something else.
Attachment #527447 -
Flags: review?(bugzilla)
|
||
Comment 1•14 years ago
|
||
Ok, so I don't know what's going on here.
Without the patch the account wizard is displaying fine, height and width seem to be set appropriately. I've also tried it on aurora and get no warning that these are obsolete.
With the patch I get:
JavaScript strict warning: chrome://messenger/content/accountcreation/emailWizard.js, line 251: reference to undefined property document.body
JavaScript error: chrome://messenger/content/accountcreation/emailWizard.js, line 251: document.body is undefined
My only thought is that there's actually a different set of functionality between chrome and content.
|
|
||
Updated•14 years ago
|
Attachment #527447 -
Flags: review?(mbanner) → review-
|
||
Comment 2•14 years ago
|
||
Bug 585877 removed document.height and document.width for HTML documents only.
XUL documents still have height and width, although it appears to be the same as window.innerHeight/Width.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•