[Share] Open Attachment in Mac QuickLook (or Linux Sushi) #450

jakowag · 2025-01-11T22:03:43Z

jakowag
Jan 11, 2025

Description

Restores the missing quicklook feature for Zotero7 on a Mac. Press the spacebar and get a preview of the file, press again to close it.

Updates the conversation in 132, especially the comment posted by @YaoLiMuMu in #132 (comment)_.

Building upon work from: @windingwind @l0o0 @YaoLiMuMu @lxysl

Event

Program Startup

Operation

Script

Data

/**
 * Restores the missing quicklook feature for Zotero7 on a Mac or Linux computer. Press the spacebar and get a preview of the file, press again to close it.
 * @jakowag @windingwind @l0o0 @YaoLiMuMu @lxysl
 * @usage press spacebar
 * @link https://github.com/windingwind/zotero-actions-tags/discussions/
 * @see https://github.com/windingwind/zotero-actions-tags/discussions/
 */

(function () {

const ZoteroPane = require("ZoteroPane");
const Zotero_Tabs = require("Zotero_Tabs");
const document = require("document");

async function getAttachmentPath(item) {
    if (item.isAttachment() && !item.isNote()) {
        return await item.getFilePathAsync();
    }
    else if (item.isRegularItem() && !item.isAttachment()) {
        let attachments = await item.getAttachments();
        for (let attachmentID of attachments) {
            let attachment = await Zotero.Items.getAsync(attachmentID);
            if (attachment.attachmentContentType === 'application/pdf') {
                return await attachment.getFilePathAsync();
            }
        }
    }
    return null;
}

async function openFile(item) {
    let filePath = await getAttachmentPath(item);
    if (!filePath) {
        return false;
    }
    // macbook path : -p /usr/bin/qlmanage
    // linux path : /usr/bin/sushi
    let applicationPath = "/usr/bin/qlmanage" // IM ON A MAC, ADJUST THIS IF YOU HAVE LINUX
    let args = ['-p', filePath] // THIS TOO
    Zotero.Utilities.Internal.exec(applicationPath, args);
}

async function oneKey(event) {
    if (event.code != "Space") return;
    let item = ZoteroPane.getSelectedItems()[0];
    if (typeof item === 'undefined') {
        Zotero.debug("No item selected for quicklook.");
        return false;
    }
    var key = String.fromCharCode(event.which);
    if (Zotero_Tabs.selectedID === "zotero-pane") {
		await openFile(item);
		ZoteroPane.collectionsView.selectItems([item.id]); // reselect the current item to close it directly 
		return;
    }
}

document.getElementById('zotero-items-tree').addEventListener("keydown", oneKey, false);
})()

Anything else

Change the following block if you are on linux:

let applicationPath = "/usr/bin/qlmanage"
    let args = ['-p', filePath]

to

let applicationPath = "/usr/bin/sushi"
    let args = [filePath]

Ignore this warning for now. Unfortunately, there is no event yet which is triggered when an item gets selected and populates an item variable. The other events are not really appropriate for now afaik.

This script is using ZoteroPane.getSelectedItems, which is NOT recommended for getting items in scripts and can lead to unexpected behavior. Please use itemor items variables passed to the script instead.

kithairon · 2025-05-08T07:48:13Z

kithairon
May 8, 2025

Hi Jakob,
Happy user of your quicklook plugin in Zotero actions tags. Thanks for restoring the QuickLook function for Mac folks.
Recenty I started to have issues with your script. I posted in the ActionTags repo but windingwind suggested I contact the author of the script directly. Here my issue:

Is there an existing issue for this?
No
I have searched the existing issues
Are you using the latest Zotero and the latest plugin?
I have confirmed I'm using the latest Zotero and the latest plugin
Environment
OS: macOS Sonoma 14.7.5
Zotero Version: 7.1-beta.26+8c7ab1eb6
Plugin Version: 2.1.0 5 Jan 2025
Describe the bug
Upon invoking the Actions & Tags Quicklook function with the spacebar, two dozen or so instances of the qlmanage process are evoked. The Preview opens fine but the many instantiations of qlmanage need to be closed again manually. Any ideas? Thanks for your help.

Yours is currently the most convincing preview implementation.

Debug Output
[JavaScript Error: "Resource null/attachment/library/items/MA9JRKF6/header2.jpg@22 not found"]

[JavaScript Error: "Resource null/attachment/library/items/MA9JRKF6/header1.jpg@22 not found"]

[JavaScript Error: "AsyncChannel request failed with status 2152398850" {file: "chrome://zotero/content/ZoteroProtocolHandler.jsm" line: 1447}]

[JavaScript Error: "Error processing feed from https://api.equinoxpub.com/issues/BSR/current.rss:

Error: Processing failed"]