Chinese searching highlighting does not show up in some scenario

[blueswen]

Contribution guidelines

• I've read the contribution guidelines and wholeheartedly agree

I've found a bug and checked that ...

• ... the problem doesn't occur with the mkdocs or readthedocs themes
• ... the problem persists when all overrides are removed, i.e. custom_dir, extra_javascript and extra_css
• ... the documentation does not mention anything about my problem
• ... there are no open or closed issues that are related to my problem

Description

There is a Chinese searching highlighting bug. If target query string is behind other Chinese text, searching highlighting mark does not show up. I think it's because match expression in highlighter/index.ts can not deal with this scenario.

I am not good at regex, but I found it could solved by adding another | in first group of expression (replace (^|${config.separator}|\\b) with (^|${config.separator}|\\b|)). Though it seems to work, I am not sure that could cause any side effect or not.

Expected behaviour

With query parameter h=公司 then modify match expression first group to (^|${config.separator}|\b|), and searching highlighting is working fine.

Actual behaviour

With query parameter h=公司, but searching highlighting is not working.

Steps to reproduce

1. Markdown content about TSMC from wiki:

   ## TSMC
   
   台灣積體電路製造股份有限公司（英語：Taiwan Semiconductor Manufacturing Co., Ltd.），簡稱台積電、台積、台積公司或TSMC[4]，與旗下公司合稱時則稱作台積電集團[5][6]，是臺灣一家從事晶圓代工的公司，為全球規模最大的半導體製造廠，總部位於新竹科學園區，主要廠房則分布於新竹市、臺中市、臺南市。
   

2. Search with 公司 (means company)

Package versions

• Python: Python 3.9.10
• MkDocs: version 1.3.0
• Material: 8.3.2+insiders.4.17.2

Configuration

site_name: My Docs

theme:
  name: material

System information

• Operating system: MacOS 12.3.1
• Browser: Chrome Version 102

[squidfunk]

Fixed in a60ac14. I've tested your proposed solution and it seems that we can just remove the word boundary character, which solves the problem at hand. I've checked several words with and without the word boundary and came to the conclusion that it should work reasonably well. As already noted on Gitter, I'll have to revisit highlighting in the future again, but since the change fixes the issue now without seemingly introducing new problems, we'll go with that for now.

[blueswen]

It looks great! Thanks for helping.

[squidfunk]

Released as part of 8.3.5+insiders-4.18.1.