lambda-nvda / lambdaNvda

NVDA Add-On for LAMBDA Software
GNU General Public License v2.0
5 stars 4 forks source link

Make the last character unselected can be reported #39

Closed nvdaes closed 7 years ago

nvdaes commented 7 years ago

For testing carefully before merging

albzan commented 7 years ago

Yeah, That's a way but it reports unselected also for long text:

  1. write an expression
  2. go to the beginning of the line
  3. press ctrl+end
  4. press left arrow (for instance you have copied it)

... you will listen this long peice of text repeated wit the "unselected" announcement. That's why I set that control. What do you think? Alberto

nvdaes commented 7 years ago

Really I don't know. I follow your exact steps and nothing is repeated. For me it works fine.

2017-05-17 19:55 GMT+02:00, albzan notifications@github.com:

Yeah, That's a way but it reports unselected also for long text:

  1. write an expression
  2. go to the beginning of the line
  3. press ctrl+end
  4. press left arrow (for instance you have copied it) ... you will listen this long peice of text repeated wit the "unselected" announcement. That's why I set that control. What do you think? Alberto

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/nvdaaddons/lambda/pull/39#issuecomment-302174093

albzan commented 7 years ago

Yeah sorry, I was wrong with steps:

  1. write an expression like 20-{510-[(412-3)/9+12(15/5+1)]}/7
  2. go to the beginning of the line (home)
  3. press shift+end to select the line (i.e. you want to copy it
  4. press left arrow

... you will listen this long peice of text repeated wit the "unselected" announcement. You are unable to understand what's the character under the cursor That's why I set that control. What do you think? Alberto

nvdaes commented 7 years ago

A, I understand now. I think it's better that extra info is spoken, but no wrong or missing things. Just an opinion.

2017-05-17 20:49 GMT+02:00, albzan notifications@github.com:

Yeah sorry, I was wrong with steps:

  1. write an expression like 20-{510-[(412-3)/9+12(15/5+1)]}/7
  2. go to the beginning of the line (home)
  3. press shift+end to select the line (i.e. you want to copy it
  4. press left arrow

... you will listen this long peice of text repeated wit the "unselected" announcement. You are unable to understand what's the character under the cursor That's why I set that control. What do you think? Alberto

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/nvdaaddons/lambda/pull/39#issuecomment-302193157

nvdaes commented 7 years ago

Another option, if possible, would be to make a jack in the move caret scripts, so that if just arrows are pressed and text is selected, change selection is not spoken.

2017-05-17 20:49 GMT+02:00, albzan notifications@github.com:

Yeah sorry, I was wrong with steps:

  1. write an expression like 20-{510-[(412-3)/9+12(15/5+1)]}/7
  2. go to the beginning of the line (home)
  3. press shift+end to select the line (i.e. you want to copy it
  4. press left arrow

... you will listen this long peice of text repeated wit the "unselected" announcement. You are unable to understand what's the character under the cursor That's why I set that control. What do you think? Alberto

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/nvdaaddons/lambda/pull/39#issuecomment-302193157

albzan commented 7 years ago

Eh, That's definetly a matter of taste. Personally I find it boring when working with a braille display... Because when you select some text and then you press a CR, you listen the speech that start and you have to stop it with Control... I don't know...

nvdaes commented 7 years ago

Yes, this is true. Maybe a setting to choose this behavior?

2017-05-17 21:00 GMT+02:00, albzan notifications@github.com:

Eh, That's definetly a matter of taste. Personally I find it boring when working with a braille display... Because when you select some text and then you press a CR, you listen the speech that start and you have to stop it with Control... I don't know...

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/nvdaaddons/lambda/pull/39#issuecomment-302196131

albzan commented 7 years ago

Yes we can set it up easilly...

albzan commented 7 years ago

Would you like to update the pull request? We can set an option: silentUnselection and add an extra condition on the If... Thanks

nvdaes commented 7 years ago

Hi, updated now. Please look if it's according to your style (I tried it) and so on. Cheers.

2017-05-17 21:06 GMT+02:00, albzan notifications@github.com:

Would you like to update the pull request? We can set an option: silentUnselection and add an extra condition on the If... Thanks

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/nvdaaddons/lambda/pull/39#issuecomment-302197634