Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Announce number of character repititions instead of truncating to 5 #43

Closed
nvaccessAuto opened this issue Jan 1, 2010 · 5 comments
Closed

Announce number of character repititions instead of truncating to 5 #43

nvaccessAuto opened this issue Jan 1, 2010 · 5 comments
Assignees
@jcsteh
Labels
component/speech enhancement
Milestone
2011.2

Comments

@nvaccessAuto
Copy link

Reported by jteh on 2008-03-18 20:13
Currently, when speaking repeated characters, NVDA simply truncates any repititions after the first five characters. For example, if 10 dashes are encountered, this will be announced as 5 dashes only. This gives no indication of how many repititions actually occurred. It would be more useful to announce the number of times a character occurred if there are more than a certain number of repititions; 2 repititions is probably ideal. For example, two dashes would be announced as "--", but three dashes would be announced as "- 3 times".

Questions

  • Should this be configurable?
    • Is the current behaviour preferable in some cases? If so, there could be an option to switch back to the old behaviour.
    • Should the number of occurrences of a repeated character before this behaviour is triggered be configurable? I think this is overkill; 3 occurrences (2 repititions) should be fine, as "- 3 times" is the same number of syllables as "---".
      Blocked by New text symbol processing framework #332
@nvaccessAuto
Copy link
Author

Attachment repeated_chars.patch added by jteh on 2008-03-19 03:21
Description:
Proof of concept implementation.

@nvaccessAuto
Copy link
Author

Comment 2 by jteh on 2008-03-19 03:23
This patch is a proof of concept implementation.

Change from the spec above:

  • The behaviour is triggered after 3 repeats; i.e. 4 of the same character. Otherwise, "..." gets announced as ". 3 times", which is rather annoying.

Problems:

  • This needs to be integrated with the punctuation code somehow to avoid expansion when punctuation expansion is disabled. Currently, if speak punctuation is disabled, 5 dashes will produce "5 times", even though the dash is later removed.
    • This will get even more tricky once we start using the synth to handle punctuation expansion. We'll need to figure out what punctuation the synth is announcing in order to know when to expand for repeats.
  • How should we handle whitespace?
    • Currently, whitespace gets handled as well, but, for example, "30 times" without knowing it is a space character is bad.
    • Should this avoid handling whitespace?

In hindsight, this idea may be more trouble than it's worth. :)

@nvaccessAuto
Copy link
Author

Comment 5 by jteh on 2009-07-24 06:22
Bzr branch: http://bzr.nvaccess.org/nvda/repeatedAndIndentation/

@nvaccessAuto
Copy link
Author

Comment 6 by jteh on 2011-04-14 03:19
This is already fixed in the new code for #332. On further reflection, I think indentation needs to be handled separately.

@nvaccessAuto
Copy link
Author

Comment 7 by jteh on 2011-04-18 03:12
862d836
Changes:
Milestone changed from None to 2011.2
State: closed

@nvaccessAuto nvaccessAuto added this to the 2011.2 milestone Nov 10, 2015
This was referenced Nov 10, 2015
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcsteh jcsteh

Labels
component/speech enhancement
Projects
None yet
Milestone
2011.2
Development

No branches or pull requests
2 participants
@jcsteh @nvaccessAuto