this method does more than *click*, so I think it should be named differently.. in my opinion, a more appropriate name would be `_toggleSettingsView` and create a new method `closeSettingsView` - that way our code will describe the intent of the call. (`_toggleSettingsView` would only be called by `[open|close]SettingsView`)

one of the main reasons why we are abstracting the common functions is not only to reduce duplication, but to make the code easier to understand.