Yeah, that's true. We were originally performing the action on the menu item, but that would be better now that things have come full circle.
@jaimeyr, that still doesn't help me understand. What you have now are two buttons, blue and red. If you push the blue button and counter is not 1, counter will become 2. Otherwise no matter how many times you push, it won't do anything. If you push the red button and counter is 2, it will become 3, otherwise it won't do anything no matter how many times you push.
Is that the intent, just to add 1 to the counter with 2 different buttons and only if it's 1 or 2? You could do that with one menu item handler shared by both buttons that just checks to see if the current counter variable is 1 or 2 and increments it if so.
But, I have no clue what you're doing with that counter, so to say if it's correct or not... I dunno. Simply incrementing a counter variable isn't going to cause any changes on the screen if that's what you're hoping. Without more code, you'd only see those button presses doing something if you stopped in the debugger and looked at the value of counter.