-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Preserve <br>? #4
Comments
Sure, PRs are always welcome! See CONTRIBUTIONS for how to send one. At a minimum, please add a new test with the content you are trying to parse, and then modify the source so that your new text is parsed correctly (while ensuring that existing tests also continue to pass.) |
Decided to use mozilla/readability instead since it's a bit more robust. Thanks! |
True, it’s been around longer & has more maintainers. Though, it’s written in JavaScript, which precludes many usages of it, especially in Android apps. Crux’s predecessor, Snacktory, started as a Java clone of Readability. |
In Android I'm injecting it into a WebView and extracting output with webview.evaluateJavascript 😅 |
I tried extracting https://us13.campaign-archive.com/?u=67bd06787e84d73db24fb0aa5&id=c3e998f811&e=7bc177b38a
and then rendering it. Looks broken since the
are extracted. Can I send a PR to add them back? What's the best way to do this?
The text was updated successfully, but these errors were encountered: