why did you PeekInt(bank,12) rather than
PeekInt(bank,16)
PeekInt(bank,16)
Delete that line, and it'll work. You don't need to store a menu item into a bank if you aren't going to change it (e.g. check/uncheck it, or add more submenus). The only thing you need is that '2', which you don't need to store anywhere as it's given by the event system (in EventData()).
I can't explain more on indenting than I already did, you indent the scope of a command, not the command itself.
The relevance of whitelines is to make it more readable, more comfortable to navigate at first sight, and to make it clear which instructions belong together.