All users were logged out of Bugzilla on October 13th, 2018

In <viewZoomOverlay.js>, "Error: textZoomMenu has no properties", (When opening "Tools > Search Addresses...")

VERIFIED FIXED

Status

VERIFIED FIXED
15 years ago
13 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 obsolete attachment)

(Reporter)

Description

15 years ago
Found while working on bug 162432 (Dv1 part):
{
Error: textZoomMenu has no properties
Source File: chrome://communicator/content/viewZoomOverlay.js
Line: 229
}

It happens because of
{
--> ABSearchDialog.xul
    -> mailWindowOverlay.xul -> viewZoomOverlay.xul -> viewZoomOverlay.js ->
textZoomMenu
}
This error has no(!!?) consequence beside cluttering the JS.Console :-|

From neil's email:
{
What's happening is that viewZoomOverlay is only designed to be used in windows
with a zoom menu. This means that mailWindowOverlay.xul should only be used in
windows with a zoom menu. I think I fixed this in SearchDialog.xul by removing
mailWindowOverlay.xul but that meant that I had to write out the status bar
(rather than using the overlay) and manually include utilityOverlay.xul as part
of the fix. Hopefully something similar will fix all of these errors at once :-)
}

I'm not going to do it (now), then I file this separate bug:
*Doing it the way neil suggests, might undo some of bug 162432 (Dv1 part) patch:
 which will be fine. (I'm fixing the other errors in the meantime !)
*May be there could be a way to move the status bar related code to a different
but common place ??
(Reporter)

Comment 1

15 years ago
I often forget it these days:
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.6) Gecko/20040113] (W98SE)
(Reporter)

Comment 2

15 years ago
Created attachment 140202 [details] [diff] [review]
(Av1) <ABSearchDialog.xul>
[Checkin: See comment 4]

Would fix
{{
Error: redeclaration of const NS_BAYESIANFILTER_CONTRACTID
Source File: chrome://messenger/content/mailCommands.js
Line: 423
--> ABSearchDialog.xul
    -> mailWindowOverlay.xul -> mailCommands.js !
    -> mailCommands.js // removed

Error: redeclaration of const MSG_FLAG_IMAP_DELETED
Source File: chrome://messenger/content/mailWindowOverlay.js
Line: 28
--> ABSearchDialog.xul
    -> mailWindowOverlay.xul -> mailWindowOverlay.js !
    -> mailWindowOverlay.js // removed
}}
When opening "Tools > Search Adresses...";

but removing/replacing 'mailWindowOverlay.xul' might be a better/harder way to
go !?
Product: Browser → Seamonkey

Updated

14 years ago
Assignee: sspitzer → mail
(Reporter)

Updated

14 years ago
Blocks: 293305
(Reporter)

Updated

14 years ago
Depends on: 263951
(Reporter)

Comment 3

14 years ago
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.8) Gecko/20050511] (release) (W98SE)

Bug still there in MAS v1.7.x branch.

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050701] (nightly) (W98SE)

And line changed a little in MAS Trunk.

{{
Error: textZoomMenu has no properties
Source File: chrome://communicator/content/viewZoomOverlay.js
Line: 233
}}
This has been fixed by the patch I just checked in on bug 263951 (comment 16)
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
(Reporter)

Updated

13 years ago
No longer blocks: 293305
(Reporter)

Updated

13 years ago
Attachment #140202 - Attachment description: (Av1) <ABSearchDialog.xul> → (Av1) <ABSearchDialog.xul> [Checkin: See comment 4]
Attachment #140202 - Attachment is obsolete: true
(Reporter)

Comment 5

13 years ago
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/20050825 SeaMonkey/1.0a]
(nightly) (W98SE)

V.Fixed
Status: RESOLVED → VERIFIED
Component: MailNews: Search → Address Book
Keywords: testcase
Summary: In <viewZoomOverlay.js>, "Error: textZoomMenu has no properties", (When opening "Tools > Search Adresses...") → In <viewZoomOverlay.js>, "Error: textZoomMenu has no properties", (When opening "Tools > Search Addresses...")
You need to log in before you can comment on or make changes to this bug.