#4160 closed defect (fixed)
DescriptionButton in activity toolbar: text not visible
| Reported by: | erikos | Owned by: | manuq |
|---|---|---|---|
| Priority: | High | Milestone: | |
| Component: | Sugar | Version: | 0.97.x |
| Severity: | Major | Keywords: | olpc-test-passed |
| Cc: | Distribution/OS: | OLPC | |
| Bug Status: | Assigned |
Description
The text in a Palette is themed as white by default.
Possible fix:
diff --git a/gtk3/theme/gtk-widgets.css.em b/gtk3/theme/gtk-widgets.css.em
index 9bd6d01..f4b6c75 100644
--- a/gtk3/theme/gtk-widgets.css.em
+++ b/gtk3/theme/gtk-widgets.css.em
@@ -407,6 +407,10 @@ SugarPaletteWindowWidget {
color: @white;
}
+SugarPaletteWindowWidget .view {
+ color: @black;
+}
+
Change History (5)
comment:1 Changed 11 years ago by manuq
comment:2 Changed 11 years ago by erikos
- Keywords olpc-test-pending added
- Resolution set to fixed
- Status changed from new to closed
Pushed as 599c6730decba9c36dab4949dcb4d350898bc12d
comment:3 Changed 11 years ago by greenfeld
- Keywords olpc-test-passed added; olpc-test-pending removed
I have not noticed any oddly colored text in activities for the past few months, and I believe all the kludges we had added to various activities to theme themselves better were removed.
comment:4 Changed 11 years ago by dnarvaez
- Component changed from sugar-artwork to Sugar
Note: See
TracTickets for help on using
tickets.
Good catch!
This was regressed in b2b0f261 "Style insensitive text - SL #4139", when I removed the color rule from .view {} . I think that change is good and palettes should set it, as your patch does. Please push.