Page MenuHomePhabricator

Instead of returning html from the highlighter, use unicode reserved characters as the pre/post highlight signals.
Closed, ResolvedPublic

Description

This would allow us to push sanitization out to the edge where it is supposed to be. It was suggested that this is how bing is internally handling highlighting.

Event Timeline

EBernhardson raised the priority of this task from to Needs Triage.
EBernhardson updated the task description. (Show Details)
EBernhardson added a project: CirrusSearch.
EBernhardson subscribed.

16:37 < JustinO> ebernhardson: bing uses '\uE000' & '\uE001' to highlight

Nice idea. This also has advantages for clients that cannot render HTML easily, such as mobile apps.

jayvdb renamed this task from Instead of returning html from the higlighter, use unicode reserved characters as the pre/post highlight signals. to Instead of returning html from the highlighter, use unicode reserved characters as the pre/post highlight signals..Feb 11 2016, 12:47 AM
jayvdb set Security to None.

16:37 < JustinO> ebernhardson: bing uses '\uE000' & '\uE001' to highlight

Specifically, see "Hit Highlighting / Bing": https://msdn.microsoft.com/en-us/library/gg447081.aspx

Aklapper removed a project: Discovery-ARCHIVED.
EBernhardson claimed this task.

This was done some years ago, it's marked as T173172. This seems to be effectively the same ticket.