[Ext] Tab Utilities 1.5 - light but featureful

[Brummelchen]

v22 is going to be released next days - folders are prepped. anyway - this become ridiculous.

[vertigoback]

... and ff22 is out. Just had to disable the Add-On. I guess ithinc is not around any longer, so thanks for making my live easier for so long.

[Limerick]

Perhaps he died as he has never given any news since 1.6pre4?

[jetman]

Perhaps he died as he has never given any news since 1.6pre4?

Dude, don't be so over-dramatic. Life intervenes sometimes and I doubt this is a full-time job for him/her....

[Ryrynz]

Yeah life intervenes and you you can't stick to your word even 2 months after you said you'd do something.
Back to Tabmix plus for me.

[blackwind]

As far as I can tell, TU (1.6pre4) works perfectly in Fx22 if you simply comment out one of the Fx16 fixes in tabutils.js.

BEFORE:

Code: Select all

  TU_hookCode("gBrowser.addTab",
    [/if \(arguments.length == 2[^{}]*\) {[^{}]*}/, "$&" + (function() {
      if (!isBlankPageURL(aURI)) {
        let t = aFromExternal && isTabEmpty(this.selectedTab) && this.selectedTab || this.getBlankTab();
        if (t) {
          let b = this.getBrowserForTab(t);
          return t;
        }
      }
    }).toString().replace(/^.*{|}$/g, "")],
    [/(?=return t;)/, gBrowser.addTab.toString().match(/var (uriIsBlankPage|uriIsNotAboutBlank).*|if \(uriIsNotAboutBlank\) {([^{}]|{[^{}]*})*}/g).join("\n")] // Bug 716108 [Fx16]
  );


AFTER:

Code: Select all

  TU_hookCode("gBrowser.addTab",
    [/if \(arguments.length == 2[^{}]*\) {[^{}]*}/, "$&" + (function() {
      if (!isBlankPageURL(aURI)) {
        let t = aFromExternal && isTabEmpty(this.selectedTab) && this.selectedTab || this.getBlankTab();
        if (t) {
          let b = this.getBrowserForTab(t);
          return t;
        }
      }
    }).toString().replace(/^.*{|}$/g, "")]/*,
    [/(?=return t;)/, gBrowser.addTab.toString().match(/var (uriIsBlankPage|uriIsNotAboutBlank).*|if \(uriIsNotAboutBlank\) {([^{}]|{[^{}]*})*}/g).join("\n")] // Bug 716108 [Fx16]*/
  );


I'm not sure what the holdup is from ithinc. This fix couldn't have been easier to track down.

[Floyd77]

blackwind, thanks a lot! I've just modified the file as you said and it works like a charm! For anyone who doesn't know how to do this, I've uploaded the modified xpi here:
(Edited to post new link)

[blackwind]

It seems the above fix kills the "reuse blank tabs" function. Here's a better one:

Code: Select all

  TU_hookCode("gBrowser.addTab",
    [/if \(arguments.length == 2[^{}]*\) {[^{}]*}/, "$&" + (function() {
      if (!isBlankPageURL(aURI)) {
        let t = aFromExternal && isTabEmpty(this.selectedTab) && this.selectedTab || this.getBlankTab();
        if (t) {
          let b = this.getBrowserForTab(t);
          return t;
        }
      }
    }).toString().replace(/^.*{|}$/g, "")],
    [/(?=return t;)/, gBrowser.addTab.toString().match(/var (uriIsAboutBlank).*|if \(!uriIsAboutBlank\) {([^{}]|{[^{}]*})*}/g).join("\n")] // Bug 716108 [Fx22]
  );


[rgp]

Fix #2 works great!

Thank you blackwind!

[Limerick]

@blackwind, perhaps your fix has nothing to do with the Multi-Row tab bar (I am absolutely not a coder or developper ), but I have change the code right above in the existing extension on FF21 (which is okay with 1.6pre4), and I lost the MRtb.
So I haven't try with FF22...

EDIT : rgp, would you mind publishing your .xpi as Floyd77 kindly did previously ?

[rgp]

xpi with blackwind's fix #2, appears to work well in FF 22.

http://www.mediafire.cnn780jhkuqh3xt

[Floyd77]

Limerick, this is the new xpi with the second blackwind fix applied: http://www.mediafire.coad/vw86gwu1saw11 ... re4-fx.xpi

Multi-row tab bar works good, but I don't know how to test "reuse blank tabs" because I don't know what is supposed to do nor how to configure it xD

[Limerick]

Thanks to both of you guys for your .xpi's.

But I don't for what reason, MTrb doesn't work with both of them...

For each of your file, I :

- uninstalled previous TU version (1.6pre4 from ithinc), restarded FF21, installed your files, restarted again and... no more MRtb
- kept previous TU version, upgraded with your files, restarded FF21 and... no more MRtb
- uninstalled your files, resinstalled 1.6pre4 from ithinc and... recovered MRtb

I wonder why your files don't work for me

[Floyd77]

Test with Firefox 22, for Firefox 21 you don't need this fix.

[Limerick]

Okay, you mean that it is normal that this fix doesn't work with FF21?
I mean this is normal that the fix seems buggy with FF21, but not with FF22?

EDIT : thanks to all you three. I did what you said Flyod (install FF22 prior to install the fix), and I recovered the MRtb

BTW, would any developper (like you blackwind) take over ithinc's work in order to keep his brilliant and useful tool updated ?