The contextmenu now checks `visible` on items once instead of twice.

[bbrala]

The menu checked visible status twice, once to check if the menu should be opened and once more when the menu was opened. The logic all moved to the show method now so the check is only done once. This should fix issue #612