[fenrir-screenreader] Re: Change in copy last echo to clipboard

  • From: Storm Dragon <storm_dragon@xxxxxxxxxxxxxx>
  • To: fenrir-screenreader@xxxxxxxxxxxxx
  • Date: Sat, 7 Mar 2020 07:32:59 -0500

Howdy Chrys,

I fixed it. It now checks to make sure the rstripped text is not "", so you can 
copy just spaces now if you want.

Thanks,
Storm

On Fri, Mar 06, 2020 at 07:44:33PM +0100, Chrys wrote:

Nice patch storm,
Awesome to see growing an other main developer of fenrir.

I wonder if we should go a step more and already rstrip() the text in 
outputmanager.speak when its longer 1 character (to not break echo for space 
character itself if needed) to reduce the amount of data we send to the TTS? 
Maybe this would give a small performance gains for some less optimized TTS 
systems?


Von meinem iPhone gesendet

Am 06.03.2020 um 19:27 schrieb Storm Dragon <storm_dragon@xxxxxxxxxxxxxx>:

Howdy All,

I made a change to the way copy last presented text is copied to the clipboard. 
Now any white space on the right of the copied text is removed. So, for 
example, if you have the line minus the quotes:

"this is a test"

and you pressed copy lasst to clipboard, and you had last used review line or 
previous line, you would get the remainder of the line, at least 60 spaces in 
this case, tacked on to the end of this, so you would paste something like 
this, without the quotes:

"This is a test                                                "

With my change, you should now get the expected output of just:

"This is a test"

This meanss, if you need sspaces after the pasted text, you will have to add 
them manually. So far, I have not needed this ever, and in fact I tend to have 
to remove the spaces manually, which is why I made the change.

Please test, and make sure the change works and that everyone is happy with it.

Thanksss,
Storm

--
⛈🐲
Accessible low cost computers for everyone! Get your slice of the Pi: 
https://asliceofthepi.com
My youtube channel: https://www.youtube.com/channel/UCdaTl5vl404OaRxAJPvb8Ew
get my public PGP key: gpg --keyserver wwwkeys.pgp.net --recv-key 43DDC193

--------------------------------------------
MailingList: fenrir-screenreader@xxxxxxxxxxxxx
Website: https://linux-a11y.org
Wiki: https://wiki.linux-a11y.org/doku.php?id?nrir
Code: https://github.com/chrys87/fenrir
IRC Live Support:
Network: irc.linux-a11y.org
Room: #a11y


--
⛈🐲
Accessible low cost computers for everyone! Get your slice of the Pi: 
https://asliceofthepi.com
My youtube channel: https://www.youtube.com/channel/UCdaTl5vl404OaRxAJPvb8Ew
get my public PGP key: gpg --keyserver wwwkeys.pgp.net --recv-key 43DDC193

Attachment: signature.asc
Description: PGP signature

Other related posts: