Issue 39175 - icons and statusbar not compatible with high-contrast theme
Summary: icons and statusbar not compatible with high-contrast theme
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: 680m65
Hardware: All Linux, all
: P2 Trivial (vote)
Target Milestone: OOo 2.0
Assignee: eric.savary
QA Contact: issues@framework
URL:
Keywords: accessibility, oooqa, regression
Depends on:
Blocks:
 
Reported: 2004-12-16 23:59 UTC by lohmaier
Modified: 2005-05-04 10:32 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
screenshot showin OOo with dark high-contrast theme (62.04 KB, image/png)
2004-12-17 00:01 UTC, lohmaier 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description lohmaier 2004-12-16 23:59:44 UTC 
Switch gnome to high-contrast mode (inverted).

-> OOo's toolbar icons don't change (except icons in stylist)
Esp. icons for text-alignment and font-effects are unreadable.

furthermore the font-color in the statusbar stays black -> unreadable.

please see attached screenshot. (Shows OOo 1.1.3 as well for comparison)
Comment 1 lohmaier 2004-12-17 00:01:05 UTC 
Created attachment 20593 [details]
screenshot showin OOo with dark high-contrast theme
Comment 2 lohmaier 2004-12-17 00:02:38 UTC 
setting keywords.
Comment 3 stefan.baltzer 2004-12-17 10:03:48 UTC 
SBA: Component set to "Framework", target set to OOo 2.0. Reassigned to ES.
Comment 4 eric.savary 2005-02-04 14:29:06 UTC 
Same result on unxsoli with Gnome.
Comment 5 philipp.lohmann 2005-02-09 18:10:00 UTC 
This was framework code; toolbars didn't take the correct background, statusbar
didn't let its base class know about DataChanged.

cd: please have a look at my changes to
framework/source/uielement/toolbarmanager.cxx and statusbar.cxx

fixed in CWS vcl36
Comment 6 carsten.driesner 2005-02-10 08:02:17 UTC 
cd->pl: You are right that my little status bar wrapper forgot to call the base
class, but why we now have to use GetDisplayBackground() instead of
GetBackground()? I checked our old code for OOo 1.1.x and we also used there
GetBackground().
Comment 7 philipp.lohmann 2005-02-10 10:03:56 UTC 
Because since native widgets toolbars are transparent. The only reason why the
code still worked on Windows is that the accessibility theme is an "Old style"
(not Luna) theme so that the toolbars are not themed in OOo and therefore not
transparent.
Comment 8 philipp.lohmann 2005-02-16 14:38:06 UTC 
please verify in CWS vcl36
re-open issue and try to reassign to es@openoffice.org
Comment 9 philipp.lohmann 2005-02-16 14:38:13 UTC 
try to reassign to es@openoffice.org
Comment 10 philipp.lohmann 2005-02-16 14:38:17 UTC 
try to reset resolution to FIXED
Comment 11 eric.savary 2005-03-15 00:14:06 UTC 
verified in CWS vcl36
Comment 12 eric.savary 2005-05-04 10:32:15 UTC 
closed