ericcornelissen/pinned-tabs-for-atom

Error pinning multiple untitled tabs

Closed this issue · 7 comments

bergerr commented

I had an untitled pinned tab and tried to pin another one, and it popped an error with the trace below:

/Users/rxb2/.atom/packages/pinned-tabs/lib/state.js:126
Hide Stack Trace
TypeError: Cannot read property 'id' of undefined
at /Users/rxb2/.atom/packages/pinned-tabs/lib/state.js:126:53
at Array.findIndex ()
at PinnedTabsState.movePaneItem (/Users/rxb2/.atom/packages/pinned-tabs/lib/state.js:126:30)
at PinnedTabs.reorderTab (/Users/rxb2/.atom/packages/pinned-tabs/lib/pinned-tabs.js:347:18)
at /Users/rxb2/.atom/packages/pinned-tabs/lib/pinned-tabs.js:196:35

The tab abr also rapidly flickered between the pinned tab and the tab I was trying to pin, until I unpinned the original one. After trying a few more times I was able to pin both untitled tabs.

ericcornelissen commented

Thanks for the report @bergerr, I can reproduce it and it seems to be a real problem (though I don't think it is very common to pin two new untitled tabs 😅).

I think I found a solution that fixes the problem. Would you mind testing this package on branch bug-61/pin-multiple-untitled-tabs for me to see if it did? (if you need help with how to do that, please ask)

bergerr commented

Sure, how do I install the package from a specific branch?

ericcornelissen commented

To the best of my knowledge you have to clone the repository into your .atom/packages directory (make sure to remove the package directory first), check the branch out (run npm install) and reload or restart Atom.

bergerr commented

The branch works fine for me. 👍

ericcornelissen commented

Thanks for testing, I'll publish a new release based on that branch as soon as possible 👍

ericcornelissen commented

The patch should now be published, let me know if the problem was not (completely) fixed 🙂

chrismheath commented

Used to have the same issue with pinned tabs - all works for me now too. Thanks for the fix!

  • 🎉1