Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Asking the community to help #16

Open
eamodio opened this issue May 19, 2017 · 4 comments
Open

Asking the community to help #16

eamodio opened this issue May 19, 2017 · 4 comments

Comments

@eamodio
Copy link
Owner

eamodio commented May 19, 2017

Unfortunately, at this point in time, I'm unable to really support this extension. I no longer use Visual Studio nor do I even have an environment setup to develop and test this extension.

I certainly don't want to pull this extension from the marketplace or even remove its stated Visual Studio 2017 support because as far as I'm aware it still works very well for most people (even on Visual Studio 2017). Although I know some people are experiencing issues -- gotta love the 1 star reviews 😢. So I'm reaching out to the community for help.

I'd be happy to do my best to support and merge PRs, and push out tested updates.

I look forward to working together with the community.

Thanks

@toddsay
Copy link

toddsay commented May 22, 2017

I'm running VS2017, but with no problems I'm aware of. If I can reproduce any problems experienced by others I'd be happy to work on them, but that sounds like a large IF at this point.

@eamodio
Copy link
Owner Author

eamodio commented May 23, 2017

@toddsay thank you -- my guess (though it is a complete guess) is that many of the users with issues are hitting: #7. @vitidev has made changes to resolve it on a fork, but I don't currently have any way to verify and test them.

@vitidev
Copy link

vitidev commented May 28, 2017

PresentationFramework.Aero2 and his ListBoxChrome does not exists in win7. To fix the bug, I simply replaced ListBoxChrome by default wpf ListBox and remove PresentationFramework.Aero2 dependency

ListBox works, but the appearance may differ from the ListBoxChrome (I never see ListBoxChrome). so I did not make PR. After all, the author for some reason chose ListBoxChrome.

Also I need to remove
RenderMouseOver="{TemplateBinding IsMouseOver}" RenderFocused="{TemplateBinding IsKeyboardFocusWithin}

Looks like this
image 1

@sbenedict
Copy link

I've forked my own version with the following changes:

  • Added support for files external to the solution
  • Added Export/Import functions
  • Added ability to store groups to a file in solution folder. Enabled by adding a blank file: {solution root}\.vs\{solution name}\SaveAllTheTabs.json
  • Fixed fails to save groups to settings store when too large
  • Fixed delete confirmation dialog sizing

I would like to add UI support for selecting to store groups to the solution folder rather than the user store, but found this feature usable in its current state for my needs.
Note I did not add vitidev's fix for Win7 or test in anything other than VS2017.

https://github.com/sbenedict/SaveAllTheTabs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants