Open Bug 1703164 Opened 4 months ago Updated 26 days ago

convert top level <window> to <html> throughout the Thunderbird code base

Categories

(Thunderbird :: General, task)

Tracking

(thunderbird_esr78 wontfix)

ASSIGNED
Tracking Status
thunderbird_esr78 --- wontfix

People

(Reporter: mkmelin, Assigned: mkmelin)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open)

Attachments

(8 files)

Bug 1572062 converted the 3pane. For the most part styling has been adjusted already, and other preparations are now in place.

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

Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
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
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

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

Regressions: 1712535
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/ac7596a1da7b
followup: move closing MOZ_OPENPGP to right position. r=henry

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

Is it planned to include feed-subscriptions.xhtml in this party?

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.

@alta88 do you have a phabricator account? Did the conversion, but can't have you review without it.

I don't, the 2FA stuff was disinviting given no usage anymore.

I noticed the layout of this dialog was rather messed up, so fixed it in the process.

It is still rather messed up, with some accumulated regressions along the way. Things that should be done here:

  1. Eliminate the platform specific css (one exception for osx only).
  2. 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.
  3. No need for #contentPane margin.
  4. Initial dialog (without persisted xulStore dimensions) is too small; tree is only 2 rows high.
  5. It would be better to use html elements (except tree and menu) and grid/flex model for the detail info box.
  6. themeableDialog doesn't seem to have the dark theme blue for checkbox/radio elements; it either should or maybe preferences.css can be included.

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.

Regressions: 1719336
No longer regressions: 1719336
You need to log in before you can comment on or make changes to this bug.