After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 505306 - Orca's color filtering combo box should be greyed out if the filters are not available
Orca's color filtering combo box should be greyed out if the filters are not ...
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: magnification
2.21.x
Other All
: Normal enhancement
: 2.22.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on: 502893
Blocks:
 
 
Reported: 2007-12-23 21:40 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:33 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
revision 1 (4.15 KB, patch)
2007-12-23 21:43 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2007-12-23 21:40:14 UTC
Our new Advanced Settings dialog has a combo box in which a user can specify a color filter from libcolorblind.  Currently we have no way of knowing whether a user has libcolorblind installed or not, therefore the control is always available, but doesn't always work.

We should grey it out if the color filters are not available.  We need this information from gnome-mag (see bug 502893).
Comment 1 Joanmarie Diggs (IRC: joanie) 2007-12-23 21:43:39 UTC
Created attachment 101522 [details] [review]
revision 1

Carlos has attached a patch to the gnome-mag bug we're depending on. (See http://bugzilla.gnome.org/attachment.cgi?id=101399&action=view)

This patch has been tested with gnome-mag with and without Carlos' patch and with and without libcolorblind installed.  It seems to be doing the right thing.

Please test.

(Rich, please look over the changes I made to the setup dialog.  Thanks!)
Comment 2 Rich Burridge 2008-01-02 18:04:56 UTC
> Please test.

This seems to work just fine.

> (Rich, please look over the changes I made to the setup dialog.  Thanks!)

Changes look good. Thanks for using glade-2 !
Comment 3 Joanmarie Diggs (IRC: joanie) 2008-01-02 19:56:35 UTC
Thanks Rich.

> Changes look good. Thanks for using glade-2 !

;-)

Patch committed.  Moving to pending. 

Comment 4 Willie Walker 2008-01-04 22:51:32 UTC
I tested this patch with gnome-mag with and without Carlos' patch and
with and without libcolorblind installed and the combo always comes up greyed.  I think my colorblind build is bad, though, since I cannot run the colorblind applet.  If this works as expected for you, I think it's OK to close this bug as FIXED.
Comment 5 Joanmarie Diggs (IRC: joanie) 2008-01-04 23:28:40 UTC
I've never been able to run the colorblind applet either.

The order of everything matters though.  You have to first build libcolorblind.  Then you have to (re)autogen.sh gnome-mag (as that's when it looks for libcolorblind) and rebuild it. I'll give it a try in Solaris.

In terms of Hardy, it did work as expected until gnome-mag stopped building in Hardy. <shrugs>
Comment 6 Willie Walker 2008-01-05 01:43:33 UTC
(In reply to comment #5)
> I've never been able to run the colorblind applet either.

Ha!  I was able to add it to the panel and then it magically appeared when I rebooted my machine.  It takes over Shift+Ctrl+c to enable filtering and Shift+Ctrl+n to switch between filters. I can run this just fine when Orca is running, and when I do so, the filters take effect in the magnifier.  But, the combo still comes up greyed for me.  In any case, the fact that the applet works makes me wonder if we really need this in the Orca preferences UI.  What do you think?
Comment 7 Joanmarie Diggs (IRC: joanie) 2008-01-08 20:57:07 UTC
As discussed in team meeting, we are going to keep it in the Orca preferences UI.  Closing this bug as FIXED.