Add the Ability to Set the Hotbar Slot for the Sprinkler

[thesinmin]

Have you searched the existing suggestions?

Yes

What is your suggestion?

Rather than the current hard-coded expectation of a user having the sprinkler in slot 1 of the hotbar, I'd suggest adding an option to set it to any slot.

How will this improve Natro Macro?

People who have their hotbars setup differently would not have to re-learn their hot keys after implementing Natro.

Note that I have added this to several versions locally but getting it into the actual code would save me having to update the macro each time a new version comes out and would make it available to any others who might be weird work differently like I do.

Note2: I'm happy to work on coding this if the idea is acceptable to the team. This would be my first open-source submission, so I'm not promising an immediate turnaround.

What is your Discord username?

SinMin#7776

[Axetar]

Yeah! Go ahead an make a pull request!

[zspz]

I'm reopening this because it is a good suggestion and would simplify things a lot. I am still seeing newer users have other items wrongly set to hotbar slot 1 when using the macro.

Instead of having the option to set it (which can still lead to users choosing the wrong option / forgetting to set it), I would instead detect which slot the sprinkler is in and save that for use throughout the macro.

[thesinmin]

That sounds like a great idea. Guess it's time for me to read up on the gdip library,
With that said, would it then make sense to just assign the hotbar items at startup rather than leaving it to the user to choose each one?

[zspz]

@thesinmin
Sorry for the incredibly late response! I didn't see this comment.
Yes, I think that can definitely be added in a future update, instead of assigning slots 2-7 as users currently do in Boost tab.
This would require some extra consideration, including how the user can set options for dynamic slots.
So I think it would be fine to detect Sprinkler on startup (i.e. after the ;check UIPI section) for now, and assign that slot to a global variable (between sc002 (1) and sc008 (7)) which is called instead of Send {%SC_1%}.

[peacocksir]

im going to be working on this