Add Event Trigger for Infinite Fuel Toggle

[coleminer0112]

Allows custom fuel resources to watch for infinite fuel in vMenu within other client-sided scripts.

Event name: "vMenu:InfiniteFuelToggled"
Passes one arg, the boolean representing whether the box is checked or not

[MichaelCoding25]

I'm pretty sure you also need to register the event, so people can just AddEventHandler to it in their script.

[coleminer0112]

I'm pretty sure you also need to register the event, so people can just AddEventHandler to it in their script.

It doesn't require registering unless it's going across the Net, which it isn't - it is a client side only event

[TomGrobbe]

I have a suggestion for an additional change, that'd make this more useful, whenever the resource loads and it sets the original value form the saved preferences, it should trigger the event once so your other resource has a chance of knowing what the status is once a player joins and doesn't toggle it manually. Or exposing an export function that gets the current value.

[TomGrobbe]

Other than that it's a useful feature

[TomGrobbe]

Could you rebase your changes based on the latest development branch and resolve the conflicts please?

[TomGrobbe]

Comments need to be resolved

[coleminer0112]

I have a suggestion for an additional change, that'd make this more useful, whenever the resource loads and it sets the original value form the saved preferences, it should trigger the event once so your other resource has a chance of knowing what the status is once a player joins and doesn't toggle it manually. Or exposing an export function that gets the current value.

The current development version has no userDefault for this setting, so it may not be necessary to do an initial check!

Also, I rebased, but I am not a Git-master... so I am unsure why it is now 3 commits despite only the 1 change.