Remove LightweightThemeManager
Categories
(Toolkit :: Add-ons Manager, enhancement, P3)
Tracking
()
People
(Reporter: ntim, Unassigned)
References
(Blocks 2 open bugs)
Details
LightweightThemeManager isn't involved in the flow of WebExtension themes at all, so we can remove it once lightweight themes are unsupported.
|
||
Updated•6 years ago
|
|
||
Comment 1•5 years ago
|
||
(In reply to Tim Nguyen :ntim from comment #0)
LightweightThemeManager isn't involved in the flow of WebExtension themes at
all
It still is on Android, isn't it? So that'd have to be rewired, too, before being able to get rid of the LWTManager.
|
Reporter | |
Comment 2•5 years ago
|
||
(In reply to Jan Henning [:JanH] from comment #1)
(In reply to Tim Nguyen :ntim from comment #0)
LightweightThemeManager isn't involved in the flow of WebExtension themes at
allIt still is on Android, isn't it? So that'd have to be rewired, too, before being able to get rid of the LWTManager.
I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)
|
|
||
Comment 3•5 years ago
•
|
||
(In reply to Tim Nguyen :ntim from comment #2)
I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)
No, I do mean the Manager.
|
|
Reporter | |
Comment 4•5 years ago
|
||
(In reply to Jan Henning [:JanH] from comment #3)
(In reply to Tim Nguyen :ntim from comment #2)
I think you mean LightweightThemeConsumer.jsm, not LightweightThemeManager.jsm :)
No, I do mean the Manager.
That also needs to be rewired on the Desktop version too, so it will be done at the same time.
|
|
Reporter | |
Updated•4 years ago
|
|
||
Updated•2 years ago
|
Description
•