Comment 15 for bug 1424815

Revision history for this message
Mike Rylander (mrylander) wrote :

Arg... I was afraid of this.

So, with the new highlighting code, we get properly formatted HTML fragments from the backend and where we can use them we inject them directly. The good news is that the code, as it stands at the moment after resolving conflicts with that branch, would force a choice between the features rather than breaking strings mid-highlight (I believe). The bad news is that some significant work would need to happen if it were to be able to handle the Display Field highlighted data.

Also, we need to be careful about using "id" attributes on potentially duplicatable elements. I don't /think/ that's happening here, but it would be easy to slip in.

In any case, it does look useful. Thanks, all, for working on it.