Jump to content

[1.1.3] Part Wizard (1.2.5) [22-Jun-2016]


ozraven

Recommended Posts

  • 2 weeks later...

Aaand here we go:

 
D:\SteamLibrary\SteamApps\common\Kerbal Space Program>ckan show PartWizard
Part Wizard: Part Wizard is a vehicle design utility plugin that adds a few conveniences when building your next strut/booster carrier.

Module info:
- version:      1.1.2
- authors:      ozraven
- status:       stable
- license:      restricted

Depends:
- Toolbar

Provides:
- PartWizard

Resources:
- homepage: http://forum.kerbalspaceprogram.com/threads/80124

Showing 8 installed files:
- GameData/PartWizard
- GameData/PartWizard/ATM.cfg
- GameData/PartWizard/Icons
- GameData/PartWizard/Icons/partwizard_active_toolbar_24_icon.tga
- GameData/PartWizard/Icons/partwizard_inactive_toolbar_24_icon.tga
- GameData/PartWizard/Plugins
- GameData/PartWizard/Plugins/PartWizard.dll
- GameData/PartWizard/README

- - - Updated - - -

{snip}, wrong license selected... will be fixed immediately.

Edited by Gargamel
Portions Redacted by moderator
Link to comment
Share on other sites

  • 1 month later...

Just heard the guys on Kerbalcast mention this after a listener recommended it to them. The specific thing they mentioned it can do (save a part you've gizmoed too far into its parent part) is something I've wanted since the first day I used 0.90.

I'm surprised more people aren't singing your praises, and I'm downloading this now!

Link to comment
Share on other sites

  • 2 months later...
Version 1.2.0 release, see OP for details, or straight to Curse for download.

If you might be interested, I could fork your mod and submit a pull request to add support for both Stock and Blizzy toolbar, with hot switching. I do this In Ship Manifest. I'm sure you could do it your self tho just by looking at my spagetti code. :D

Link to comment
Share on other sites

If you might be interested, I could fork your mod and submit a pull request to add support for both Stock and Blizzy toolbar, with hot switching. I do this In Ship Manifest. I'm sure you could do it your self tho just by looking at my spagetti code. :D

I would very much accept such a PR, with many thanks. :) While I am firmly in the Blizzy Toolbar camp, I'm more than happy to accommodate other preferences. It should be pretty straight forward - the Toolbar integration is entirely within PartWizardPlugin.cs. Thanks Papa_Joe.

Link to comment
Share on other sites

I would very much accept such a PR, with many thanks. :) While I am firmly in the Blizzy Toolbar camp, I'm more than happy to accommodate other preferences. It should be pretty straight forward - the Toolbar integration is entirely within PartWizardPlugin.cs. Thanks Papa_Joe.

I'm with you. Blizzy's toolbar is FAR superior in every respect. But I believe in choices for the community. I'll get right on it. :D

Also, I will make Blizzy a soft dependency, so that the user is not forced to install blizzy if they choose not to use it.

Edited by Papa_Joe
Link to comment
Share on other sites

I'm with you. Blizzy's toolbar is FAR superior in every respect. But I believe in choices for the community. I'll get right on it. :D

Also, I will make Blizzy a soft dependency, so that the user is not forced to install blizzy if they choose not to use it.

Thanks for the effort Papa_Joe.

Link to comment
Share on other sites

I would very much accept such a PR, with many thanks. :) While I am firmly in the Blizzy Toolbar camp, I'm more than happy to accommodate other preferences. It should be pretty straight forward - the Toolbar integration is entirely within PartWizardPlugin.cs. Thanks Papa_Joe.

Ok, before I submit the pull request, I have a bit of re-factoring to do, but I also wanted to post a couple of screenshots so you can see what I did. If you are good with it, I'll clean up and submit the PR.

Blizzy:

oSZUBcK.png

Switched to stock:

ZXeAV3o.png

Note where I placed the option. It persists to the config just like the icon strings do.


PART_WIZARD_SETTINGS
{
toolbarIsBlizzy = True
toolbarActiveIcon = PartWizard/Icons/partwizard_active_toolbar_24_icon
toolbarInactiveIcon = PartWizard/Icons/partwizard_inactive_toolbar_24_icon
PART_WIZARD_WINDOW
{
x = 1388
y = 677
width = 300
height = 400
}
}

Link to comment
Share on other sites

Ok, before I submit the pull request, I have a bit of re-factoring to do, but I also wanted to post a couple of screenshots so you can see what I did. If you are good with it, I'll clean up and submit the PR.

Note where I placed the option. It persists to the config just like the icon strings do.

The toggle seems fine where you put it, though I think it may be time to add a settings window, in my next update. In your experience, would it be possible to simply use Toolbar when its present, and stock if it is not? (I wonder if a consensus may have formed from Ship Manifest's thread, or elsewhere you have read.)

The only change request I do have is to make the setting string "toolbarIsStock" rather than "toolbarIsBlizzy". Thank you for taking the time to pitch in on the code for stock toolbar support, and I'll post the 1.2.1 release ASAP, once you have things ready to send over on your end. :)

Link to comment
Share on other sites

The toggle seems fine where you put it, though I think it may be time to add a settings window, in my next update. In your experience, would it be possible to simply use Toolbar when its present, and stock if it is not? (I wonder if a consensus may have formed from Ship Manifest's thread, or elsewhere you have read.)

The only change request I do have is to make the setting string "toolbarIsStock" rather than "toolbarIsBlizzy". Thank you for taking the time to pitch in on the code for stock toolbar support, and I'll post the 1.2.1 release ASAP, once you have things ready to send over on your end. :)

I'll rescind my pull request, and make the changes. As for the behavior, it is currently set to automatically use Toolbar by default. if toolbar is not present, it reverts to stock. So that is what you are asking for I believe. I added the option for those that may have toolbar and "still" want to use stock.

I'll change the the option to toolbarIsStock and if you like I can change to language in the toggle to "Use Stock Toolbar" and have it Off by default.

Edited by Papa_Joe
Link to comment
Share on other sites

I'll rescind my pull request, and make the changes. As for the behavior, it is currently set to automatically use Toolbar by default. if toolbar is not present, it reverts to stock. So that is what you are asking for I believe. I added the option for those that may have toolbar and "still" want to use stock.

Ah, I was mostly wondering if by automating the decision the toggle could be removed from the UI. The toggle feels a bit awkward there, but as there's no proper place to put settings, so there aren't many choices. :)

Cool, and again, thanks for the PR! I expect I'll be able to get the new build out tomorrow morning, EDT.

Link to comment
Share on other sites

Ah, I was mostly wondering if by automating the decision the toggle could be removed from the UI. The toggle feels a bit awkward there, but as there's no proper place to put settings, so there aren't many choices. :)

Cool, and again, thanks for the PR! I expect I'll be able to get the new build out tomorrow morning, EDT.

I agree, it did seem a bit awkward there. I thought about putting it at the bottom after the status display, but wasn't sure, hence the pics for your review. I can still remove the option if you wish, or if you make a settings window you can move it there then.

Your choice.

Link to comment
Share on other sites

I agree, it did seem a bit awkward there. I thought about putting it at the bottom after the status display, but wasn't sure, hence the pics for your review. I can still remove the option if you wish, or if you make a settings window you can move it there then.

Your choice.

It's fine where it is, if anything it will become motivation to add a settings window. ;)

Link to comment
Share on other sites

It's fine where it is, if anything it will become motivation to add a settings window. ;)

Sounds good.

Pics.

Blizzy installed, Blizzy active | Blizzy installed, Stock active | Blizzy not installed.

oq0XV7i.pngyt8YIKw.png8PLcJOT.png

Config:


PART_WIZARD_SETTINGS
{
toolbarIsStock = True
toolbarActiveIcon = PartWizard/Icons/partwizard_active_toolbar_38_icon
toolbarInactiveIcon = PartWizard/Icons/partwizard_inactive_toolbar_38_icon
PART_WIZARD_WINDOW
{
x = 1522
y = 633
width = 300
height = 400
}
}

PR submitted.

For the Stock icon Images, I just resized the canvas for the images to 38x38. Linked here if you want them.

https://github.com/PapaJoesSoup/PartWizard/releases/tag/1.2.0.0x

Edited by Papa_Joe
Link to comment
Share on other sites

  • 1 month later...
This thread is quite old. Please consider starting a new thread rather than reviving this one.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...