Skip to content

fix(index.html): 修复脚本内 HTML 注释解析报错并补齐 CSS line-clamp 属性 - #80

Merged
albert4719 merged 1 commit into
eternity4719:mainfrom
parveen0029:fix/index-script-syntax
Oct 5, 2026
Merged

albert4719 merged 1 commit into
eternity4719:mainfrom
parveen0029:fix/index-script-syntax

Conversation

@parveen0029

Copy link
Copy Markdown
Contributor

修复问题 / Changes Summary

  1. 修复内联脚本中的语法解析异常 (Inline Script Syntax Error)

    • 在 index.html 中,第 550 行正则字面量直接包含了 /)。
    • 在 HTML 规范及 VS Code / HTML 词法解析器中,<script> 内部未转义的 <!-- 会被误识别为 HTML 注释起始,从而破坏后续 JavaScript 语法高亮并触发多处语法报错。
    • 修复方式:改用十六进制字符转义 \x3c 替代 <,即 /^\x3c!--\s成本标签:\s(.?)\s-->/。正则匹配逻辑 100% 保持一致,同时彻底消除解析报错。
  2. 补齐标准 CSS 属性 (CSS Compatibility)

    • 在 .toc-sub a span 中,原有规则仅声明了 -webkit-line-clamp: 2;。
    • 修复方式:补充标准属性 line-clamp: 2;,符合现代 Web CSS 规范并消除 linter 警告。

没有业务逻辑改动,测试运行正常。

@albert4719
albert4719 merged commit ed42576 into eternity4719:main Oct 5, 2026
@albert4719

Copy link
Copy Markdown
Contributor

感谢 PR,已合并。补充一句:这里原本并不影响浏览器。<script> 里的 "" 时才会出问题,这一行里 "-->" 紧跟着就收住了,页面一直正常解析。改成 \x3c 对编辑器和 linter 友好,正则匹配结果完全一样,所以照收。line-clamp 那处和 -webkit- 前缀并存也不影响。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants