hypeJunction/Elgg — comment on commit 2970f9ec1703d98ef9494fbb4664d6d4cec6068f
File:  line 0
By cash:

* Using the values passed as $vars['tabs'] as input into output/url is better than what we were doing. That's good.
* Removing the encoding since output/url handles that is good
* I'm seeing some spacing issues - assume this is tabs vs spaces. That needs to be cleaned up.
* We're not requiring that an id be passed in for the ul so this needs some logic to fix that. As it's currently written, it will create a blank id and throw a notice because the 'id' key is not defined in $vars