Results 570 comments of Tobias Diez
trafficstars

Cool! Can you try to update safari and/or xcode. These keys should be supported in more recent versions according to https://github.com/keepassxreboot/keepassxc-browser/issues/294#issuecomment-696318072

Yeah, it's expected that there are gone be some problems as the the Zotero plugin contains some extra checks for their old-style Safari extension, like https://github.com/JabRef/JabRef-Browser-Extension/blob/master/Zotero/inject.js#L73 Maybe setting `Zotero.isSafari` to...

@Siedlerchr did you made any progress towards this?

Any progress @Siedlerchr? I've read that the webextension api support of Safari has been improved in the meantime, so this might now be easier.

Good suggestion to ask the user if she wants to fallback to a simple import if there is no zotero translator for this page. For firefox, the buttons in the...

The code here is responsible for handling the "no translator found"-case: https://github.com/JabRef/JabRef-Browser-Extension/blob/9aee2a0a350f1ec7fc2a177e8a059c13c0fcd380/background.js#L107-L109 here is how zotero handles that case https://github.com/zotero/zotero-connectors/blob/172daa1e7c2c78f390049eef4c996b4adf874107/src/browserExt/background.js#L606-L615 https://github.com/zotero/zotero-connectors/blob/172daa1e7c2c78f390049eef4c996b4adf874107/src/browserExt/background.js#L801-L803 https://github.com/zotero/zotero-connectors/blob/172daa1e7c2c78f390049eef4c996b4adf874107/src/common/translate_item.js#L75-L85 Let me know if you get stuck somewhere!

For the first point, instead of (or in addition to?) https://github.com/JabRef/JabRef-Browser-Extension/blob/9aee2a0a350f1ec7fc2a177e8a059c13c0fcd380/background.js#L87 you could set the tabInfo in `lookForTranslators` and enrich it by information whether there is a translator for it...

This has been implemented in #442.

This looks like a bug in the translator itself, but I have to double check to confirm this. If you have the time, you can try to install the zotero...