Skip to content

fix: modify document internationalization conversion issue#464

Open
James-9696 wants to merge 1 commit intodevfrom
fix-docs-probblem
Open

fix: modify document internationalization conversion issue#464
James-9696 wants to merge 1 commit intodevfrom
fix-docs-probblem

Conversation

@James-9696
Copy link
Copy Markdown

@James-9696 James-9696 commented Apr 8, 2026

PR

修复以下问题:
123
已修复:
image

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our Commit Message Guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Summary by CodeRabbit

  • Documentation
    • Updated the mention feature documentation sidebar label and page title, removing the @ symbol for a cleaner presentation.

@github-actions github-actions bot added the bug Something isn't working label Apr 8, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

Walkthrough

The changes remove the @ symbol from the mention feature's display label and heading across documentation. The sidebar navigation item and documentation title are both updated from @提醒 to 提醒, with all links and content remaining unchanged.

Changes

Cohort / File(s) Summary
Documentation & Navigation
packages/docs/fluent-editor/.vitepress/sidebar.ts, packages/docs/fluent-editor/docs/demo/mention.md
Removed leading @ symbol from mention feature label in sidebar navigation and documentation title heading.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A @ symbol bid goodbye,
From labels high and headings nigh,
Now "提醒" stands clean and bright,
The mention feature gleams in light! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to fix a 'document internationalization conversion issue', but the actual changes only remove '@' symbols from Chinese text labels in sidebar navigation and a markdown heading. Use a more accurate title like 'fix: remove @ prefix from mention demo navigation and title' or 'fix: update mention demo display text' that directly describes the actual changes made.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-docs-probblem

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant