Make triangle icon usable

[rodrigolzd]

The grouping triangle icon should be clickable and just as functional as the text, it seems more natural to use the icon as a control to expand and collapse the lists.

[deleolajide]

Agreed. A PR to fix this is most welcome

[luciash]

This can be closed, right?

[deleolajide]

Yes, thanks

[luciash]

Ah, sorry, the triangle icon is still not toggleable on click... oops :) I thought the issue was actually the opposite (text not clickable to toggle).

Edit: Doh, the caret is sometimes clickable and sometimes it is not (very intermittent) - weird! It is like of you click in the middle of the caret exactly it does not work... but slightly left or right of it the toggle gets triggered.

[rodrigolzd]

so, this is going to be open again?

[luciash]

@deleolajide When I click the triangle I get in the Chrome DevTools console:

Uncaught TypeError: Cannot read property 'classList' of null
    at child.toggleRoomsList (converse.js:58793)
    at wrapper (converse.js:13440)
    at HTMLDivElement.handler (converse.js:1005)
  | toggleRoomsList | @ | converse.js:58793
  | wrapper | @ | converse.js:13440
  | handler | @ | converse.js:1005