It seems strange to have this HTML both here and in the template. Since it appears that setValueInEditor is called whenever render is called, I think that the template shouldn't include this HTML (just the 'ol' to which this method attaches items).

We should probably do that in general for the other editors too, unless we want to change the code to re-render the template each time the data changed (which wouldn't be bad). But that's beyond the scope of this pull request.