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

Issues with selected text with quotes in it #16

Closed
orschiro opened this issue Oct 17, 2016 · 3 comments
Closed

Issues with selected text with quotes in it #16

orschiro opened this issue Oct 17, 2016 · 3 comments

Comments

@orschiro
Copy link

Hi @ZachSaucier,

I have a problem reading an email in Gmail.

Since it contains private information, I am forwarding you this email directly.

I hope that's fine with you.

Thanks,

-Robert

@ZachSaucier
Copy link
Owner

ZachSaucier commented Oct 17, 2016

Thanks for bringing this up. There was an error caused by the quotes because I was failing to escape them properly. Now that is fixed, but I realize that the way Chrome builds their selection function is that it returns only text, not any DOM, which leaves the selected content pretty ugly.

So, I switched up how it works so that the DOM is selected instead of just the text. It should be fixed in the next version (I'll push the changes later today).

@ZachSaucier ZachSaucier changed the title Issues with reading email in Gmail Issues with selected text with quotes in it Oct 17, 2016
@orschiro
Copy link
Author

Wow, thanks so much for having found a fix so quickly!

On Mon, Oct 17, 2016 at 2:32 PM, Zach Saucier notifications@github.com
wrote:

Thanks for bringing this up. There was an error caused by the quotes
because I was failing to escape them properly. Now that is fixed, but I
realize that the way Chrome builds their selection function is that it
returns only text, not any DOM, which leaves the selected content pretty
ugly.

So, I switched up how it works so that the DOM is selected instead of just
the text. It should be fixed in the next version (I'll push the changes
now).


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#16 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKbw4vuEK8yFgBEx59xQdrUpoWTLPO-ks5q02rKgaJpZM4KYSPx
.

@ZachSaucier
Copy link
Owner

After I uploaded the change listed above in version 1.0.6 I realized in the shower that I didn't implement it correctly. I've now fixed the bug, so version 1.0.7 should work correctly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants