Unneeded preprocessing of messengerLWTheme.css

RESOLVED FIXED in Thunderbird 43.0

Status

Thunderbird
Theme
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 43.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
messengerLWTheme.css is preprocessed to include linuxShared.css/osxShared.css/windowsShared.css. But this included files are copied to messenger/skin/sharedPlatform.css and the with a @import added in tabmail.css. This makes this preprocessing unneeded.
(Assignee)

Comment 1

2 years ago
Created attachment 8645880 [details] [diff] [review]
noLWThemeProcessing.patch

Remove the preprocessing and move the file to shared as it is the same for all platforms.

This patch needs the aero removal patches applied first.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8645880 - Flags: review?(aleth)

Updated

2 years ago
Attachment #8645880 - Flags: review?(aleth) → review+

Updated

2 years ago
Keywords: checkin-needed

Comment 2

2 years ago
url:        https://hg.mozilla.org/comm-central/rev/2a346480be3947faa5b50f66e6513f04359d3276
changeset:  2a346480be3947faa5b50f66e6513f04359d3276
user:       Richard Marti <richard.marti@gmail.com>
date:       Mon Aug 10 20:57:04 2015 +0200
description:
Bug 1192969 - Remove unneeded preprocessing of messengerLWTheme.css r=aleth

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0

Comment 3

2 years ago
Could this patch somehow cause bug 1193143?
You need to log in before you can comment on or make changes to this bug.