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 354460 - [requirement] Military spelling
[requirement] Military spelling
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: speech
1.0.x
Other All
: Normal normal
: 2.18.0
Assigned To: Rich Burridge
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-09-05 16:04 UTC by Willie Walker
Modified: 2008-07-22 19:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to implement this feature. (8.60 KB, patch)
2006-09-18 18:27 UTC, Rich Burridge
none Details | Review
Patch to fix the new problem. (865 bytes, patch)
2006-10-16 21:36 UTC, Rich Burridge
none Details | Review

Description Willie Walker 2006-09-05 16:04:34 UTC
Orca should offer military spelling (e.g., "alpha" "bravo" "charlie") as an option.  For more information, see: http://cvs.gnome.org/viewcvs/*checkout*/orca/docs/doc-set/orca.html#URSPEECH
Comment 1 Rich Burridge 2006-09-05 16:18:24 UTC
Tracking.
Comment 2 Rich Burridge 2006-09-07 19:50:22 UTC
There doesn't appear to be anything under:
http://cvs.gnome.org/viewcvs/*checkout*/orca/docs/doc-set/orca.html#URSPEECH
that specifies when you should spell in military mode as opposed to normal
mode.

How should this be done?
Comment 3 Rich Burridge 2006-09-13 17:01:24 UTC
From:

http://www.faqs.org/faqs/radio/phonetic-alph/full/

the phonetic alphabet is:

Alpha Bravo Charlie Delta Echo Foxtrot Golf Hotel India
Juliet Kilo Lima Mike November Oscar Papa Quebec Romeo
Sierra Tango Uniform Victor Whiskey Xray Yankee Zulu
Comment 4 Willie Walker 2006-09-13 20:13:37 UTC
Interesting URL!  The thing I see as being necessary is some way to handle other languages that include more than just A-Z.  For example, the Swedish entry looks like it might have 29 entries. Gnopernicus does its work in speech/libsrs/srs-speech.c, but it seems to limit things to A-Z.  I'm not sure this is really what users want/need, though, since I believe the goal of "military spelling" for screen readers is a bit larger than the original goal of real military spelling.  That is, I believe it is like military spelling in that helps disambiguate characters that may be otherwise confusable when spoken via TTS (e.g., "s" and "f", "p" "t" and "d"), but the screen reader case may cover the complete alphabet for the language.
Comment 5 Rich Burridge 2006-09-18 18:27:44 UTC
Created attachment 72994 [details] [review]
Patch to implement this feature.

The attached patch does the following:

* Replaces the util.isDoubleClick() routine with util.getClickCount
  which return the count of the number of clicks a user has made to 
  one of the keys on the keyboard.
* Implement phonetic (military) spelling on a triple-click for
  Numpad-2 (flat review current character)
  Numpad-5 (flat review current word)
  Numpad-8 (flat review current line)
  This includes using the UPPERCASE_VOICE for upper-case letters.
* Fixes up a traceback is _reviewCurrentCharacter() if string was None.

Double-click will continue to ordinarily spell the character/word/line.
Four or more clicks will be treated as if the user had just done a single
click.
Comment 6 Rich Burridge 2006-09-18 18:34:01 UTC
Changes checked into CVS HEAD. At the moment it will do
phonetic spelling of a-z and A-Z. Any other characters it tries
to phonetically spell will just be spoken as normal. 

This is on a par with gnopernicus so I'm closing as FIXED.
Comment 7 Rich Burridge 2006-09-18 18:35:49 UTC
Oh, the other thing I should add is that I've only tested this 
with gedit where it seems to work nicely.

OpenOffice and Evolution might be another story. If you find 
problems with them, please file fresh bugs and I'll take a look 
at them.
Comment 8 Mike Pedersen 2006-10-16 19:48:45 UTC
After working with this feature for a while having this functionality available on most flat review keys really isn't useful.  I'd like to have it restricted to the "say-line, say word, and say character keys"  ie numpad 8, 5, and 2.
Comment 9 Rich Burridge 2006-10-16 20:33:01 UTC
Just to clarify, where *don't* you want military spelling
that currently has it? 

I ask because the calls to where phonetic spelling is done, 
are in _reviewCurrentLine(), _reviewCurrentItem() and 
_reviewCurrentCharacter() in default.py; methods that are called
from Numpad-[7,8,9], Numpad-[4,5,6] and Numpad-[1,2,3].

It's probably possible to make it work for just Numpad-[8,5,2]
but the code might be ugly.
 
Comment 10 Mike Pedersen 2006-10-16 21:04:43 UTC
The problem is when you do for example prior or next character really quickly you get military spelling when you really just wanted the next character.  
Comment 11 Rich Burridge 2006-10-16 21:36:26 UTC
Created attachment 74842 [details] [review]
Patch to fix the new problem.

The only two places where this was broken was in review previous or
next characters. Review preview or next word or line was doing the 
right thing.
Comment 12 Rich Burridge 2006-10-16 21:39:31 UTC
New changes applied to CVS HEAD. Closing as fixed (again ;-) ).