[Bug] Dark mode icons - Githubissues

mjthoraval commented 6 months ago

Is there an existing issue for this?

[X] I have searched the existing issues

Are you using the latest Zotero and the latest plugin?

[X] I have confirmed I'm using the latest Zotero and the latest plugin

Environment

OS: Windows 10
Zotero Version: 7.0.0-beta.63+52856ae69 (64-bit)
Plugin Version: 1.1.4-beta.56

Describe the bug

The icons for the note popup window need to be updated for the dark mode:

Somehow, they are also not in the same order:

Debug Output

1709057596332 addons.xpi WARN Checking C:\Program Files\Zotero\distribution\extensions for addons

[JavaScript Error: "NS_NOINTERFACE: " {file: "resource://gre/modules/ComponentUtils.jsm" line: 90}] createInstance@resource://gre/modules/ComponentUtils.jsm:90:36

1709057598139 addons.xpi WARN Addon with ID undefined already installed, older version will be disabled

[JavaScript Error: "Error: Addon must include an id, version, and type" {file: "resource://gre/modules/addons/XPIProvider.jsm" line: 1710}]

[JavaScript Error: "NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsITelemetry.getHistogramById]" {file: "resource://gre/modules/TerminatorTelemetry.jsm" line: 87}]

[JavaScript Error: "NS_ERROR_NOT_AVAILABLE" {file: "resource://gre/modules/PartitioningExceptionListService.jsm" line: 92}]

[JavaScript Error: "uncaught exception: undefined"]

[JavaScript Error: "TypeError: emptyListMessage is null" {file: "chrome://zotero/content/standalone/standalone.js" line: 820}] ZoteroStandalone</this.updateAddonsPane/<@chrome://zotero/content/standalone/standalone.js:820:7

[JavaScript Error: "Passing a URI to Zotero.File.getContentsAsync() is deprecated -- use Zotero.HTTP.request() instead"]

[JavaScript Error: "o.current is null" {file: "resource://zotero/note-editor/editor.js" line: 6}]

[JavaScript Error: "NotFoundError: Could not get children of file(C:\Users\mjthoraval\AppData\Roaming\Zotero\Zotero\Crash Reports\events) because it does not exist" {file: "resource://gre/modules/CrashManager.jsm" line: 682}] _getUnprocessedEventsFiles/<@resource://gre/modules/CrashManager.jsm:682:12

1709057725044 addons.xpi WARN Download of https://github.com/windingwind/zotero-pdf-translate/releases/latest/download/zotero-pdf-translate.xpi/download/v1.1.0-beta.31/zotero-pdf-translate.xpi failed: 404 Not Found

appName => Zotero, version => 7.0.0-beta.63+52856ae69, os => Windows 10.0 19045, locale => en-GB, extensions => Better Notes for Zotero (1.1.4-beta.56, extension), Notero (0.5.15, extension, disabled), Actions and Tags for Zotero (1.0.0-beta.34, extension, disabled), Translate for Zotero (1.1.0-beta.30, extension, disabled), Open PDF for Zotero (0.0.11, extension, disabled), Better BibTex for Zotero (6.7.162, extension, disabled), Zotero Better Authors (4.1.3, extension, disabled), Zotero Attanger (1.1.3, extension, disabled), ZoteroPreview (0.7.2, extension, disabled)

(3)(+0093178): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000000): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000001): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0000016): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] {}

Anything else?

No response

windingwind commented 6 months ago

Thanks, planned

windingwind commented 6 months ago

thanks for the feedback. fixed in v1.1.4-beta.57

mjthoraval commented 6 months ago

Thanks. The icons are now compatible with the dark mode. But the changing order is still not fixed: 1) Click on some text in the note. 2) Click on a note link: 848e6a500ec60ae8073af0c4953381f 3) Click on another link. 4) Click back on the same link: fb549ce07872f269e31af3dc4705589

1709908947255 addons.xpi WARN Checking C:\Program Files\Zotero\distribution\extensions for addons

[JavaScript Error: "NS_NOINTERFACE: " {file: "resource://gre/modules/ComponentUtils.jsm" line: 90}] createInstance@resource://gre/modules/ComponentUtils.jsm:90:36

1709908948539 addons.xpi WARN Addon with ID undefined already installed, older version will be disabled

[JavaScript Error: "Error: Addon must include an id, version, and type" {file: "resource://gre/modules/addons/XPIProvider.jsm" line: 1710}]

[JavaScript Error: "NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsITelemetry.getHistogramById]" {file: "resource://gre/modules/TerminatorTelemetry.jsm" line: 87}]

[JavaScript Error: "NS_ERROR_NOT_AVAILABLE" {file: "resource://gre/modules/PartitioningExceptionListService.jsm" line: 92}]

[JavaScript Error: "Passing a URI to Zotero.File.getContentsAsync() is deprecated -- use Zotero.HTTP.request() instead"]

[JavaScript Error: "NotFoundError: Could not get children of file(C:\Users\mjthoraval\AppData\Roaming\Zotero\Zotero\Crash Reports\events) because it does not exist" {file: "resource://gre/modules/CrashManager.jsm" line: 682}] _getUnprocessedEventsFiles/<@resource://gre/modules/CrashManager.jsm:682:12

[JavaScript Error: "Component returned failure code: 0x8052000e (NS_ERROR_FILE_IS_LOCKED) [mozIStorageService.backupDatabaseFile]" {file: "chrome://zotero/content/xpcom/db.js" line: 1058}]

appName => Zotero, version => 7.0.0-beta.66+d0299a1ac, os => Windows 10.0 19045, locale => en-GB, extensions => Better Notes for Zotero (1.1.4-beta.57, extension), Notero (0.5.15, extension, disabled), Actions and Tags for Zotero (1.0.0-beta.34, extension, disabled), Open PDF for Zotero (0.0.11, extension, disabled), Zotero Better Authors (4.1.3, extension, disabled), Zotero Attanger (1.1.3, extension, disabled), Translate for Zotero (1.1.0-beta.31, extension, disabled), Better BibTex for Zotero (6.7.164, extension, disabled), ZoteroPreview (0.7.8, extension, disabled)

(3)(+1289960): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0000018): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0028589): [Better Notes for Zotero] {}

(3)(+0001432): [Better Notes for Zotero] {}

(3)(+0000016): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0000001): [Better Notes for Zotero] {}

(3)(+0004616): [Better Notes for Zotero] {}

(3)(+0000000): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000001): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000000): [Better Notes for Zotero] [Warning] Creating element button with no namespace specified. Found multiply namespace matches.

(3)(+0000000): [Better Notes for Zotero] {}

(3)(+0038474): itemTree.render(). Displaying Item Tree

github-actions[bot] commented 5 months ago

This issue is stale because it has been open for 30 days with no activity.

windingwind commented 5 months ago

fixed in v1.1.4-beta.61

windingwind / zotero-better-notes

[Bug] Dark mode icons #902

Is there an existing issue for this?

Are you using the latest Zotero and the latest plugin?

Environment

Describe the bug

Debug Output

Anything else?