709718 - Vertical lines between off-time hours have the wrong color

Status: RESOLVED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

(In reply to Decathlon from bug 698760 comment 8)
> In the rotated view, in the time bar, the vertical lines between the "off-time" 
> hours are white (or missing) instead of gray.

These lines should be fixed and given an appropriate color.

Comment 1

[Richard Marti (:Paenglab)]

Attachment: Give back the color

With removing the -moz-border-right-colors the right color is shown again.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: Give back the color v2

Unbitrotted patch

Comment 3

[Decathlon]

Comment on attachment 586731 [details] [diff] [review]
Give back the color v2

The same should be done for the code inside the selectors with the window[systemcolors] rule (maybe a separate patch for comm-central?)

I've also noticed that in rotate view and with the setting for the optimized color activated, there are some lines with different colors (see next screenshot). An issue from bug 671119, but maybe you could fix it here without opening another bug.

Comment 4

[Decathlon]

Attachment: some different lines in rotate view with optimized colors setting activated

Comment 5

[Richard Marti (:Paenglab)]

Pushed to comm-central: http://hg.mozilla.org/comm-central/rev/1ab3229278a6

Comment 6

[Richard Marti (:Paenglab)]

(In reply to Decathlon from comment #3)
> Comment on attachment 586731 [details] [diff] [review]
> Give back the color v2
> 
> The same should be done for the code inside the selectors with the
> window[systemcolors] rule (maybe a separate patch for comm-central?)

Fixed with the check-in.

(In reply to Decathlon from comment #4)
> Created attachment 586807 [details]
> some different lines in rotate view with optimized colors setting activated

Also fixed.

Comment 7

[Richard Marti (:Paenglab)]

Comment on attachment 581313 [details] [diff] [review]
Give back the color

Do you give approval for aurora and maybe beta (only for the initial issue)?

Bug 698760 who introduced this issue has Target Milestone: 1.0b8

Comment 8

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 581313 [details] [diff] [review]
Give back the color

Since this is super low risk, lets take it for beta.

Comment 9

[Richard Marti (:Paenglab)]

Pushed to comm-aurora: <http://hg.mozilla.org/releases/comm-aurora/rev/9907a8d0e3a6>

Comment 10

[Richard Marti (:Paenglab)]

Pushed to comm-beta: <http://hg.mozilla.org/releases/comm-beta/rev/9f92db34aff0>