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 525644 - Pidgin buddy status changes cause Orca to display "cell" in braille
Pidgin buddy status changes cause Orca to display "cell" in braille
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: braille
2.21.x
Other All
: Normal normal
: 2.22.2
Assigned To: Rich Burridge
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-04-01 20:36 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-04-19 17:14 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22

Attachments
Revision #1. (2.52 KB, patch)
2008-04-07 20:43 UTC, Rich Burridge 		committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2008-04-01 20:36:26 UTC 
Steps to reproduce:

1. Launch Pidgin

2. Give focus to a buddy

3. Give focus to some other app and wait until the buddy changes status.

Expected results:  The braille display would continue to display what it was displaying in the app with focus.

Actual results: The braille display displays "cell"
Comment 1 Joanmarie Diggs (IRC: joanie) 2008-04-01 22:16:43 UTC 
(Assigned the wrong bug to Rich.  Sorry for the spam.)
Comment 2 Rich Burridge 2008-04-07 20:43:12 UTC 
Created attachment 108810 [details] [review]
Revision #1.

This turned out to be a little more involved than I expected.

There were four different kinds of events that could cause
"... Cell" to be written to the braille display:

object:text-changed:insert
object:property-change:accessible-value
object:property-change:accessible-name
object:text-changed:delete

I had to add new methods for the last three to the gaim.py script.

In all four methods, we now look for a role hierarchy that indicates
that this event is for a buddy list table cell. If it's found, then
we just return, otherwise we pass the event onto the parent class.

Patch not committed yet. Please test.
Comment 3 Joanmarie Diggs (IRC: joanie) 2008-04-07 21:34:26 UTC 
So far so good on the testing.
Comment 4 Mike Pedersen 2008-04-07 21:47:30 UTC 
I've been running this for about an hour and so far I'm liking it.
Comment 5 Willie Walker 2008-04-15 20:47:25 UTC 
Given that Mike and Joanie have tested this, feel free to commit!  Thanks!
Comment 6 Rich Burridge 2008-04-19 17:14:23 UTC 
Patch committed to SVN trunk and gnome-2-22 branch. Closing as FIXED. Thanks.