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 512303 - (ff3) table captions are not being presented
(ff3) table captions are not being presented
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.21.x
Other All
: Normal normal
: 2.22.0
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
: 512309 (view as bug list)
Depends on:
Blocks: 404403
 
 
Reported: 2008-01-26 22:59 UTC by Mesar Hameed
Modified: 2008-07-22 19:33 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
caption_testcase (515 bytes, text/html)
2008-01-26 23:01 UTC, Mesar Hameed
  Details
revision 1 (1.50 KB, patch)
2008-01-27 01:51 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
revision 2 (1.94 KB, patch)
2008-01-27 23:45 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Mesar Hameed 2008-01-26 22:59:54 UTC
Please describe the problem:
Please see attached example.
I can just about see that the caption is being displayed (sorry sight is v poor)
but orca isnt announcing it.
Obviously this is a contrived example, but there are tables out there with useful captions.


Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?
yes

Other information:
Comment 1 Mesar Hameed 2008-01-26 23:01:45 UTC
Created attachment 103790 [details]
caption_testcase
Comment 2 Joanmarie Diggs (IRC: joanie) 2008-01-27 01:47:42 UTC
*** Bug 512309 has been marked as a duplicate of this bug. ***
Comment 3 Joanmarie Diggs (IRC: joanie) 2008-01-27 01:51:46 UTC
Created attachment 103797 [details] [review]
revision 1

Please test.  Thanks!
Comment 4 Mesar Hameed 2008-01-27 19:07:23 UTC
Thanks Joanie,
now we are hearing: <captiontext> caption
and on the braille display: <captiontext>
maybe we should indicate that it is a table caption in the braille too?
Thanks

PS. sorry for dubble bugging, i wasnt sure if the two were related or not.
Comment 5 Joanmarie Diggs (IRC: joanie) 2008-01-27 20:46:17 UTC
Thanks for testing Jon!

> maybe we should indicate that it is a table caption in the braille too?

Sounds good to me.  But I defer to our UI guy on such matters.  Mike, thoughts?

If we agree that this should be done, I'll open a new bug for it as I suspect it will be an Orca-wide braille generator change (as opposed to a Gecko.py change).
 
> PS. sorry for dubble bugging, i wasnt sure if the two were related or not.

Hey, no problem.  To be perfectly honest, I didn't know if they were related or not until I fixed this one and the other one went away. <smile>   Gosh I like it when that happens.... A lot.... Besides, I'm starting to agree with Rich that one bug per issue is far preferable to a large uber bug.  

In that spirit, does the patch fix your reported bug? <smile>
Comment 6 Mesar Hameed 2008-01-27 20:56:09 UTC
indeed it fixes the reported bug. :)
I'll lookout for the corresponding braille bug.
Thank you for the good work.
Comment 7 Joanmarie Diggs (IRC: joanie) 2008-01-27 23:45:18 UTC
Created attachment 103845 [details] [review]
revision 2

Heh.  Turned out it was a Gecko.py thing.  When arrowing around in a document, we typically don't tack on rolenames in the braille.  We have special handling for headings and links.  This patch adds special handling for captions.

Please test.
Comment 8 Mesar Hameed 2008-01-28 00:02:07 UTC
excellent.
Both abbriviated and non-abbriviated braille are working fine.

you truely squashed this one :)
Comment 9 Mike Pedersen 2008-01-28 21:54:29 UTC
This is great.  It will be interesting to see how many sites have these captions that I never noticed.  
Comment 10 Joanmarie Diggs (IRC: joanie) 2008-01-28 22:41:25 UTC
Thanks guys.  Patch committed.  Moving to pending.