Sneak Attack is an infinite conditional resource you want to use as much as possible unlike Battlemaster abilities which only have a few uses. SA would work well on a toggle while constantly having to switch Menacing Attack on and off would just be annoying. Sneak Attack also needs clear visual feedback on mouse over whether or not it will be applied to the attack so you can see it at a glance before attacking.
Menacing Attack should only have one button and execute on the currently equipped weapon. And equipping A and B weapon set buttons need to be separated from attack, which already happens when you click on an enemy. Ideally we would have a few more fast equip slots too.
