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 354487 - apostrophe should not delimit a word boundry
apostrophe should not delimit a word boundry
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 17:48 UTC by Mike Pedersen
Modified: 2008-07-22 19:20 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Patch to hopefully fix the problem. (1.17 KB, patch)
2006-09-08 14:44 UTC, Rich Burridge
none Details | Review

Description Mike Pedersen 2006-09-05 17:48:41 UTC
Please describe the problem:
When using echo by word, typing an ' causes the word to be spoken before whatever comes after the '.

Steps to reproduce:
1. Set echo by word in the orca config UI
2. open gedit
3. type Mike's 


Actual results:
You hear Mike as soon as the ' is pressed

Expected results:
You should hear Mike's when the space is pressed after the "s" 

Does this happen every time?
yes

Other information:
Comment 1 Rich Burridge 2006-09-05 19:25:51 UTC
The problem here is the current simplistic definition
of the util.isWordDelimiter() routine.

    return (character in string.whitespace) \
           or (character in string.punctuation)

Comment 2 Rich Burridge 2006-09-07 17:42:49 UTC
I think we are all agreed that all of the string.whitespace
should act as word delimiters. That would be:

' \t\n\r\v\f'

i.e. Space, Tab, Newline, Carriage Return, Vertical Tab and Form feed.

Mike, which of the following from string.punctuation should
we also always use as word delimiters?

!#$%&'()*+,-./:;<=>?@[\]^_`{|}~


Comment 3 Mike Pedersen 2006-09-07 21:02:07 UTC
!*+,-./:;<=>?@[\]^_{|}

I think this should do it.  
Comment 4 Rich Burridge 2006-09-08 14:44:06 UTC
Created attachment 72416 [details] [review]
Patch to hopefully fix the problem.

Thanks Mike.
Comment 5 Rich Burridge 2006-09-08 14:47:26 UTC
Changed checked into CVS HEAD. Closing as FIXED.