omnivore-app / logseq-omnivore

Logseq plugin to fetch articles and highlights from Omnivore
MIT License
298 stars 32 forks source link

Highlighted English text displays incorrectly #68

Closed artemith closed 1 year ago

artemith commented 1 year ago

The synced highlighted English text displays incorrectly like screenshot. The Chinese text displays correctly.

Logseq: 0.8.16 Omnivore plug-in: 1.11.2 OS: Windows 10

English: image

Chinese: image

sywhb commented 1 year ago

Hey @artemith, thanks for reporting.

Do the other English articles also have the same issue? I'm not able to replicate it and it looks correct on my machine.

Screenshot 2023-01-31 at 5 33 00 PM

It looks like some formatting issue. Could you also try to resync Omnivore data and see?

artemith commented 1 year ago

Hey @artemith, thanks for reporting.

Do the other English articles also have the same issue? I'm not able to replicate it and it looks correct on my machine. Screenshot 2023-01-31 at 5 33 00 PM

It looks like some formatting issue. Could you also try to resync Omnivore data and see?

Thanks for your reply! I just tested again. I found the articles collected by Microsoft Edge's Omnivore integration has this problem, whille those collected by Firefox's doesn't.

sywhb commented 1 year ago

@artemith Oh sorry, I didn't test on Edge. Let me try again

sywhb commented 1 year ago

Hey @jacksonh, do you have a Windows device that can be used to test on? I could not reproduce this issue on the Edge browser and the articles @artemith saved seem to be in a strange format

sywhb commented 1 year ago

Original HTML: https://gist.github.com/sywhb/6f72068117133f1ff1f4cb6cc1712c06

sywhb commented 1 year ago

Hey @jacksonh, do you have a Windows device that can be used to test on? I could not reproduce this issue on the Edge browser and the articles @artemith saved seem to be in a strange format

Hey @jacksonh, could you help me with this one?

artemith commented 1 year ago

The bug does not appear in Omnivore plug-in 1.11.8 and Logseq 0.9.1.

jacksonh commented 1 year ago

@artemith thank you for following up on this one.

artemith commented 1 year ago

Thank you for your hard work!