Replace use of obsolete dialogOverlay.xul, in MailNews Core

RESOLVED WORKSFORME

Status

--
minor
RESOLVED WORKSFORME
4 years ago
2 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

(Blocks: 2 bugs)

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

4 years ago
Ftr,
http://mxr.mozilla.org/mozilla-central/source/toolkit/obsolete/content/dialogOverlay.xul

Last use in /mailnews:
{
/mailnews/import/content/importDialog.xul
    * line 8 -- <?xul-overlay href="chrome://global/content/dialogOverlay.xul"?>
}

And, wrt bug 226956:
{
8 <?xul-overlay href="chrome://global/content/dialogOverlay.xul"?>
9 
10 <!DOCTYPE window [
...
17 <window xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
}
NB: Though there is no |class="dialog"|.

Comment 1

4 years ago
I think we have done something like this lately, maybe for TB.

Comment 2

2 years ago
If the importDialog is split into a separate bug, maybe we could look at the /editor/ here:
http://mxr.mozilla.org/comm-central/search?string=globalOverlay.xul&case=1&find=%2Feditor%2F&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=comm-central

Comment 3

2 years ago
Actually the correct search is:
http://mxr.mozilla.org/comm-central/search?string=%2FdialogOverlay.xul&find=&findi=&filter=%5E%5B%5E%5C0%5D*%24&hitlimit=&tree=comm-central

And that one does not find any uses in /mailnews anymore, except the import wizard, that is in a separate bug.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.