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

Apostrophes break title attribute #1

Open
donkirkby opened this issue Jun 14, 2014 · 1 comment
Open

Apostrophes break title attribute #1

donkirkby opened this issue Jun 14, 2014 · 1 comment

Comments

@donkirkby
Copy link

In processResults(), you are building the anchor tag with string concatenation. That fails when the text example contains an apostrophe, and the title attribute is truncated.
To see the bug, search the Bieber corpus for, "in the morning". The first entry is, "in the closet" with this full text:

Dad: "Because hunny there's no sun in the closet."

When you hover over the entry, the tooltip only shows this:

Dad: "Because hunny there

@donkirkby
Copy link
Author

It's an interesting project, by the way. I've done some very basic things with NLP on my Vograbulary project, but I intend to do more exploration.

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

No branches or pull requests

1 participant