This is an issue that we discussed before. :)  We don't really need this method if it's only being used once. We could simply put the `wait_for_element_visible` into the `sign_in` method. The problem is that the `sign_in` method is defined in the `Base` class and the `_logout_menu_item_locator` is defined in the `HeaderRegion`. When we last discussed this I suggested two options:

1. Get rid of the `HeaderRegion` class and just put everything into `Base`.
2. Move the `sign_in` method into `HeaderRegion`.

I still stand by that and think we should do one or the other. It doesn't make sense to define methods that are only ever called once.