Closed
Bug 1448655
Opened 7 years ago
Closed 7 years ago
Replace thunderbirdOverlay.xul with inlining in messenger.xul
Categories
(Thunderbird :: General, defect)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 61.0
People
(Reporter: Paenglab, Assigned: Paenglab)
References
Details
Attachments
(1 file)
|
3.48 KB,
patch
|
jorgk-bmo
:
review+
|
Details | Diff | Splinter Review |
Also gloda has a overlay.
|
Assignee | |
Comment 1•7 years ago
|
||
A overlay for only adding a script. Also this script could be eliminated as it only does:
ChromeUtils.import("resource:///modules/gloda/public.js");
If you want, I can also add this line to a already loaded script. But which one? And when we then don't need this file, we do also not need to package it. Does SM use this file?
|
||
Comment 2•7 years ago
|
||
Comment on attachment 8962145 [details] [diff] [review]
thunderbirdOverlay.patch
Let's not refactor too much here and take it as you've done it.
Attachment #8962145 -
Flags: review?(jorgk) → review+
|
|
Assignee | |
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/7eff4ebe1243
Replace thunderbirdOverlay.xul with inlining in messenger.xul. r=jorgk
|
|
||
Updated•7 years ago
|
Target Milestone: --- → Thunderbird 61.0
You need to log in
before you can comment on or make changes to this bug.
Description
•