Also the method `_makeExternalLinkMarkup` could inline the check for `text` avoiding the pitfall of assigning to parameter so
```
_makeExternalLinkMarkup: function(href, text) {
    if (!text) {
        text = href;
    }

    return String.format('<a href="{0}" target="_blank" class="external">{1}</a>', href, text);
},
```

becomes 
```
_makeExternalLinkMarkup: function(href, text) {
    return String.format('<a href="{0}" target="_blank" class="external">{1}</a>', href, (text || href));
}
```