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

Hot swap dev button properly placed #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ilyvion
Copy link

@ilyvion ilyvion commented May 25, 2024

In addition to the titular change, I've also done a few small quality-of-life changes:

  • HotSwap.csproj now has a property RimWorldVersion that can be set to 1.4 to build for 1.4 and 1.5 to build for 1.5. Defaults to 1.5 if not overridden. This sets the right OutputPath and depends on the correct Krafs.Rimworld.Ref version in each case.
  • Fixed a tiny bug in HotSwapKeyDefOf.cs where the DefOfHelper.EnsureInitializedInCtor was called on the wrong type
  • Created and assigned a custom hot swap icon to the dev button instead of using the icon for paste:
    HotSwap
  • Also added a mod icon for extra snazziness on 1.5:
    ModIcon

I have not updated the dlls because I don't think it's sensible to rely on external contributors providing safe ones in PRs.

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

Successfully merging this pull request may close these issues.

1 participant