Remove $scope.$emit('hide-menu'); fixes #2662

[juliusakula]

Removing this line of code keeps the menu open until the user is done with it. This allows for multiple actions while the menu stays open.

[PaulL1]

@juliusakula : I think we need to decide whether we'd want to leave the menu open for all actions, or just for column hiding related actions. I think the existing behaviour is more what people would expect in most cases. I'll consult the guys to see what they thing.

[farooqu]

I think leaving the menu open for all actions makes sense. You can imagine a grid being loaded with many hidden columns and a user may want to un-hide multiple columns. Not having to re-open the menu each time would make lots of sense.

[juliusakula]

I still think this fix would be helpful

[PaulL1]

I'm planning to implement somethign similar when I get around to it, but this one I think I can't merge because:

1. It applies to all menus including the column menus. I don't think that makes sense, it needs to apply only to the gridMenu
2. I'd like it to apply only to the column show/hide within the grid menu, not to other things like exporting a file
3. I'd like it to be switchable so that people can choose to keep the current behaviour if they want.

[PaulL1]

I've implemented a similar concept, but with an option on it so that only those menus and menu items that we want to keep open would stay open. Closing this.