search

Panakotta00 / FicsIt-Networks

Control, Monitor, Manage and Automate your Satisfactory.
https://ficsit.app/mod/FicsItNetworks
GNU General Public License v3.0
157 stars 51 forks source link

Make WidgetInteraction a dependency. #140

Closed ptsd closed 3 years ago

ptsd commented 3 years ago

Hi,

I have a couple of WIP mods that require adding a WidgetInteraction component to the player character. As your (Awesome!) mod does the same thing, it seems sensible to pull that small bit of functionality out to a seperate SML dependency.

I'll hold off publishing "PTSDInteraction" to SML until/unless you say it's ok, as it's basically a ctrl-c ctrl-v of your implementation.

PTSDInteraction Source

This PR:

I've also added the ability to register widgets as desiring KB focus on primarybutton press, with a caveat listed in the PTSDInteraction readme.

I'm unable to properly test FIN with this change as the %ASSETS% patch doesn't seem to be public and ALPAKIT won't complete without some BP changes.

I have tested by adding PTSDInteraction as a dependency to a fresh mod, and all interaction stuff works as expected.

On a side note, your amazing mod is a brilliant reference for anyone learning Satisfactory C++ modding, huge respect for making the source public 👍

Panakotta00 commented 3 years ago

Thanks ^^

I think it would rather be better to put it directly into SML instead of a separate dependency mod.

With a separate dependency mod I'm at the mercy of that mod in terms of game updates etc. and that's something nearly no one wants.

ptsd commented 3 years ago

Makes a lot more sense :)

I'll have a look at adding it.