ucjsDownloadsManager.uc.js won't open window in Firefox 120 about firefox-scripts

[LummoxJR]

There appear to be further issues with lacking a Clear Downloads button, so I'll keep investigating. The font has gone wonky too. It looks like window.ucjs_downloadManagerMain() is failing with an error, which causes the init not to complete successfully. The download box does open however.

[benzBrake]

Where did you download ucjsDownloadsManager.uc.js？4 months ago The XPCOMUtils.defineLazyModuleGetter line was changed to ChromeUtils.defineESModuleGetters。https://github.com/alice0775/userChrome.js/blob/824ae9ae02a91a21ad6e8703f9f8cdf8941bd0a8/117/ucjsDownloadsManager.uc.js#L105C7-L107

[LummoxJR]

Ah, my script might be out of date then. I'll take a look and compare to see what else I need to change.

[LummoxJR]

So I updated to Alice's newest version for 117 and it didn't fix the issue, as of its current build. Looks like I'll need to report a bug there at some point. The remaining error I had was in ucjs_downloadManagerMain.init():

			// xxx remove in-content css
-			var elements = document.childNodes;
-			for (var i = 0; i <= elements.length; i++) {
-				var element = elements[i];
-				if (element.nodeValue && element.nodeValue.indexOf("chrome://browser/skin/downloads/contentAreaDownloadsView.css") > -1) {
-					document.removeChild(element);
-					break;
-				}
-			}
+			let element = document.querySelector('link[href$="skin/downloads/contentAreaDownloadsView.css"]');
+			if(element) element.parentNode.removeChild(element);

There are two problems with the old code. The first is that the <= in that loop was wrong all along, but it never triggered because of the fact that the loop was finding the right element before and breaking out before it got beyond the length of the array. The second problem is because this search for the CSS link was brittle, when the structure of the download manager changed in Firefox 120 it no longer found the old CSS link to remove.