To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=35482





------- Additional comments from [email protected] Tue Jul 14 
16:32:08 +0000 2009 -------
I think that would be fine if the HC option were enabled by default. If someone
has a dark theme, they can disable it.

"Application::GetSettings().GetStyleSettings().GetHighContrastMode()" should
then be used in place of every current instance of "BackgroundColor.IsDark".
GetHighContrastMode() should check "BackgroundColor.IsDark" if and only if the
HC flag -- enabled by the "Automatically detect high contrast mode of operating
system" option -- is on. At this point, if BackgroundColor.IsDark returns below
the acceptable threshold, then is the OOo HC mode enabled by returning true from
GetHighContrastMode().

To save cycles, you might be able to just check BackgroundColor.IsDark once on
startup (and when the HC Flag changes) and set someother flag that
GetHighContrastMode() would use to check this.

Granted, I have not looked at the code base (and so maybe everything I've said
here is already clear and I didn't need to bother saying anything), but would
doing something like this work?:
sed -e
"s/BackgroundColor.IsDark/GetSettings().GetStyleSettings().GetHighContrastMode()/g"
*

Where would that not work? Why else would the BackgroundColor.IsDark be used
that is not an accessibility thing?

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to