StyloaiX editor is broken

[srjjgm]

Firefox 148 beta broke the StyloaiX editor. The window opens, but everything is blank and nothing works.

This extension is extremely important. Can anyone help?

[117649]

The reason is the UC object is not load into dialog window.

Looks like related to onemen/TabMixPlus#494

[117649]

So I see.
Dialog window does not have this 'about:blank' → 'realUrl' double dip in events pool.
@onemen @srjjgm @AlexVallat

I did this

observe: function (aSubject) {
    if (aSubject.document.isUncommittedInitialDocument) {
      const parent = aSubject.parent;
      aSubject.addEventListener("DOMContentLoaded", (aEvent) => {
        /*NEW*/if(aEvent.target.location != 'about:blank') this.handleEvent(aEvent);/*NEW*/
        else parent.addEventListener("DOMContentLoaded", this, {once: true, capture: true})
      }, {once:true})
    } else {
      aSubject.addEventListener('DOMContentLoaded', this, {once: true, capture: true});
    }
  },

in userChrome.js.

[onemen]

@srjjgm

Please report how it work, so we can include this is the script for all users

[srjjgm]

@onemen @117649

It worked! You can include this fix in the script.
Thank you very much!

[quivenda]

I'm using Firefox Dev Edition 148.0b1 (64-bit).

The script all work, but the icons not showing at all (Loader, StyloaiX, etc) use the configuration files from Tab Mix Plus onemen/TabMixPlus/issues/458.

[onemen]

@quivenda

try this:

observe: function (aSubject) {
    if (aSubject.document.isUncommittedInitialDocument) {
      const parent = aSubject.parent;
      aSubject.addEventListener("DOMContentLoaded", (event) => {
        if(event.target.URL !== 'about:blank') {
          this.handleEvent(event);
        } else {
          parent.addEventListener("DOMContentLoaded", this, {once: true, capture: true})
        }
      }, {once:true})
    } else {
      aSubject.addEventListener('DOMContentLoaded', this, {once: true, capture: true});
    }
  },

[onemen]

@117649

maybe this is better?
if (event.target.baseURI !== "chrome://browser/content/browser.xhtml") {

[117649]

@117649

maybe this is better?
if (event.target.baseURI !== "chrome://browser/content/browser.xhtml") {

You've tried it?
I don't really have any ideas on all the edge cases may be out there.

As for the icons I remember it's more of a style thing. The elements tag of the button have been changed. The style property have to match it. Which are inline within the script.

[onemen]

@117649

It all work:
if (event.target.baseURI !== "chrome://browser/content/browser.xhtml") {
if(event.target.URL !== 'about:blank') {
if(event.target.location != 'about:blank') {

I prefer not to use event.target.location with !=

[117649]

I prefer not to use event.target.location with !=

Very well.
But I still kinda curious about why.

[onemen]

I think it is better to use !event.target.isInitialDocument since it is standard lifecycle of property of the documents

event.target.location.href !== 'about:blank' is also working

[117649]

@onemen
You know how to fix FF toolbox display eval-ed code instead of script content while script has eval in sandbox?

[onemen]

@117649

Your screenshoot show .jsm file
I don't know what code you need to run

[117649]

@117649

Your screenshoot show .jsm file
I don't know what code you need to run

That doesn't matter. It's basically same as .js now.
The problem however won't care if it's js or esm file.