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

Some possible enhancements #33

Closed
Gitbreast opened this issue Aug 23, 2022 · 3 comments
Closed

Some possible enhancements #33

Gitbreast opened this issue Aug 23, 2022 · 3 comments

Comments

@Gitbreast
Copy link

Gitbreast commented Aug 23, 2022

  1. Perhaps add a Google Images cover crawler is a good idea. I used to search metadatas and covers in Calibre, the Google Images seems like the most powerful cover download engine which the books cover are sooo many that every book could find its cover(especially in Non-English languages). Calibre searches the title and author in Google Images and picks up the top 20 for user to choose, which guarantees the quantity, the accurancy and the quality.

Search Url Exp from Calibre

https://www.google.com/search?as_st=y&tbm=isch&as_q=Sword+of+Destiny+%28The+Witcher+%232%29+Andrzej+Sapkowski&as_epq=&as_oq=&as_eq=&cr=&as_sitesearch=&safe=images&tbs=isz:lt,islt:svga,iar:t,ift:jpg

P1: The Logs in Calibre
image
P2: Results: Top 20 covers
image

  1. Is it possible and useful to add a progress bar for showing the import progress so that avoid the user be anxious to close Logseq accidentally?

Plus, thanks for the fix about non-Enghlish importing. It works now but leave each block a long long highlight-id, and the icon before is changed.
image

@theBenForce
Copy link
Owner

Thank you for the suggestions @Gitbreast!

  1. If you hit next from the import dialog that page is pulling information from Amazon, but for some reason it doesn't work unless I'm testing the plugin. I used amazon because I can pull the ASIN and generate a link to open the book from the logseq page. Trying to find a workaround, logseq is locked down pretty tight.

  1. A progress bar is a great idea!

  2. The highlight-id can be hidden. Right now it's using the ❓ since it can't detect if it's a note or highlight. I'll add an option to let you select the default type.

@theBenForce
Copy link
Owner

Created issues #34 and #35 to track suggested enhancements.

@Gitbreast
Copy link
Author

@theBenForce Thanks for the tip! I successfully hid the highlight-id, it looks better now. For suggestion 1, "locked down pretty tight" sounds like Logseq's api or something like that limiting the develop? I'll wait and see if the plugin could pull my books' covers from Amazon correctly in later version, for now I could just paste the cover as an easy alter. Because the number of books limited by reading speed, copy and paste metadata is already fast enough compared to that. XD

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

2 participants