Is there a reason for building HTML inside the directive here? It shouldn't have any performance impact but it seems more reasonable to interpolate with {{}} inside the template