Changing theme on a Reddit snapshot makes Zotero unresponsive

mjthoraval
edited 11 days ago
I have imported the webpage from a Reddit post (Save to Zotero - Web Page with Snapshot):
https://www.reddit.com/r/zotero/comments/16qkyw1/plugins_problem/

After selecting the snapshot, it shows the following error in the Debug Output:
[JavaScript Error: "Failed to attach Declarative Shadow DOM."]
Then, I open the snapshot. If I try to change the theme, I get the following error:
https://s3.amazonaws.com/zotero.org/images/forums/u265723/w33dckwr0dc1zjuw5cek.png
A script on this page may be busy, or it may have stopped responding. You can stop the script now, or you can continue to see if the script will complete.

Script: resource://zotero/reader/reader.js line 43084 > eval:795

Debug ID D658002257
Zotero 7.1-beta.14+2e9b9d407 (64-bit)
Windows 10
  • AbeJellinek
    I can't reproduce. Does this still happen if you restart Zotero? If so, could you email the snapshot file to support@zotero.org and include a link to this thread?
  • mjthoraval
    It still happens after restarting Zotero.
    File sent.
  • AbeJellinek
    That page has an extraordinarily large amount of style code, and the theme engine needs to parse and rewrite all of it. We'll look into preventing that dialog, but there's little we can do about the freeze itself (the Dark Reader browser extension, which snapshot theming is based on, has the same issue).
  • mjthoraval
    Thank you for looking into it.
    It would be nice to be able to avoid the problem, as a simple click on the item in the items list already triggers the problem through the attachment preview.

    Specifically for Reddit, I am interested in getting the full thread of discussions, including all the replies. But the reddit translator only captures the original post. This is why I need to use the webpage snapshot. But that is causing the problem, so I cannot keep it into my library.
  • AbeJellinek
    Snapshotting old.reddit.com should work much better.
  • mjthoraval
    That works. Thank you.
  • AbeJellinek
    And the next beta will include a heuristic to disable theming on pages with a lot of styles (like that one).
Sign In or Register to comment.