-
Notifications
You must be signed in to change notification settings - Fork 38
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
Optional integration with the Toolbar mod #42
Optional integration with the Toolbar mod #42
Conversation
Updates for 1.0
Fixed bug w/Distributed Warehouse
One thing I forgot to mention: blizzy's toolbar requires 24x24 icons or smaller, so scaled-down versions of the icons will need to be created. I can add those, but I'm not sure what your release process is, so I'm not sure where you'd want them in the repositories. |
Very nice - if you could just toss them in the same location as the stock ones that would be good and I can merge this in. |
I've added 24x24 versions of the icons next to the original icons. They're named with a "24" suffix, so they're easy to distinguish. |
Optional integration with the Toolbar mod
I've added code to automatically make use of the Toolbar mod if it is available, and continue using the stock toolbar when the Toolbar mod isn't available. I put the toolbar wrapper in USITools and make use of it in both USI-LS and in MKS, so there will be 3 separate pull requests (one for each repository).
While I was at it I noticed MKS issue 533 and fixed it (the toolbar button would just keep opening new windows instead of opening/closing the same window). This fix applies to both the stock toolbar and the Toolbar mod.