Tree View for Mumble page about steamlug.org HOT 15 CLOSED

I will merge this later unless someone has any objections?

from steamlug.org.

Did you end up making an expanding version?

from steamlug.org.

No.

from steamlug.org.

We were also talking about popping these in
http://mumble.sourceforge.net/Mumble_URL

from steamlug.org.

Don't think that the expanding version is worth the effort.

Also, pretty sure that the url-ything is hard to achieve as it only knows the current path to the channel, and not the full path.

from steamlug.org.

I feel that the expanded version is a bit long, especially since we've got so many channels now.

I'm not sure I follow you on path stuff. If that's just the page not keeping track of the path as it iterates through the channels, that'll pretty trivial to keep track of.

from steamlug.org.

Ah cool, I take it you make it then?

Long? We got more content on the other pages, like servers?

from steamlug.org.

I'm happy to give you a hand working it out if you need, but I'm not going to mess around with it on my own as there are other, higher priority features that need my attention, and I'm comfortable with the live page as it is now.

If you want to implement the Mumble channels as sortable tabular data like the servers page, then go nuts.

IIRC, the reason we started looking at this initially was to reduce the amount of space taken up with the current channel list and make the number/"location" of online users more readable. Not having an expanding tree seems to work against that.

from steamlug.org.

Merged the change.

from steamlug.org.

Can you please revert these changes and commit them to the Master (staging) branch (preferrably as a pull request that can be reviewed).

Was removing the user list as well intentional?

from steamlug.org.

No, it has been pending for over a year. Nobody bothered to accept it.

Yes, users are shown in the list now.

from steamlug.org.

Revert the commit and apply it to the Master (staging) branch, please. Do not let the live branch get ahead of Master.

"Nobody bothered to accept it" is also a poor response when there was unresolved discussion here along with an offer to assist with making changes.

from steamlug.org.

Why does it need reverting; can it not be additionally applied to the other branch if necessary?

from steamlug.org.

Or do that, but then that'll make for a messy merge next time something else gets pushed to live.

from steamlug.org.

Sorted with #108
Thanks

from steamlug.org.