Warning: Error in parsing value for 'background-color Source File: chrome://messenger/content/messengercompose/messengercompose.xul

RESOLVED FIXED in Thunderbird 49.0

Status

Thunderbird
Theme
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 49.0

Thunderbird Tracking Flags

(thunderbird46 wontfix, thunderbird47 fixed, thunderbird48 fixed, thunderbird49 fixed, thunderbird_esr4547+ fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

a year ago
This warning appears when opening a message in a window. The strange thing is, this happens only on Linux although Windows is doing the same.
(Assignee)

Comment 1

a year ago
Created attachment 8757694 [details] [diff] [review]
variableFix.patch

Add sharedPlatform.css to messageWindow.css. OS X is not affected by this issue because it doesn't use variables here. I also moved the toolbars code from messenger.css to primaryToolbar.css to have the rules in the same files as OS X and Windows.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8757694 - Flags: review?(acelists)

Comment 2

a year ago
Comment on attachment 8757694 [details] [diff] [review]
variableFix.patch

Review of attachment 8757694 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, this solves the warnings for me.

Could you link in sharedPlatform.css on OS X too, for consistency, even if it is not used yet?
Attachment #8757694 - Flags: review?(acelists) → review+
(Assignee)

Comment 3

a year ago
Created attachment 8757702 [details] [diff] [review]
variableFix.patch v2

(In reply to :aceman from comment #2)
> Comment on attachment 8757694 [details] [diff] [review]
> variableFix.patch
> 
> Review of attachment 8757694 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Thanks, this solves the warnings for me.
> 
> Could you link in sharedPlatform.css on OS X too, for consistency, even if
> it is not used yet?

Then it's simpler to add it in the XUL file.
Attachment #8757694 - Attachment is obsolete: true
Attachment #8757702 - Flags: review?(acelists)

Comment 4

a year ago
Can't it be imported into primaryToolbar.css where it is used? (The first patch imported into messageWindow.css).
(Assignee)

Comment 5

a year ago
Then it's two times imported on main window, in tabmail.css and primaryToolbar.css. messageWindow.css/xul is only used for the standalone window where is no tabmail.css (this is why I wrote, it's strange why it worked on Windows).

I'm planning to remove the sharedPlatform.css and move the variables to messenger.css. But I'm planning to uplift this bug and the planned changes are maybe to risky to made them now.

Comment 6

a year ago
Comment on attachment 8757702 [details] [diff] [review]
variableFix.patch v2

Review of attachment 8757702 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Attachment #8757702 - Flags: review?(acelists) → review+
(Assignee)

Comment 7

a year ago
http://hg.mozilla.org/comm-central/rev/80cff8d20b1b
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 49.0
(Assignee)

Comment 8

a year ago
Comment on attachment 8757702 [details] [diff] [review]
variableFix.patch v2

[Approval Request Comment]
User impact if declined: incorrect toolbar background color on standalone window with LW-theme enabled
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low
Attachment #8757702 - Flags: approval-comm-esr45?
Attachment #8757702 - Flags: approval-comm-beta?
Attachment #8757702 - Flags: approval-comm-aurora?

Updated

a year ago
Attachment #8757702 - Flags: approval-comm-aurora? → approval-comm-aurora+

Comment 9

a year ago
Aurora (TB 48):
https://hg.mozilla.org/releases/comm-aurora/rev/65bf40a3d665
status-thunderbird46: --- → wontfix
status-thunderbird47: --- → affected
status-thunderbird48: --- → fixed
status-thunderbird49: --- → fixed
status-thunderbird_esr45: --- → affected
tracking-thunderbird_esr45: --- → ?
Comment on attachment 8757702 [details] [diff] [review]
variableFix.patch v2

http://hg.mozilla.org/releases/comm-beta/rev/400a26ec0828
Attachment #8757702 - Flags: approval-comm-beta? → approval-comm-beta+

Updated

a year ago
status-thunderbird47: affected → fixed
Comment on attachment 8757702 [details] [diff] [review]
variableFix.patch v2

http://hg.mozilla.org/releases/comm-esr45/rev/b050956b8c55
Attachment #8757702 - Flags: approval-comm-esr45? → approval-comm-esr45+

Updated

a year ago
status-thunderbird_esr45: affected → fixed
tracking-thunderbird_esr45: ? → 47+
You need to log in before you can comment on or make changes to this bug.