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

web clipper not handling tables, other fine points well #35

Open
pcause opened this issue Jan 5, 2022 · 2 comments
Open

web clipper not handling tables, other fine points well #35

pcause opened this issue Jan 5, 2022 · 2 comments
Labels
enhancement New feature or request

Comments

@pcause
Copy link

pcause commented Jan 5, 2022

Was in browser at this page https://github.com/qownnotes/web-companion/issues/new and wanted to clip. I right clicked the page and selected the "Send page to QOwnNotes". The page was sent but the content was not translated to markdown well. In particular a table of laptops was not captured/rendered well. Joplin's web clipper gives a pretty faithful translation. Their web clipper is open source so you might look at their logic to see if there are ideas to improve your capture/trranslate logic to get better results.

I've attached a zip file with the web page as exported by Firefox, the markdown exported ny QOwnNotes and the markdown as exported by Joplin.

sorry fotgot the files

clipper.zip

Expected behaviour

More faithful markdown translation especially the tables.

Actual behaviour

Steps to reproduce

go the the URL provides and right click the page and select the QoenNotes/Send Page option.

Output from the debug section in the settings dialog in QOwnNotes

using 22.1.3

@pbek
Copy link
Member

pbek commented Jan 5, 2022

Where did you get tables on https://github.com/qownnotes/web-companion/issues/new?
I'm open to pull requests, the html to markdown transformation is done by https://github.com/pbek/QOwnNotes/blob/1e61524efa60697f5cb4c79c726b6bdfee749934/src/mainwindow.cpp#L7524-L7580.

You can also do transformations directly in a script with a hook: https://www.qownnotes.org/scripting/hooks.html#websocketrawdatahook

@pbek pbek added the enhancement New feature or request label Jan 5, 2022
@pcause
Copy link
Author

pcause commented Jan 5, 2022

sorry pasted the wrong link. here is the right one: https://www.ultrabookreview.com/42630-intel-evo-laptops/

this is the part of the page t gets converted to a table in Joplin:

image

this is joplin preview

image

this is the joplin markdown

image

in Qown preview we see the content where the table starts through almost the rest of the article is missing. Not just the table, but the rest of the article

image

and here is the markup

image

Abd gere is a view of the HTML source at the start of the table on the web page

image

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

No branches or pull requests

2 participants