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 303930 - Speech is same for Lower or Upper Case
Speech is same for Lower or Upper Case
Status: RESOLVED FIXED
Product: gnopernicus
Classification: Deprecated
Component: speech
0.10.x
Other All
: Normal normal
: ---
Assigned To: remus draica
remus draica
AP2
Depends on:
Blocks:
 
 
Reported: 2005-05-12 15:50 UTC by tvuong
Modified: 2005-06-27 09:13 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
proposed patch (756 bytes, patch)
2005-05-17 08:17 UTC, remus draica
reviewed Details | Review
Proposed patch (798 bytes, patch)
2005-06-14 10:24 UTC, Oana Serb
committed Details | Review

Description tvuong 2005-05-12 15:50:25 UTC
Please describe the problem:
Whether keyboard "caps lock" On or Off, gnopernicus speech engine say the same 
thing. Should speech engine announce lower or upper case per setting or at 
least caps is locked or not ... ?

This would be issues where case sensitive applications are applicable (i.e 
password, etc.)

Setup:
1. Fedora Core 3
2. Gnopernicus 0.10.7
3. Gnome Panel 2.10.2


Steps to reproduce:
1. Bring up Gnopernicus, listen for speech
2. Bring up gnome terminal
3. With No "Caps Lock", type in any alphabets, listen
4. With "Caps Lock", type in any alphabets, listen

Actual results:
Same speech for both lower or upper case, no different

Expected results:
Need to be distinguished for lower or upper case

Does this happen every time?
yes

Other information:
tbd
Comment 1 remus draica 2005-05-17 08:17:49 UTC
Created attachment 46529 [details] [review]
proposed patch
Comment 2 remus draica 2005-05-17 08:20:33 UTC
Comment on attachment 46529 [details] [review]
proposed patch


This patch adds "upper" string before any upper char in "char by char" and
"military" modes.
Comment 3 bill.haneman 2005-05-18 13:44:32 UTC
RFE, not AP1.  Marking AP2 (which is generous).
Comment 4 Oana Serb 2005-06-14 10:24:56 UTC
Created attachment 47748 [details] [review]
Proposed patch

Changed "upper" into "capital".
Comment 5 korn 2005-06-25 00:56:29 UTC
I suggest "cap" rather than capital for at least the English version, and a note
to translators that this should be as brief a string as possible.  "Cap" is what
I've seen Windows screen readers so - you want this to be as brief as possible.
 Otherwise I think this is a fine patch.
Comment 6 Oana Serb 2005-06-27 09:10:25 UTC
Comment on attachment 47748 [details] [review]
Proposed patch

>Index: gnopernicus/speech/libsrs/srs-speech.c
>===================================================================
>RCS file: /cvs/gnome/gnopernicus/speech/libsrs/srs-speech.c,v
>retrieving revision 1.8
>diff -u -r1.8 srs-speech.c
>--- gnopernicus/speech/libsrs/srs-speech.c	25 May 2005 12:01:05 -0000	1.8
>+++ gnopernicus/speech/libsrs/srs-speech.c	14 Jun 2005 10:22:03 -0000
>@@ -378,11 +378,11 @@
> 	    index = srs_sp_letter_get_index_for_spell (letter);
> 
> 	    text = g_string_new ("");
>-/*	    if (g_unichar_isupper (letter)) 
>+	    if (g_unichar_isupper (letter)) 
> 	    {
>-		g_string_append (text, _("upper"));
>+		g_string_append (text, _("cap"));
> 		g_string_append (text, " ");
>-	    }*/
>+	    }
> 	    if (index >= 0)
> 		g_string_append (text,
> 			    tout->spelling == SRS_SPELLING_CHAR ?
Comment 7 Oana Serb 2005-06-27 09:11:04 UTC
Changed "capital" into "cap".
Patch committed to CVS head.