Panorama-tab-groups: Export doesn't work (clicking "Save backup" does nothing)

I'll check on this, I don't really use the backup feature myself (and someone else has worked on this in the past), I'm sorry I didn't miss this feature breaking. I'll go check on it when I can and see why it doesn't work.

The code stops at this line:
https://github.com/projectdelphai/panorama-tab-groups/blob/master/src/js/backup.js#L106
The console says TypeError: Argument 1 of StructuredCloneHolder.deserialize is not an object.

🤔

Sadly, it seems the debugger doesn't work properly, I can't even step into the code and look for the part which stops.

This states that it's coming from here:
https://searchfox.org/mozilla-central/source/toolkit/components/extensions/ExtensionChild.jsm#1016

The odd thing is, this logs correct:

browser.sessions.getWindowValue(windows[wi].id, 'groups').then((data) => {
    console.log('groups', data);
});
browser.sessions.getWindowValue(windows[wi].id, 'groupIndex').then((data) => {
    console.log('groupIndex', data);
});
browser.sessions.getWindowValue(windows[wi].id, 'activeGroup').then((data) => {
    console.log('activeGroup', data);
});

and this not:

const groups = await browser.sessions.getWindowValue(windows[wi].id, 'groups');
const groupIndex = await browser.sessions.getWindowValue(windows[wi].id, 'groupIndex');
const activeGroup = await browser.sessions.getWindowValue(windows[wi].id, 'activeGroup');
console.log('groups', groups);
console.log('groupIndex', groupIndex);
console.log('activeGroup', activeGroup);

Just wanted to say that this feature is also extremely important to me as well. If Firefox's "Restore previous session" gets unchecked, as happened to me with the v.65 update, you get the "Do you want to close all tabs" when you exit and can't exit without doing it. This causes all tab groups to be lost. Keeping a backup lets you import those tab groups back in. It's a lifesaver.

In case of @HGondorff an automatic backup would be nice. But this would be a different feature.

Just want the Save Backup button to work again, please. Hope someone figures it out.

I've lost all my tab groups multiple times with various Tab Groups extensions over the years (including this one). I don't have the logs/details to reproduce it, but I'd simply open up Firefox one day and find all tab groups as one huge group.

Posting this to +1 a backup/save feature fix

Figured it out, it was extremely stupid. update incoming

It's fixed, basically when some options were added to the options page in the shape of a \ which meant that the save backup button was counted as the button for the form. And because it's not a proper form, the button didn't do anything. Closed the form properly and the button works now. Easy mistake to make and easy to fix, just hard to find :)

git bisect was particularly helpful

@projectdelphai Something got eaten in your comment. ;) Did you use an HTML tag? Markdown understands HTML so if you post code you need to wrap it in backticks or we won't see it.

The update is not coming through. I'm still stuck with 0.8.7. I am tempted to delete and download the new version but am afraid of losing my tabs in the process. I have tried setting the updates to default and automatic to no avail.

@HGondorff this link should help you out if you really can't update. That being said, have you tried clicking the little gear on the right hand side of the PTG options page? It'll allow you to automatically download the new update. Firefox only checks once a day, so that might be why your update hasn't come in yet.

If you have tried to manually update, then there might be something else wrong.

@mgol lol yeah i escaped the html tags so it should be good now, thanks for letting me know. Didn't realize backticks helped escape too, I just used back slashes by default and that worked too.

Ah yes, the little gear I didn't know was there :) All is well and my anxiety level has declined. My sincere gratitude for the fix!

anytime :)