-
Notifications
You must be signed in to change notification settings - Fork 82
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
In compatible with Zotero 6.0 #44
Comments
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 |
On osx, being able to page / leaf through your documents via So when you need to page through a nr of documents |
@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 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 |
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. |
It works in Mac. |
@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? |
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. |
@mronkko Thanks! We will deeply appreciate your help and look forward to the new ZoteroQuickLook Plugin. |
@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.
|
@zoelir729 I truly appreciate your timely help! |
@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.
|
It works. |
Good job!! |
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) |
你好,邮件已收到,我会尽快处理,如有急事请联系13547906739
|
For compatibility with Zotero 6.x. Refer to [issuecomment](mronkko#44 (comment)) and [comment](mronkko#44 (comment)).
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 |
That was my favorite way to quick check files. Hopefully we can get working again. I am not a MAC user at the moment =( |
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. |
…w focus in Linux // <zoteroquicklook.pl> a bug fix that preventing the new sushi preview window from losing window focus <.gitignore>: ignore .xpi/.zoteroplugin files for version management <linux_build.sh>: add shell script to perform auto-pack for .xpi plugin, compatiale with Zotero 6.0 <README.md>: modify installing section and the part description of non-Windows system 1) installation note for zotero 6.0 2) add path check for quicklook application 3) Gnome Suhi installation guide <.rdf> update zotero6.0 version support and update plugin version numbers to 1.4.3 // Close mronkko#44, mronkko#43
…w focus in Linux // <zoteroquicklook.pl> a bug fix that preventing the new sushi preview window from losing window focus <.gitignore>: ignore .xpi/.zoteroplugin files for version management <linux_build.sh>: add shell script to perform auto-pack for .xpi plugin, compatiale with Zotero 6.0 <README.md>: modify installing section and the part description of non-Windows system 1) installation note for zotero 6.0 2) add path check for quicklook application 3) Gnome Suhi installation guide <.rdf> update zotero6.0 version support and update plugin version numbers to 1.4.3 // Close mronkko#44, mronkko#43
…w focus in Linux // <zoteroquicklook.pl> a bug fix that preventing the new sushi preview window from losing window focus <.gitignore> ignore .xpi/.zoteroplugin files for version management <linux_build.sh> add shell script to perform auto-pack for .xpi plugin, compatiale with Zotero 6.0 <README.md> modify installing section and the part description of non-Windows system 1) installation note for zotero 6.0 2) add path check for quicklook application 3) Gnome Suhi installation guide <.rdf> update zotero6.0 version support and update plugin version numbers to 1.4.3 <TODO> list Bugs/Low-priority bugs/Ideas // Close mronkko#44, mronkko#43
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! |
你好,邮件已收到,我会尽快处理,如有急事请联系13547906739
|
Zotero 7 联用QuickLook的解决方法(A solution for using QuickLook in Zotero 7 or 6):https://mp.weixin.qq.com/s/wlwzEsbxjTOctLdWZk_g_w |
你好,邮件已收到,我会尽快处理,如有急事请联系13547906739
|
Another solution for using QuickLook for Zotero 7 can be found here: #47 |
The plugin seems to be incompatible with the Zotero beta.
The text was updated successfully, but these errors were encountered: