Skip to content

fix(docs): fix broken anchor links in zh-CN and ja README navigation#1507

Open
qyangcv wants to merge 1 commit intoopen-compass:mainfrom
qyangcv:main
Open

fix(docs): fix broken anchor links in zh-CN and ja README navigation#1507
qyangcv wants to merge 1 commit intoopen-compass:mainfrom
qyangcv:main

Conversation

@qyangcv
Copy link
Copy Markdown

@qyangcv qyangcv commented Apr 3, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 3, 2026 09:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the top navigation links in the Chinese (zh-CN) and Japanese (ja) READMEs to fix broken in-page anchor links.

Changes:

  • Updated zh-CN README navigation href fragments for section links.
  • Updated ja README navigation href fragments for section links.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/zh-CN/README_zh-CN.md Adjusts top-of-page navigation anchors for key sections.
docs/ja/README_ja.md Adjusts top-of-page navigation anchors for key sections.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +12 to +16
<a href="#️-快速开始">🏗️ 快速开始 </a> •
<a href="#-评测结果支持的数据集和模型">📊 数据集和模型 </a> •
<a href="#️-开发指南">🛠️ 开发指南 </a> •
<a href="#-vlmevalkit-的目标">🎯 我们的目标 </a> •
<a href="#️-引用">🖊️ 引用 </a>
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These navigation links don’t match the explicit anchor IDs used later in this README. The sections define stable IDs like quickstart, data-model-results, development, goal-of-vlmevalkit, and citation via <a id="…"></a>, so these href values should point to #quickstart, #data-model-results, #development, #goal-of-vlmevalkit, and #citation (instead of the current auto-slugged/emoji-based fragments).

Suggested change
<a href="#️-快速开始">🏗️ 快速开始 </a> •
<a href="#-评测结果支持的数据集和模型">📊 数据集和模型 </a> •
<a href="#️-开发指南">🛠️ 开发指南 </a> •
<a href="#-vlmevalkit-的目标">🎯 我们的目标 </a> •
<a href="#️-引用">🖊️ 引用 </a>
<a href="#quickstart">🏗️ 快速开始 </a> •
<a href="#data-model-results">📊 数据集和模型 </a> •
<a href="#development">🛠️ 开发指南 </a> •
<a href="#goal-of-vlmevalkit">🎯 我们的目标 </a> •
<a href="#citation">🖊️ 引用 </a>

Copilot uses AI. Check for mistakes.
Comment thread docs/ja/README_ja.md
Comment on lines +13 to +16
<a href="#️-クイックスタート">🏗️Quickstart </a> •
<a href="#️-開発ガイド">🛠️Development </a> •
<a href="#-vlmevalkitの目標">🎯Goal </a> •
<a href="#️-引用">🖊️Citation </a>
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The href fragments here include an invisible variation-selector character () (e.g., #️-クイックスタート). GitHub’s generated heading IDs for ## 🏗️ クイックスタート / ## 🛠️ 開発ガイド / ## 🖊️ 引用 won’t include that character, so these links likely remain broken. Update them to use the actual generated slugs (e.g., #-クイックスタート, #-開発ガイド, #-引用) or add explicit <a id="…"></a> anchors and link to those.

Suggested change
<a href="#-クイックスタート">🏗️Quickstart </a> •
<a href="#-開発ガイド">🛠️Development </a> •
<a href="#-vlmevalkitの目標">🎯Goal </a> •
<a href="#-引用">🖊️Citation </a>
<a href="#-クイックスタート">🏗️Quickstart </a> •
<a href="#-開発ガイド">🛠️Development </a> •
<a href="#-vlmevalkitの目標">🎯Goal </a> •
<a href="#-引用">🖊️Citation </a>

Copilot uses AI. Check for mistakes.
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