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 414926 - [pending] Orca should announce the level of headings in Firefox
[pending] Orca should announce the level of headings in Firefox
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
2.17.x
Other All
: Normal enhancement
: 2.20.0
Assigned To: Willie Walker
Orca Maintainers
Depends on:
Blocks: 404403
 
 
Reported: 2007-03-05 14:31 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2008-07-22 19:26 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
Patch for speech (1.64 KB, patch)
2007-03-14 21:49 UTC, Willie Walker
committed Details | Review
patch to add braille (626 bytes, patch)
2007-04-06 02:39 UTC, Joanmarie Diggs (IRC: joanie)
none Details | Review
revised as per Will. (1.05 KB, patch)
2007-04-09 16:43 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review

Description Joanmarie Diggs (IRC: joanie) 2007-03-05 14:31:02 UTC
According to the Firefox spec:

> The letters "h" and "Shift+h" will be used to move between headings. 
> When  the heading is reached, orca will speak and braille the header followed
> by "heading and the heading level. 

We've implemented H/Shift+H, but Orca is not currently announcing the level.

Filing this as it as come up in recent discussion on the Orca list.
Comment 1 Willie Walker 2007-03-14 21:29:39 UTC
According to http://developer.mozilla.org/en/docs/Accessibility/ATSPI_Support, it looks like the attributes of a heading might contain some useful information for us:

Supports Text interface<br />  <br />  Supports object attribute "level", see <a href="#obj-attr">object attributes</a>  section ***

The easiest way to solve this might be to add checks to Gecko.py:getUtterancesFromContents  and Gecko.py:updateBraille to check for the heading role and then incorporate the level attribute if it exists (obj.attributes gets you the list of attributes).
Comment 2 Willie Walker 2007-03-14 21:49:14 UTC
Created attachment 84605 [details] [review]
Patch for speech

Here's a simple patch for speech. I'm not sure what to do about braille yet.
Comment 3 Mike Pedersen 2007-03-14 21:51:35 UTC
If this wouldn't be to much trouble putting "h" and then the number for the level at the end of the header line would be great.  
Comment 4 Joanmarie Diggs (IRC: joanie) 2007-03-28 00:02:05 UTC
Since we needed a GetHeadingLevel() for the new heading-by-level structural navigation, and since the functionality in the speech patch is quite helpful, I went ahead and checked in this patch.  Braille to follow soon!
Comment 5 Joanmarie Diggs (IRC: joanie) 2007-04-06 02:39:55 UTC
Created attachment 85876 [details] [review]
patch to add braille

This patch does exactly what Mike specified in his comment, namely:

> If this wouldn't be to much trouble putting "h" and then the number for the
> level at the end of the header line would be great.  

And, no, it wasn't any trouble at all. <smile>  It did raise a couple of questions in my mind, however:  

1. What, if anything, do we want to do w.r.t. localization?  Should  the "h" be translated so that it is the first letter of the language's word for "heading"?

2. Are their other roles besides links and headings for which additional information should be provided, be it the role itself or additional information such as the nesting level of a nested list?

Thanks!!
Comment 6 Willie Walker 2007-04-09 15:12:20 UTC
Cool!  Thanks!

> 1. What, if anything, do we want to do w.r.t. localization?  Should  the "h" be
> translated so that it is the first letter of the language's word for "heading"?

Probably should do something like:

# Translators: the 'h' below represents a heading level attribute for content 
# that you might find in something such as HTML content (e.g., <h1>).
# The translated form is mean to be a single character followed by a numeric
# heading level, where the single character is to indicate 'heading'.
#
regions.append(braille.Region(" " + + _("h%d" % level)))
Comment 7 Joanmarie Diggs (IRC: joanie) 2007-04-09 16:43:46 UTC
Created attachment 86054 [details] [review]
revised as per Will.

Thanks Will.  I checked this change in.
Comment 8 Willie Walker 2007-04-09 20:43:58 UTC
> Thanks Will.  I checked this change in.

Complete with typo ("mean to be" versus "meant to be").  D'Oh!  Sorry about that.  :-(
Comment 9 Joanmarie Diggs (IRC: joanie) 2007-04-10 00:09:12 UTC
My bad for not noticing it.  I fixed it when I was checking in all of the other changes to Gecko.py. :-)
Comment 10 Mike Pedersen 2007-04-13 16:40:36 UTC
This one looks good.  I think it is safe to close.
Comment 11 Willie Walker 2007-04-13 18:27:11 UTC
Closing as FIXED.  Thanks!