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 511468 - Ekiga chat window accessibility problem.
Ekiga chat window accessibility problem.
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
0.2.x
Other opensolaris
: Normal normal
: 2.26.3
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
: 583947 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-01-23 07:17 UTC by Tim Miao
Modified: 2009-05-27 01:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed (interim) fix - version 1 (3.21 KB, patch)
2009-05-22 18:45 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Tim Miao 2008-01-23 07:17:29 UTC
Steps to reproduce:
1. Launch ekiga and setup a h323 chat.
2. Invoke screen reader, orca.
3. Chat with another setup ekiga.

Bug observations:
When user typing words in his chatting window, the message sent by other can not be read by default, user has to change focus to the chat pane and get the message. If user did not change focus position, it's impossible to get informed that there is a new message coming.
Comment 1 Willie Walker 2008-01-23 14:01:10 UTC
This may call for a script in Orca, much like what was done for GAIM/Pidgin.
Comment 2 Damien Sandras 2008-09-14 18:54:05 UTC
Can you propose a patch? Is it still current in 3.00 ?
Comment 3 Willie Walker 2008-09-15 12:52:34 UTC
The Orca team hasn't had a chance to look into these issues yet, but my hopes are that we can handle this in a script in Orca.  Reassigning this to Orca for now.
Comment 4 Joanmarie Diggs (IRC: joanie) 2009-05-19 00:19:53 UTC
Retargeting for 2.27.3 since I'm now looking more closely at Orca's support for Ekiga.
Comment 5 Joanmarie Diggs (IRC: joanie) 2009-05-22 18:45:20 UTC
Created attachment 135192 [details] [review]
proposed (interim) fix - version 1

1. It seems that we are not getting any text-inserted events initially. "Tickling" the hierarchy changes that. Go figure....

2. We need special handling to present the text in the chat history.

This patch addresses both of the above.

Also, please note that this patch does NOT add in all the functionality (e.g. chat room history, distinguishing between focused and non-focused chats, etc., etc.) we have in pidgin. It just gets the messages speaking -- which is better than what we currently have.

Will, please review. Thanks! Also, thoughts for inclusion in 2.26.3?
Comment 6 Willie Walker 2009-05-24 14:19:43 UTC
(In reply to comment #5)
> Created an attachment (id=135192) [edit]
> proposed (interim) fix - version 1
> 
> 1. It seems that we are not getting any text-inserted events initially.
> "Tickling" the hierarchy changes that. Go figure....
> 
> 2. We need special handling to present the text in the chat history.
> 
> This patch addresses both of the above.
> 
> Also, please note that this patch does NOT add in all the functionality (e.g.
> chat room history, distinguishing between focused and non-focused chats, etc.,
> etc.) we have in pidgin. It just gets the messages speaking -- which is better
> than what we currently have.
> 
> Will, please review. Thanks! Also, thoughts for inclusion in 2.26.3?

This is simple and isolated to the Ekiga script.  I think it looks great and improves Ekiga nicely.  Definitely go for master and gnome-2-26.  Thanks!
Comment 7 Joanmarie Diggs (IRC: joanie) 2009-05-24 16:45:32 UTC
Thanks Will. Patch committed to both master and the gnome-2-26 branch. Closing as FIXED.
Comment 8 Joanmarie Diggs (IRC: joanie) 2009-05-26 23:01:25 UTC
Jacob reported on the Orca list that this fix causes us to speak the call duration constantly. I was able to reproduce this issue using the Echo test. What distinguishes the two text objects is that the chat history's parent is a scroll pane. Therefore the following one-line change should solve the problem:

~~~~~~
--- a/src/orca/scripts/apps/ekiga.py
+++ b/src/orca/scripts/apps/ekiga.py
@@ -58,7 +58,8 @@ class Script(default.Script):
         - obj: the accessible object to examine.
         """
 
-        if obj and obj.getRole() == pyatspi.ROLE_TEXT:
+        if obj and obj.getRole() == pyatspi.ROLE_TEXT \
+           and obj.parent.getRole() == pyatspi.ROLE_SCROLL_PANE:
             state = obj.getState()
             if not state.contains(pyatspi.STATE_EDITABLE) \
                and state.contains(pyatspi.STATE_MULTI_LINE):

~~~~~~

Because this is a safe, one-line change which solves the problem, is limited to the Ekiga script, is pylinted, yadda, yadda, yadda, I've gone ahead and committed this change both the the gnome-2-26 branch and to master.
Comment 9 Joanmarie Diggs (IRC: joanie) 2009-05-27 01:16:55 UTC
*** Bug 583947 has been marked as a duplicate of this bug. ***