Notification drawer should close when the user clicks outside of the drawer

[benjaminapetersen]

This is a fairly common convenience feature for a drawer/panel/modal. Here are a few examples:

• a side nav panel
• another side panel
• another side panel

I believe a click event on the document is needed, then a test to e.target (exists && matches current element) to implement, but can look into this more.

[beanh66]

@benjaminapetersen In this case, staying open is actually by design to allow users to go through notifications and potentially investigate them in the main content area at the same time. We added the X close button in the top right corner as another means to closing the panel (aside from clicking the bell) in order to make it more clear. We should be consistent with PatternFly and keep this functionality as is for now but in the meantime we will check with ManageIQ to see if they've had any complaints since it's been implemented in that product for a few releases.

[serenamarie125]

@beanh66 ManageIQ hasn't had any complaints either. This is something that we can test, or ask for user feedback if it's an issue. @benjaminapetersen I agree that it is a convenience feature, but unsure if it makes sense in this specific use case. Interested in your thoughts.

[benjaminapetersen]

User testing/feedback definitely makes sense! Click outside seems to be a common convention, though I suppose alternatives like esc key could also be considered (but I suspect less discoverable/more intuitive to power users only). I think requiring a manual movement of the mouse to the "x" (or bell) as the only way to close the panel gets a little bit annoying when you are opening/closing the panel frequently.

Definitely like the "x", thats a big win for usability so far!