fix topic bar to allow prefixes

[Cherrg]

• do not select if escape is pressed
• allow prefixes by adding current request content to result list
• remove/escape html-tags before insert on page

Update

• sort typed word to top
• remove already present topics from dropdown list

fix #7126 (Unable to create a new topic because it's a substring of an existing one)

Signed-off-by: Michael Gnehr michael@gnehr.de

[mrsdizzie]

This looks better thank you! Would it be possible instead of (or in addition to) adding the current request to the list to also not show something that is already a tag (or been entered but not yet saved in the current session).

So as an example here:

It shouldn't show golang as a suggestion since it has already been selected and will just disappear if you try to pick it a second time.

[codecov-io]

Codecov Report

Merging #7325 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7325      +/-   ##
==========================================
- Coverage   41.24%   41.23%   -0.01%     
==========================================
  Files         466      466              
  Lines       63153    63153              
==========================================
- Hits        26045    26043       -2     
- Misses      33702    33705       +3     
+ Partials     3406     3405       -1

Impacted Files	Coverage Δ
models/unit.go	62.16% <0%> (-5.41%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff85dd3...10a0703. Read the comment docs.

[mrsdizzie]

Still looks like it shows already selected tags as suggestions (unless I'm doing something wrong):

[Cherrg]

i didn't notice that there where other class names for previously present topics and new added ones.
Should be better now

[mrsdizzie]

That looks great thank you for all of these PRs they really add up!

[silverwind]

Please don't land yet until above comment is resolved (seems I lack permission to block this PR via negative review).

[zeripath]

As per @silverwind

[zeripath]

Don't worry I'm fairly certain our merging stuff wouldn't let us merge in any case.

[silverwind]

Should be good to go.