UltraTabSaver icon indicating copy to clipboard operation
UltraTabSaver copied to clipboard

Ignore pinned tabs

Open pieter91 opened this issue 5 years ago • 6 comments

Hi! Thanks so much for this extension. It's been really helpful so far.

The only downside to the functionality I've found is that pinned tabs aren't treated any differently, meaning they are saved and closed along with all the regular tabs. When closing they also close on other opened windows.

Would it be possible to (set the plugin to) ignore any pinned tabs?

I went looking into this, but couldn't find any relevant documentation from Apple. Is there a way to detect wether a tab is pinned?

pieter91 avatar Jun 28 '20 13:06 pieter91

Hey, thank you for the feedback. You are right, I'm going to see if I found any way to don't save the pinned tabs.

PS: You can star the repo to see the updates.

I would like to know more opinions about other users to see if it is better to:

  1. Save the pinned tabs
  2. Don't save the pinned tabs
  3. Make it configurable

morsamatias avatar Jun 29 '20 21:06 morsamatias

I just opened an issue about pinned tabs in the Session Buddy repo with the same problem.

My workflow (and I guess of many other users) is: having a handful of pinned tabs which are necessary to my work as I need and use them all day long.

What I would like to see: a checkbox "Ignore pinned tabs". When activated saving a session should not save them. When closing tabs the same applies: do not touch them.

So basically the checkbox means "I really know what I am doing here, please leave my pinned tabs alone".

This (at least for me) is a necessary feature that currently (and unfortunately) prevents me from using the extension. I know, every user thinks his feature request is the most important one 😁 just wanted to leave a vote with an explanation why such a preference might be crucial to some users.

kevinpapst avatar Sep 23 '20 18:09 kevinpapst

Hi @kevinpapst thank you for the feedback!. Where is the Session Buddy repo, can you share the link please? Actually I had just an idea to fix them, but maybe it's more tedious, The user will have to add one by one the urls of the pinned tabs, and he can manage them from the menu, what do you think?

morsamatias avatar Sep 26 '20 02:09 morsamatias

@morsamatias I didn't wanted to post a link to a competitor, thats why I didn't add it initially. It is called "Session Pal" in the AppStore. Here it is: https://github.com/dwarvesf/session-buddy/ and this is my issue which I posted before I found your plugin.

I know it often looks easier from the outside, I was hoping (without knowing the available APIs) that there might be a simple solution like a bool flag isPinned on the tabs that you save. Then you could simply ignore each tab which has true as result.

Following a couple of discussions it seems there isn't. But it seems to be possible to detect with a workaround by checking the containingWindow for nil as discussed here or here. I have almost zero Swift knowledge, maybe there could be a check added somewhere in saveAllPages()?

kevinpapst avatar Sep 26 '20 10:09 kevinpapst

@kevinpapst Have you used Session Pal? It's good?

You are right it looks like getContainingWindow is the answer, give me a couple of days and I'm going to add this feature.

Thank you for the help, feel free to add more issues to keep improving this project!

morsamatias avatar Sep 26 '20 12:09 morsamatias

I also would really like to see this feature added.

Tukan3 avatar Dec 09 '20 16:12 Tukan3