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 489028 - Remove annotations of accessible objects in Gecko.py
Remove annotations of accessible objects in Gecko.py
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.20.x
Other All
: Normal normal
: 2.22.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks: 404403 476639
 
 
Reported: 2007-10-22 14:29 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:32 UTC
See Also:
GNOME target: ---
GNOME version: 2.19/2.20


Attachments
first attempt at removing obj.characterOffsetInParent (2.42 KB, patch)
2007-10-22 14:35 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
the "don't read docs literally" version (2.44 KB, patch)
2007-10-22 15:00 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review
remove obj.childrenIndices (1.49 KB, patch)
2007-10-22 16:19 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2007-10-22 14:29:13 UTC
As part of the pyatspi migration, we need to eliminate:

* obj.characterOffsetInParent
* obj.childrenIndices

(obj.unicodeText is now gone)
Comment 1 Joanmarie Diggs (IRC: joanie) 2007-10-22 14:35:35 UTC
Created attachment 97631 [details] [review]
first attempt at removing obj.characterOffsetInParent

According to the Mozilla docs (http://developer.mozilla.org/en/docs/Accessibility:Architecture):

(B) To get the offset in the parent text for a given embedded object:

   1. QI/QA to IAHyperLink
   2. If not successful, then it is not embedded in text, so it's position in parent should be determined just by IA2::indexInParent, which will return it's child offset within the parent
   3. otherwise, use IAHyperLink::getStartIndex() to find the index in parent.
Comment 2 Joanmarie Diggs (IRC: joanie) 2007-10-22 14:41:24 UTC
Ignore the above for now.  It *seemed* to work, but something is wrong.... :-(
Comment 3 Joanmarie Diggs (IRC: joanie) 2007-10-22 15:00:38 UTC
Created attachment 97634 [details] [review]
the "don't read docs literally" version

If the hyperlink interface is not implemented (e.g. in a combo box or a list), we want to return -1.  D'oh!

Will, does this patch seem correct?
Comment 4 Willie Walker 2007-10-22 15:34:18 UTC
(In reply to comment #3)
> Created an attachment (id=97634) [edit]
> the "don't read docs literally" version
> 
> If the hyperlink interface is not implemented (e.g. in a combo box or a list),
> we want to return -1.  D'oh!
> 
> Will, does this patch seem correct?
> 

Yep!  Now that the Gecko bug seems to be fixed, this is the much better way to go.  Thanks!
Comment 5 Joanmarie Diggs (IRC: joanie) 2007-10-22 16:19:13 UTC
Created attachment 97642 [details] [review]
remove obj.childrenIndices

Will please review.
Comment 6 Willie Walker 2007-10-22 16:27:16 UTC
Looks good.  Thanks!
Comment 7 Joanmarie Diggs (IRC: joanie) 2007-10-22 16:50:51 UTC
Both committed.  Closing as FIXED.