Code Monkey home page Code Monkey logo

Comments (26)

captnfab avatar captnfab commented on July 1, 2024

Crap. I'll try to reproduce that and fix it as soon as possible. I'll ping you if I need help to reproduce the bug.

from firetitle.

tomforwood avatar tomforwood commented on July 1, 2024

Same here, except all the window titles are being overwritten with the title of one window. Probably the last window to have focus when I quit firefox

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

Thx tomforwood, So, the bug is confirmed. I'll try to fix it ASAP…

from firetitle.

rsalmei avatar rsalmei commented on July 1, 2024

Unfortunately it isn't working for me too. With all updated, macOS 10.13.2, Firefox 57.0.1 and Crappy 0.1.5.
Sometimes only my "main" window gets its name back (first one), sometimes none of them...

from firetitle.

frasierlinde avatar frasierlinde commented on July 1, 2024

Same here. And I just noticed that as each window opens, it shows its previous title for a split second before resetting to "New Window."
W7SP1, FF 57.0.4, CFT 0.1.5

from firetitle.

alvaro-cuesta avatar alvaro-cuesta commented on July 1, 2024

To debug this you can go to about:debugging, enable extension debugging and then click Debug on FireTitle (and accept the connection request).

Then just paste this on the debugging console:

browser.windows.getAll()
.then(ws => Promise.all(ws.map(({ id, title }) => Promise.all([
  id,
  title,
  browser.sessions.getWindowValue(id, 'win_name'),
]))))
.then(ws => ws.forEach(([ id, title, name ]) => console.log(`[${id}] ${title} = ${name}`)))

It list window ids, their title and the saved title.

For me the window ids didn't change across restarts, but the saved names were overwritten, which means at some point browser.sessions.setWindowValue was being called with the default name, overwriting the saved settings.

While debugging this I found sometimes some of my window names were restored and others didn't. I'm pretty sure this is a race condition regarding events.

I think I found the culprit:

function actionOnNewWindow(win)
{
let pLocalStor = browser.storage.local.get();
let pCurrentWinTabs = browser.tabs.query({windowId: win.id, active: true});
let pSaved = pLocalStor.then((localStor) => { saveWindowOptionsToSession(win.id, localStor.def_win_name, localStor.def_win_pattern); });
let pDone = Promise.all([pCurrentWinTabs,pSaved]).then((currentWinTabs) => { refreshWinTitleOfTab(currentWinTabs[0].id) });
return pDone;
}

I'm not that knowledgeable about WebExtensions but I hope this can help someone else fix the issue.

from firetitle.

jpfleury avatar jpfleury commented on July 1, 2024

I too have this bug with Firefox 59 and Firefox 60 beta. Every time I restart the browser, all but one or two windows lose their custom name.

from firetitle.

cnorman-enphase avatar cnorman-enphase commented on July 1, 2024

I am also having this problem. 60.0.2 on macOS 10.12.6. Pretty much as jpfleury described. Is there a patch available?

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

Not yet, I'm sorry I haven't had time to work on it. I'll try to fix that tomorrow.
Thanks for the debug input.

from firetitle.

paolo-crisafulli avatar paolo-crisafulli commented on July 1, 2024

For the record, I also get this issue: Firefox 60.0, Ubuntu 16.04.4 LTS

Too bad, I really like this plugin (not so "crappy" indeed): combined with the Multiple Tab Handler plugin, it helps me a lot in the mourning process for Tab Groups :-)

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

I kind of blindly tried to solve this bug (since I can't reproduce it).
Version 0.1.6 should avoid the race condition reseting the title on restore.
Please try it and tell me if it does work.

from firetitle.

cnorman-enphase avatar cnorman-enphase commented on July 1, 2024

from firetitle.

cnorman-enphase avatar cnorman-enphase commented on July 1, 2024

Woops, I was wrong. I restarted my Mac and loaded Firefox and... only 2 got restored. the Rest are named "New Window"... :(

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

:(
I'll try something else.

from firetitle.

starcat13 avatar starcat13 commented on July 1, 2024

I have the same problem under Windows, can test too. Or provide any extra information, if it helps.

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

@starcat13: I'm quite sure the bug will still be there for you too. I just have to find where that race condition happens...
But thanks, I'll try to release a new version the next hour and I'll notify you so that you can do some testing.

from firetitle.

starcat13 avatar starcat13 commented on July 1, 2024

thank you!
Not a critical bug for me, I rarely restart FF.

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

0.1.8 is available :)

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

@starcat13 @cnorman-enphase : any change? did it fix it?

from firetitle.

starcat13 avatar starcat13 commented on July 1, 2024

I tried restart FF and even reboot my computer several times and each time all window names were restored.
Thank you again!

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

Great, thanks for the testing!

from firetitle.

ExchangeGuy avatar ExchangeGuy commented on July 1, 2024

Works for me!

All CFT Titles (8 named windows, 100+ tabs) restored in three scenarios: Firefox updated/restarted (by FF), killed the FF process then started FF, restarted computer (Win10). Worked great after each! (Would always fail before, I would loose all Titles.

This update is very, very much appreciated. Now a major stumbling block after 57 is gone.

Thanks,
Lee~

P.S. I put five dots..... after any Title I set with CFT. Least visible character (as multiple spaces don't work), gives a little separation between Title..... and everything else.

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

@ExchangeGuy Thanks for the feedback.

Instead of dots, you can use a non-breaking space https://en.wikipedia.org/wiki/Non-breaking_space
On windows: Alt+0160 or Alt+255
On Linux (depending on keymap): Compose, Space, Space or AltGr+Space or Shift+Space
On macOS: Opt+Space

You can add multiple non-breaking spaces, it works.

All kind of unicode chars make pretty separators too: https://en.wikipedia.org/wiki/Geometric_Shapes

from firetitle.

captnfab avatar captnfab commented on July 1, 2024

@paolo-crisafulli @jpfleury @rsalmei @tomforwood @frasierlinde and others I already notified: I believe this problem is now solved. I apologize for letting such an annoying bug alive for so long :)

from firetitle.

cnorman-enphase avatar cnorman-enphase commented on July 1, 2024

from firetitle.

paolo-crisafulli avatar paolo-crisafulli commented on July 1, 2024

Thank you, seems to work for me too!

from firetitle.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.