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 560466 - Improve the accuracy of label guess for Firefox 3
Improve the accuracy of label guess for Firefox 3
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.24.x
Other All
: Normal normal
: 2.24.2
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
: 517759 (view as bug list)
Depends on:
Blocks: 404403
 
 
Reported: 2008-11-12 06:55 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-11-17 22:34 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
revision 1 (21.98 KB, patch)
2008-11-12 07:01 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2008-11-12 06:55:20 UTC
Orca's label guess functionality should:

1. Not guess form fields as "labels" for other form fields.

2. Handle "labels" which are composed of multiple objects such as text which contains links (Orca does this already for items on the same line, but not for items on different lines).

3. Use the strings already obtained for the next/previous line.

4. Better handle "labels" which are graphics with alt text in a table/grid.
Comment 1 Joanmarie Diggs (IRC: joanie) 2008-11-12 07:01:02 UTC
Created attachment 122474 [details] [review]
revision 1

Pylinted and regression tested with updated regression tests to remove 'BUG?'s.

This also gives us a slight performance improvement (not much, but every little bit helps....):

Bugzilla Advanced Search:

ncalls  tottime  percall  cumtime  percall
    50    0.026    0.001    9.593    0.192 guessTheLabel-before
    48    0.029    0.001    6.723    0.140 guessTheLabel-after

Entries regression test:

ncalls  tottime  percall  cumtime  percall
    39    0.027    0.001    6.472    0.166 guessTheLabel-before
    39    0.029    0.001    4.760    0.122 guessTheLabel-after

Mike, please test. Thanks!
Comment 2 Joanmarie Diggs (IRC: joanie) 2008-11-12 07:02:44 UTC
*** Bug 517759 has been marked as a duplicate of this bug. ***
Comment 3 Mike Pedersen 2008-11-13 21:10:54 UTC
So far this one looks good. I'd be in favor of getting it in trunk for wider testing. 
Comment 4 Joanmarie Diggs (IRC: joanie) 2008-11-13 22:15:01 UTC
Thanks Mike. Patch committed to trunk. Moving to pending.
Comment 5 Mike Pedersen 2008-11-17 21:53:14 UTC
I have been doing quite a bit of testing with this patch.  It seems to work well.  Joanie thought it might be impacting another problem I was having but it is not so I think it is good.  
Comment 6 Joanmarie Diggs (IRC: joanie) 2008-11-17 22:09:43 UTC
Thanks Mike.

What's the verdict on branch? Commit or not?
Comment 7 Mike Pedersen 2008-11-17 22:20:30 UTC
Were it me I think I'd add it to the branch as well.  
Comment 8 Joanmarie Diggs (IRC: joanie) 2008-11-17 22:34:02 UTC
Okie dokie. Patch committed to the gnome-2-24 branch. Closing as FIXED.