This is to match the way other functions are read: elgg_view_entity_icon for example. We can change it to be second, though, for sure. It would make more sense as second argument in lots of cases.