Comment 2 for bug 1296866

Revision history for this message
Brett Bogert (bbogert24) wrote :

Khurshid Alam is correct. According to the Unity theming documenation for Trusty 14.04 the panel text should be set by the UnityPanelWidget as follows:

UnityPanelWidget,
.unity-panel {
    background-color: shade (@dark_bg_color, 1.5);

    color: @dark_fg_color; /* Text color */ <--- ***SHOULD BE SET HERE***
 }

but it is actually being set by the UnityDecoration "top" code here:

/* This will theme the top decoration, so the whole space above the window */
UnityDecoration.top {
    border: 1px solid shade (@bg_color, 0.5);
    border-bottom-width: 0;
    border-radius: 8px 8px 0 0; /* Corner radius, only the top ones should be */
    padding: 1px 8px 0 8px; /* This padding will be applied to the content of the top layout */
    background-color: shade (@dark_bg_color, 1.5); /* Decoration background */
    color: @dark_fg_color; /* The foreground color will be used to paint the text */ <--- ***ACTUALLY BEING SET HERE ***
    text-shadow: 1px 0 #333, -1px 0 #333, 0 1px #333, 0 -1px #333;
}

This makes it impossible to use a dark panel with light window treatment as the title color for light window treatment is dark making the panel label (e.g. "Ubuntu Desktop") and maximized title unreadable on a dark panel.

In short the panel text color should be set by the unity panel widget class not by the unity decoration class.

Since this problem is already bug reported I will not open another bug report but track progress under this bug report number.

Thanks,
B.Bogert