mronkko / ZoteroQuickLook

Implements QuickLook in Zotero
758 stars 82 forks source link

In compatible with Zotero 6.0 #44

Open tenglu0417 opened 2 years ago

tenglu0417 commented 2 years ago

The plugin seems to be incompatible with the Zotero beta. image

bwiernik commented 2 years ago

This plugin is not currently being maintained. The new PDF reader is much faster than QuickLook ever was. I will accept pull requests to fix the issues needed to make the plugin functional again, but I'm not going to spend time to fix them myself

jf--- commented 2 years ago

On osx, being able to page / leaf through your documents via ZoteroQuickLook does make browsing quite a pleasure. The immediacy of access is really key for doing research.

So when you need to page through a nr of documents ZoteroQuickLook still presents a huge advantage over opening the file in a reader. In terms of browsing, nothing really has changed, since either opening in the new reader or in Preview.app pretty much is the same latency.

AddictArts commented 2 years ago

@tenglu0417 Please consider a rename to remove the "beta", thanks in advance.

For Mac OS, it appears initScripts or initExecutable is no longer properly completing, the functions. When openQuickLook gets called the this.viewerBaseArguments and this.viewerExecutable are null.

I was able to fix it for Mac by putting this in to set the values.

        this.viewerBaseArguments=['-p'];
        this.viewerExecutable = Zotero.File.pathToFile("/usr/bin/qlmanage");

right after

    openQuickLook: async function(items) {

Note: the install.rdf update to 6.0.* for the maxVesion for the target.

I don't have time to create a proper fork and pull request. Cheers

bwiernik commented 2 years ago

As I've said multiple times, I am happy to accept a pull request if someone wants to make the plugin compatible with Zotero 6, but I don't have the time to make the updates myself. If you have done so, please submit a pull request.

zoelir729 commented 2 years ago

@tenglu0417 Please consider a rename to remove the "beta", thanks in advance.

For Mac OS, it appears initScripts or initExecutable is no longer properly completing, the functions. When openQuickLook gets called the this.viewerBaseArguments and this.viewerExecutable are null.

I was able to fix it for Mac by putting this in to set the values.

        this.viewerBaseArguments=['-p'];
        this.viewerExecutable = Zotero.File.pathToFile("/usr/bin/qlmanage");

right after

  openQuickLook: async function(items) {

Note: the install.rdf update to 6.0.* for the maxVesion for the target.

I don't have time to create a proper fork and pull request. Cheers

It works in Mac. And sorry to ask, the Chinese.pdf can not be opened, how to sovle it ? Thanks!

image
cannolis commented 2 years ago

@zoelir729 @AddictArts Thank all of you for discussing and fixing the issue. But I'm new to Zotero and don't actually understand where to put the code in. Could you please tell me more details about it?

mronkko commented 2 years ago

If you are new to Zotero it is probably not a good idea to start making changes to the code.

I originally developed the plugin and was using it actively until upgrading to Zotero 6. I see value in the ability browse through articles with quicklook and this is not something that the built-in viewer provides. I will probably fix this at some point, but given that this is a side project it is difficult to say when that might be.

cannolis commented 2 years ago

@mronkko Thanks! We will deeply appreciate your help and look forward to the new ZoteroQuickLook Plugin.

zoelir729 commented 2 years ago
image image image image

Zip these files (Do not zip the upper level folder), and change the .zip into the .xpi.

phguo commented 2 years ago

@zoelir729 A temporary solution:

Tools -> ZotFile Preferences -> Advanced Settings -> "☑️rename special characters (diacritics) from file name"

Update:

Works for non English non Chinese file name, but still does not work for Chinese file name.

@tenglu0417 Please consider a rename to remove the "beta", thanks in advance. For Mac OS, it appears initScripts or initExecutable is no longer properly completing, the functions. When openQuickLook gets called the this.viewerBaseArguments and this.viewerExecutable are null. I was able to fix it for Mac by putting this in to set the values.

        this.viewerBaseArguments=['-p'];
        this.viewerExecutable = Zotero.File.pathToFile("/usr/bin/qlmanage");

right after

    openQuickLook: async function(items) {

Note: the install.rdf update to 6.0.* for the maxVesion for the target. I don't have time to create a proper fork and pull request. Cheers

It works in Mac. And sorry to ask, the Chinese.pdf can not be opened, how to sovle it ? Thanks! image

cannolis commented 2 years ago

@zoelir729 I truly appreciate your timely help!

cannolis commented 2 years ago

@zoelir729 I have a quick look at the file named zoteroquicklook.js, and just after I remove the code as below in the function cleanFileName, the plugin works well for the Chinese file name.

    if (this.getPref("usefilenameworkaround") && ! Zotero.isWin){
    filename=filename.replace(/[^A-Z0-9.:\/\\_\- ]/gi,'*');
    }
zoelir729 commented 2 years ago

@zoelir729 I have a quick look at the file named zoteroquicklook.js, and just after I remove the code as below in the function cleanFileName, the plugin works well for the Chinese file name.

    if (this.getPref("usefilenameworkaround") && ! Zotero.isWin){
    filename=filename.replace(/[^A-Z0-9.:\/\\_\- ]/gi,'*');
    }

It works. Thanks!

iseesaw commented 2 years ago

Good job!!

Regulust commented 2 years ago

I tried to modify the code by following the discussions above to make ZoteroQuickLook work with Zotero 6.x in this fork version, which can be downloaded and installed directly. Thanks for all good comments above.

Also Have found another new fork version - ZoteroQuickLookReload which is compatible with Zotero 6.x too. Guys could test the compatibility on your system. (Change the extension name to .xpi or .zip to install on Zotero 6.x)

lbc1027 commented 2 years ago

你好,邮件已收到,我会尽快处理,如有急事请联系13547906739

beaugunderson commented 2 years ago

I have forked the project again because ZoteroQuickLookReload (the fork mentioned above) does not provide an un-obfuscated version of the source and I was worried about running it on my system.

My version is here (and only supports macOS): https://github.com/beaugunderson/ZoteroQuickLookNG

eliasgalvan305 commented 2 years ago

That was my favorite way to quick check files. Hopefully we can get working again. I am not a MAC user at the moment =(

cwwalter commented 2 years ago

Thanks for this @beaugunderson! I had the same worry. I found I could make the old version work just by changing the target version, but I see you did a lot of cleanup and removed calling the perl file. Your version also seems more responsive on my system.

joaofrgomes commented 1 year ago

I have forked the project again because ZoteroQuickLookReload (the fork mentioned above) does not provide an un-obfuscated version of the source and I was worried about running it on my system.

My version is here (and only supports macOS): https://github.com/beaugunderson/ZoteroQuickLookNG

Hi! I have been using one of the multiple QuickLook extensions with Zotero 6 but now, with the advent of Zotero 7, it seems they, too, will break. Do you have any plans on updating your own for the upcoming version? Thanks!

lbc1027 commented 1 year ago

你好,邮件已收到,我会尽快处理,如有急事请联系13547906739

occasional15 commented 11 months ago

Zotero 7 联用QuickLook的解决方法(A solution for using QuickLook in Zotero 7 or 6):https://mp.weixin.qq.com/s/wlwzEsbxjTOctLdWZk_g_w

lbc1027 commented 11 months ago

你好,邮件已收到,我会尽快处理,如有急事请联系13547906739

dschaehi commented 6 months ago

Another solution for using QuickLook for Zotero 7 can be found here: https://github.com/mronkko/ZoteroQuickLook/issues/47