convert top level <window> to <html> throughout the Thunderbird code base
Categories
(Thunderbird :: General, task)
Tracking
(thunderbird_esr78 wontfix)
| Tracking | Status | |
|---|---|---|
| thunderbird_esr78 | --- | wontfix |
People
(Reporter: mkmelin, Assigned: mkmelin)
References
(Blocks 1 open bug)
Details
(Keywords: leave-open)
Attachments
(86 files, 1 obsolete file)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
Bug 1703164 - convert mailnews/base/content/virtualFolderListEdit.xhtml to top level <html>. r=henry
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
Bug 1703164 - convert calendar/base/content/dialogs/publishDialog.xhtml to top level <html>. r=henry
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review |
Bug 1572062 converted the 3pane. For the most part styling has been adjusted already, and other preparations are now in place.
|
Assignee | |
Comment 1•2 years ago
|
||
|
|
Assignee | |
Comment 2•2 years ago
|
||
Depends on D113999
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/b31114f27664
convert mail/base/content/messageWindow.xhtml to top level <html>. r=henry
https://hg.mozilla.org/comm-central/rev/8f8fada0529c
convert mail/components/compose/content/messengercompose.xhtml to top level <html>. r=henry
|
|
Assignee | |
Updated•2 years ago
|
|
|
Assignee | |
Updated•2 years ago
|
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/88da4811edd9 adjust how height is calculated in mail/test/browser/folder-display/browser_openingMessages.js. rs=bustage-fix DONTBUILD
|
|
Assignee | |
Comment 5•2 years ago
|
||
|
|
Assignee | |
Comment 6•2 years ago
|
||
Depends on D114596
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/bd43c68db6da convert mail/components/addrbook/content/addressbook.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/d082e65a88d5 convert mail/components/preferences/preferences.xhtml to top level <html>. r=henry
|
||
Comment 8•2 years ago
|
||
|
|
||
Updated•2 years ago
|
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/50132988e778
Fix the Preference's font size after the HTML conversion. r=aleca
|
|
Assignee | |
Comment 10•2 years ago
|
||
|
||
Comment 11•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/ac7596a1da7b followup: move closing MOZ_OPENPGP to right position. r=henry
|
|
||
Comment 12•2 years ago
|
||
|
|
||
Updated•2 years ago
|
|
|
||
Comment 13•2 years ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/341d1f02efc5
Convert mail/components/downloads/content/aboutDownloads.xhtml to top level <html>. r=mkmelin DONTBUILD
|
||
Comment 14•2 years ago
|
||
Is it planned to include feed-subscriptions.xhtml in this party?
|
|
Assignee | |
Comment 15•2 years ago
|
||
My plan is to convert them all, but do it after 91 since there's not a huge rush and it can take time to weed out regressions, which have been subtle but annoying.
|
|
Assignee | |
Comment 16•2 years ago
|
||
@alta88 do you have a phabricator account? Did the conversion, but can't have you review without it.
|
|
||
Comment 17•2 years ago
|
||
I don't, the 2FA stuff was disinviting given no usage anymore.
|
|
Assignee | |
Comment 18•2 years ago
|
||
I noticed the layout of this dialog was rather messed up, so fixed it in the process.
|
|
||
Comment 19•2 years ago
|
||
It is still rather messed up, with some accumulated regressions along the way. Things that should be done here:
- Eliminate the platform specific css (one exception for osx only).
- The #selectFolder menu padding/margin has to be adjusted so the detail box doesn't jump when moving from folder to feed item in the tree.
- No need for #contentPane margin.
- Initial dialog (without persisted xulStore dimensions) is too small; tree is only 2 rows high.
- It would be better to use html elements (except tree and menu) and grid/flex model for the detail info box.
- themeableDialog doesn't seem to have the dark theme blue for checkbox/radio elements; it either should or maybe preferences.css can be included.
|
||
Comment 20•2 years ago
|
||
This change
https://hg.mozilla.org/comm-central/rev/d082e65a88d5#l1.103
breaks the ability to build with MOZ_UPDATER disabled. That will affect Linux distros. To fix this, we added
#ifdef MOZ_UPDATER
<script defer="defer" src="chrome://messenger/content/aboutDialog-appUpdater.js"></script>
+#else
+ <script defer="defer">
+ var { Services } = ChromeUtils.import("resource://gre/modules/Services.jsm");
+ var { XPCOMUtils } = ChromeUtils.import(
+ "resource://gre/modules/XPCOMUtils.jsm"
+ );
+ </script>
#endif
Essentially you get XPCOMUtils undefined later on. You may choose to do it some other way.
|
|
||
Comment 21•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/fa72aeb98faa convert mailnews/extensions/newsblog/feed-subscriptions.xhtml to top level <html>. r=henry
|
|
||
Comment 22•2 years ago
|
||
Pushed by geoff@darktrojan.net: https://hg.mozilla.org/comm-central/rev/649372d0b492 follow-up - Fix a broken test. rs=bustage-fix
|
|
Assignee | |
Comment 23•2 years ago
|
||
|
|
||
Comment 24•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/a0a67cd423a5 convert mailnews/base/prefs/content/AccountManager.xhtml to top level <html>. r=Paenglab
|
|
Assignee | |
Comment 25•2 years ago
|
||
|
|
Assignee | |
Comment 26•2 years ago
|
||
Depends on D124626
|
|
Assignee | |
Comment 27•2 years ago
|
||
Depends on D124627
|
|
Assignee | |
Comment 28•2 years ago
|
||
Depends on D124628
|
|
Assignee | |
Comment 29•2 years ago
|
||
Depends on D124629
|
|
Assignee | |
Comment 30•2 years ago
|
||
Depends on D124630
|
|
Assignee | |
Comment 31•2 years ago
|
||
This is the Local Folders page...
Depends on D124631
|
|
Assignee | |
Comment 32•2 years ago
|
||
Copies and Folders pane.
Depends on D124632
|
|
Assignee | |
Comment 33•2 years ago
|
||
Depends on D124633
|
|
Assignee | |
Comment 34•2 years ago
|
||
Depends on D124634
|
|
Assignee | |
Comment 35•2 years ago
|
||
Depends on D124635
|
|
Assignee | |
Comment 36•2 years ago
|
||
Depends on D124636
|
|
Assignee | |
Comment 37•2 years ago
|
||
Depends on D124637
|
|
Assignee | |
Comment 38•2 years ago
|
||
|
|
||
Comment 39•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/11d402e3146c convert mail/components/im/content/addbuddy.xhtml to top level <html>. r=freaktechnik https://hg.mozilla.org/comm-central/rev/8319cb118621 convert mail/components/im/content/imAccounts.xhtml to top level <html>. r=freaktechnik https://hg.mozilla.org/comm-central/rev/b50b29fc54a7 convert mail/components/im/content/imAccountWizard.xhtml to top level <html>. r=freaktechnik https://hg.mozilla.org/comm-central/rev/b3944f15cc0f convert mail/components/im/content/joinchat.xhtml to top level <html>. r=freaktechnik https://hg.mozilla.org/comm-central/rev/42f2868b0007 convert mailnews/base/prefs/content/am-main.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/597a9573fbcc convert mailnews/base/prefs/content/am-server.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/9fc5dd5898a6 convert mailnews/base/prefs/content/am-serverwithnoidentities.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/eba60e1ccb89 convert mailnews/base/prefs/content/am-copies.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/46578897c7e9 convert mailnews/base/prefs/content/am-junk.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/9c4fbbbb8d05 convert mailnews/base/prefs/content/am-offline.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/0a04b6526c1e convert mailnews/extensions/mdn/am-mdn.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/76e19f126207 convert mailnews/extensions/newsblog/am-newsblog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/77b3a77849ea convert mailnews/base/prefs/content/am-addressing.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 40•2 years ago
|
||
Convert to top level html, and remove <stringbundle>
|
|
||
Comment 41•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/3ff10ed0d3bd convert ./mailnews/export/content/exportDialog.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 42•2 years ago
|
||
|
|
Assignee | |
Comment 43•2 years ago
|
||
|
|
Assignee | |
Comment 44•2 years ago
|
||
|
|
Assignee | |
Comment 45•2 years ago
|
||
|
|
Assignee | |
Comment 46•2 years ago
|
||
This is difficult to trigger. For testing you can just do
window.openDialog(
"chrome://messenger-smime/content/certFetchingStatus.xhtml",
"", "chrome,modal,resizable,centerscreen", null, null
);
|
|
||
Comment 47•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/46cffd1b9a85 convert mailnews/compose/content/sendProgress.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/0660a3e1ff9e convert mailnews/import/content/importDialog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/80542f807639 convert mailnews/import/content/fieldMapImport.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/d262b0cf9324 convert mailnews/extensions/smime/msgCompSecurityInfo.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/b0c1de752f55 convert mailnews/extensions/smime/certpicker.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/eace7e5c001f convert mailnews/extensions/smime/certFetchingStatus.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 48•2 years ago
|
||
|
|
Assignee | |
Comment 49•2 years ago
|
||
|
|
Assignee | |
Comment 50•2 years ago
|
||
|
|
Assignee | |
Comment 51•2 years ago
|
||
|
|
Assignee | |
Comment 52•2 years ago
|
||
|
|
Assignee | |
Comment 53•2 years ago
|
||
|
|
Assignee | |
Comment 54•2 years ago
|
||
For testing, ucomment the last line in onload(), and run
window.openDialog("chrome://messenger/content/shutdownWindow.xhtml",
"", "chrome,modal,resizable,centerscreen", null, null
);
|
|
||
Comment 55•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/690715e763df convert mailnews/base/content/shutdownWindow.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/af327c38cbf2 convert mailnews/extensions/newsblog/feedAccountWizard.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/1be6b7b0185e convert mailnews/base/content/newFolderDialog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/88bebea13f70 convert mailnews/base/content/virtualFolderProperties.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/b87bf19809e3 convert mailnews/base/content/newmailalert.xhtml to top level <html>. r=henry
|
|
||
Comment 56•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/e5b538bbb148 followup to changeset 88bebea13f70 (virtualFolderProperties.xhtml to top level html) due to test failures in comm/mail/test/browser/search-window/browser_searchWindow.js. rs=bustage-fix DONTBUILD
|
|
||
Comment 57•2 years ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/4bbf35395a8d convert mailnews/base/content/msgSelectOfflineFolders.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 58•1 year ago
|
||
|
|
||
Comment 59•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/774194f1c1a8 convert calendar/base/content/dialogs/calendar-invitations-dialog.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 60•1 year ago
|
||
|
|
||
Comment 61•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/78d2559050ab convert mailnews/base/content/folderProps.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 62•1 year ago
|
||
|
|
Assignee | |
Comment 63•1 year ago
|
||
I think the ealier work is tripping on <stringbundle> not being present (yet) in the dom.
There is still oddness of event handling with xul:dialog. The can't be where it should be (in head).
|
|
||
Comment 64•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/9b58e351e107 virtualFolderProperties <stringbundle> removal. r=freaktechnik
|
|
||
Comment 65•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/6b719976f9e3 convert mailnews/base/content/msgSynchronize.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 66•1 year ago
|
||
|
|
Assignee | |
Comment 67•1 year ago
|
||
Virtual Folder Properties | Choose...
|
|
Assignee | |
Comment 68•1 year ago
|
||
|
|
||
Comment 69•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/c832cc4bd606 remove no longer needed body class="body" in messenger.xhtml. rs=me
|
|
||
Comment 70•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/a849f74066e4 convert mailnews/base/content/virtualFolderListEdit.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/c7b2813a799f convert mailnews/base/content/subscribe.xhtml to top level <html>. r=henry
|
|
||
Comment 71•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/42bfebd9c6d2 follow up to a849f74066e4 - fix linting. rs=eslint DONTBUILD
|
|
Assignee | |
Comment 72•1 year ago
|
||
|
|
Assignee | |
Comment 73•1 year ago
|
||
|
|
Assignee | |
Comment 74•1 year ago
|
||
|
|
Assignee | |
Comment 75•1 year ago
|
||
|
|
Assignee | |
Comment 76•1 year ago
|
||
|
|
Assignee | |
Comment 77•1 year ago
|
||
Also updating the example years...
|
|
Assignee | |
Comment 78•1 year ago
|
||
|
|
Assignee | |
Comment 79•1 year ago
|
||
|
|
Assignee | |
Comment 80•1 year ago
|
||
|
|
Assignee | |
Comment 81•1 year ago
|
||
|
|
Assignee | |
Comment 82•1 year ago
|
||
|
|
Assignee | |
Comment 83•1 year ago
|
||
|
|
Assignee | |
Comment 84•1 year ago
|
||
|
|
Assignee | |
Comment 85•1 year ago
|
||
|
|
||
Comment 86•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/88f84ab6f94b convert mailnews/base/content/markByDate.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/ec5da861d2b7 convert mailnews/base/content/renameFolderDialog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/7b31c1cfcffb convert mailnews/base/content/junkLog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/8cbc6a589508 convert mailnews/base/prefs/content/am-card-dialog.xhtml to top level <html>. r=henry
|
|
||
Comment 87•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/867a4c689d51 convert mailnews/base/prefs/content/am-smtp.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/a9aa8a3b9077 convert mailnews/addrbook/prefs/content/pref-editdirectories.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/94a95510772d convert mailnews/base/prefs/content/am-identities-list.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/62d67f138cfa convert mailnews/base/prefs/content/removeAccount.xhtml to top level <html>. r=henry
|
|
||
Comment 88•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/1ff7354b2466 follow-up to 867a4c689d51 - fix linting. rs=eslint DONTBUILD
|
||
Updated•1 year ago
|
|
|
||
Comment 89•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/2dca250deaa7 convert mailnews/base/prefs/content/SmtpServerEdit.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/1a490ff809b7 convert mailnews/base/prefs/content/am-archiveoptions.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/fc2a3734c4ce convert mailnews/base/prefs/content/am-identity-edit.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/bd0c98e22865 convert mailnews/base/prefs/content/am-server-advanced.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 90•1 year ago
|
||
|
|
Assignee | |
Comment 91•1 year ago
|
||
|
|
Assignee | |
Comment 92•1 year ago
|
||
|
||
Comment 93•1 year ago
|
||
Pulsebot comments for changesets below are not added in this bug. 😟
https://hg.mozilla.org/comm-central/rev/c385f94aa24cf49ab67d202327cc1212de2f7f67
https://hg.mozilla.org/comm-central/rev/565de8732cb20e5cb329c8dcf6701f435ec52da5
https://hg.mozilla.org/comm-central/rev/a4c122858ebe9bb672b411e093e6342a53a1243a
|
|
||
Comment 94•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/1869b1083658 convert mailnews/addrbook/content/abAddressBookNameDialog.xhtml to top level <html>. r=henry
|
|
||
Comment 95•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/7f2555122329 convert mailnews/search/content/CustomHeaders.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/f3c39b8ae82f convert mailnews/search/content/viewLog.xhtml to top level <html>. r=henry
|
|
||
Comment 96•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/197babe349e1 convert mailnews/search/content/FilterEditor.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 97•1 year ago
|
||
This is the dialog one gets by right clicking a contact "Add OTR Fingerprint" context menu.
|
|
||
Comment 98•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/1090401eeac3 convert chat/content/otr-add-fingerprint.xhtml to top level <html>. r=henry
|
|
Assignee | |
Comment 99•1 year ago
|
||
|
|
Assignee | |
Comment 100•1 year ago
|
||
This is the dialog one gets in chat for "Verify".
|
|
Assignee | |
Comment 101•1 year ago
|
||
|
|
Assignee | |
Comment 102•1 year ago
|
||
|
|
Assignee | |
Comment 103•1 year ago
|
||
|
|
Assignee | |
Comment 104•1 year ago
|
||
|
|
Assignee | |
Comment 105•1 year ago
|
||
|
|
Assignee | |
Comment 106•1 year ago
|
||
|
|
Assignee | |
Comment 107•1 year ago
|
||
|
|
Assignee | |
Comment 108•1 year ago
|
||
For testing use
var paramBlock = Cc["@mozilla.org/embedcomp/dialogparam;1"].createInstance(
Ci.nsIDialogParamBlock
);
paramBlock.SetNumberStrings(3);
paramBlock.SetString(0, "An error was encountered preparing the calendar located at http://example.com for use. It will not be available.");
paramBlock.SetString(1, "0x12334");
paramBlock.SetString(2, "blablabla");
Services.ww.openWindow(
null,
"chrome://calendar/content/calendar-error-prompt.xhtml",
"_blank",
"chrome,dialog=yes,alwaysRaised=yes",
paramBlock
);
|
|
Assignee | |
Comment 109•1 year ago
|
||
For testing
window.openDialog(
"chrome://calendar/content/calendar-conflicts-dialog.xhtml",
"calendarConflictsDialog",
"chrome,titlebar,modal",
{
item: { isEvent() { return true}, getProperty() { return "ABC"; }, get calendar() { return { name: "test"} }, title: "Test title" } ,
mode: "modify",
overwrite: false,
}
);
|
|
Assignee | |
Comment 110•1 year ago
|
||
For testing
window.openDialog(
"chrome://calendar/content/calendar-conflicts-dialog.xhtml",
"calendarConflictsDialog",
"chrome,titlebar,modal",
{
item: { isEvent() { return true}, getProperty() { return "ABC"; }, get calendar() { return { name: "test"} }, title: "Test title" } ,
mode: "modify",
overwrite: false,
}
);
|
|
Assignee | |
Comment 111•1 year ago
|
||
|
|
Assignee | |
Comment 112•1 year ago
|
||
|
|
||
Comment 113•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/85057d25d733 convert chat/content/otr-finger.xhtml to top level <html>. r=henry
|
|
||
Comment 114•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/93e4d449e781 convert chat/content/otr-auth.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/c87146f3e9a8 convert calendar/base/content/preferences/editCategory.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/b60a405286b9 convert calendar/base/content/dialogs/calendar-creation.xhtml to top level <html>. r=henry
|
|
||
Comment 115•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/5f6ca8ed2cec follow-up to c87146f3e9a80f9632d35578b67b5d48c6a4e4e6 - fix test. rs=me DONTBUILD
|
|
||
Comment 116•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/5ee9579cf4e0 convert calendar/base/content/dialogs/calendar-conflicts-dialog.xhtml to top level <html>. r=henry
|
|
||
Comment 117•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/2d2662f7a91f convert calendar/base/content/dialogs/calendar-occurrence-prompt.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/49793e9acd94 convert calendar/base/content/dialogs/calendar-event-dialog-timezone.xhtml to top level <html>. r=henry
|
|
||
Comment 118•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/8d19fb20b78a convert calendar/base/content/dialogs/calendar-alarm-dialog.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/25e0821ebf6d convert calendar/base/content/dialogs/calendar-error-prompt.xhtml to top level <html>. r=henry
|
|
||
Comment 119•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/9042d4892485 convert calendar/base/content/item-editing/calendar-item-iframe.xhtml to top level <html>. r=henry https://hg.mozilla.org/comm-central/rev/94a1c4858fd6 convert calendar/base/content/dialogs/calendar-summary-dialog.xhtml to top level <html>. r=henry
|
|
||
Comment 120•1 year ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/0621b33d2f2a convert calendar/base/content/dialogs/calendar-event-dialog-recurrence.xhtml to top level <html>. r=henry
|
||
Comment 121•1 year ago
|
||
Backed out the last patch for failures on Windows and possibly one on Mac as well.
https://hg.mozilla.org/comm-central/rev/d0c72993f2bb2a0d4dd2178cb4066340120e34a8
|
|
||
Comment 122•11 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/8e7632bbfa1d convert calendar/base/content/dialogs/publishDialog.xhtml to top level <html>. r=henry
|
|
||
Comment 123•11 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/2643d2104b14 convert calendar/base/content/dialogs/calendar-properties-dialog.xhtml to top level <html>. r=henry
|
||
Comment 124•11 months ago
|
||
(In reply to Pulsebot from comment #96)
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/197babe349e1
convert mailnews/search/content/FilterEditor.xhtml to top level <html>.
r=henry
A little overzealous replacement here:
https://hg.mozilla.org/comm-central/rev/197babe349e1#l2.102
https://hg.mozilla.org/comm-central/rev/197babe349e1#l2.152
|
||
Comment 125•9 months ago
|
||
Bug 1752288 is open to address the following problem, but I just wanted to add some context here to explain how bug 1752288 is effecting the patches here (taken from https://phabricator.services.mozilla.com/D131063#4271047):
If the window "load" event is fired before a xul:dialog's connectedCallback, then this line throws https://searchfox.org/mozilla-central/rev/9379d512333cb143ed6ee5df5a40274f40d6ce87/toolkit/content/widgets/dialog.js#324 because this._l10nButtons is not initialized. This means that the sizeToContent just below is never called, so the dialog can end up the wrong size.
This can happen because we use <script defer> (https://phabricator.services.mozilla.com/D131063#4279233), but whether it actually happens in usage can come down to seemingly random factors.
So if you (sometimes) see a dialog that does not have the right size and you see in the console
Uncaught TypeError: this._l10nButtons is undefined
this is the origin.
|
|
||
Comment 126•8 months ago
|
||
(In reply to Henry Wilkes (they/them) [:henry] from comment #125)
Bug 1752288 is open to address the following problem, but I just wanted to add some context here to explain how bug 1752288 is effecting the patches here (taken from https://phabricator.services.mozilla.com/D131063#4271047):
This same problem seems to be popping up every now and then across bugzilla. Unless we know of a solution to bug 1752288, maybe we should just stop deferring the <script> (as we do here https://searchfox.org/comm-central/rev/cd3088d72a9a397af39540a47a785e31581a8ccc/calendar/base/content/dialogs/calendar-conflicts-dialog.xhtml#30) for all the dialogs touched in this bug.
|
|
Assignee | |
Comment 127•5 months ago
|
||
|
|
Assignee | |
Updated•5 months ago
|
|
|
||
Comment 128•5 months ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/f14bf2bcae63
convert mail/components/extensions/extensionPopup.xhtml to top level <html>. r=john.bieling
|
|
Assignee | |
Comment 129•3 months ago
|
||
Bug 1752288 is not fully fixed, but for the cases affected just adding the fluent <link> is a nicer workaround.
|
|
Assignee | |
Comment 130•3 months ago
|
||
Easiest way to trigger: set mail.import.in_new_tab to false, then Events and Tasks | Import
|
|
Assignee | |
Comment 131•3 months ago
|
||
To trigger
mkdir -p .evolution/calendar/local
cp foo.ics .evolution/calendar/local
Starting with a new profile, the migration dialog will trigger.
|
|
||
Comment 132•3 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/5416b7813b53 Remove workarounds for bug 1752288 since that's now fixed. r=aleca
|
||
Comment 133•3 months ago
|
||
|
|
||
Updated•3 months ago
|
|
|
||
Comment 134•3 months ago
|
||
Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/ea1e8c983f04
Fix browser_deletion test by adding fluent link workaround. r=freaktechnik
|
|
||
Updated•3 months ago
|
|
|
||
Comment 135•3 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/e3fc93730056 convert calendar/base/content/dialogs/calendar-ics-file-dialog.xhtml to top level <html>. r=aleca
|
||
Comment 136•3 months ago
|
||
Goes with D165114 that already landed to avoid re-translation.
|
|
||
Comment 137•3 months ago
|
||
It looks like many of these migrations have a string migration for the .title XUL attribute to an HTML <title> element.
To avoid unnecessary re-translation, we can use a migration like in comment 136 (for Fluent-to-Fluent cases). If other cases come up, let's start using migrations since we have the ability now. It will also help with reducing untranslated strings, and let translators work on actual new strings.
|
|
||
Comment 138•3 months ago
|
||
Pushed by daniel@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/ff5b4d8b5e79
Fluent2Fluent migration for calendar-ics-file-window-title. r=mkmelin
|
|
||
Comment 139•2 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/2286eaa047f4 convert calendar/base/content/dialogs/calendar-migration-dialog.xhtml to top level <html>. r=aleca
|
|
Assignee | |
Comment 140•2 months ago
|
||
|
|
Assignee | |
Comment 141•2 months ago
|
||
|
|
||
Comment 142•2 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/9143759ba745 convert calendar/base/content/dialogs/calendar-providerUninstall-dialog.xhtml to top level <html>. r=aleca https://hg.mozilla.org/comm-central/rev/2bc6f7dff22b convert calendar/base/content/dialogs/calendar-uri-redirect-dialog.xhtml to top level <html>. r=aleca
|
|
||
Comment 143•2 months ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/c24b253e7adc followup - fix linting for 2bc6f7dff22b. rs=black-lint DONTBUILD
|
|
Assignee | |
Comment 144•1 month ago
|
||
|
|
||
Updated•1 month ago
|
|
|
||
Comment 145•1 month ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/191604b5663d convert mail/base/content/SearchDialog.xhtml to top level <html>. r=aleca
|
|
Assignee | |
Comment 146•13 days ago
|
||
Converts to top level html, and fixes the cutoff looks it had (from flexbox emulation removal?).
We don't do anything with encodings here anymore; title updated to reflect that.
|
|
||
Updated•11 days ago
|
|
|
Assignee | |
Comment 147•11 days ago
|
||
|
|
||
Comment 148•11 days ago
|
||
Comment on attachment 9322283 [details]
Bug 1703164 - convert calendar/base/content/dialogs/calendar-event-dialog-attendees.xhtml to top level <html>. r=#thunderbird-front-end-reviewers
Revision D172202 was moved to bug 1821435. Setting attachment 9322283 [details] to obsolete.
|
|
Assignee | |
Updated•11 days ago
|
|
|
||
Comment 149•11 days ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/75f43cc9a96c
convert mail/components/preferences/fonts.xhtml to top level <html>. r=aleca
|
|
Assignee | |
Comment 150•9 days ago
|
||
|
|
Assignee | |
Updated•7 days ago
|
|
|
||
Comment 151•7 days ago
|
||
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/1d5741997e8c
convert mail/base/content/FilterListDialog.xhtml to top level <html>. r=aleca
|
|
||
Updated•1 day ago
|
|
|
||
Comment 152•4 hours ago
|
||
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/85c2f14774a5 convert calendar/base/content/dialogs/calendar-event-dialog-recurrence.xhtml to top level <html>. r=elizabeth
Description
•