refactor forward/backwards loop

[jetersen]

we could add a subtype = "mage" to simplify the flyoutId == 1 or flyoutId == 8 or flyoutId == 11 or flyoutId == 12 😅

[Justw8]

I wanted to do that, but saw subtype wasn't passed on yet. should just pass the full flyout information as table and not seperate arguments though.

[jetersen]

let me merge the two prs and cleanup passing the flyout info 👍

[Justw8]

All the comments confuse me - usually the author himself isn't the one doing this.

Could you do whatever you want from the comments, and then I can review/merge the final changes?
And since I merged the locale PR there's a conflict that needs fixing too.

[jetersen]

Reviews are a two way street? :) This way I can communicate my thoughts 😅

[jetersen]

@Justw8 Ready for review

[jetersen]

@Justw8 FYI you can turn off comments in the file review:

I personally use the GitHub PR review as it shows the code differently than in the editor and I am usually able to catch mistake.