fix: fix the issue of failed title switching after using the quill-header-list plugin

[wuyiping0628]

解决用了quill-header-list插件后切换标题失败的问题

PR

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

• Bug Fixes
  • Improved header formatting controls in the editor demos.
  • Header levels are now applied correctly, with formatting reliably removed when the unformatted option is selected.

[coderabbitai]

Walkthrough

The Fluent Editor header toolbar handlers now convert header levels to numbers and pass false when users remove header formatting.

Changes

Header toolbar formatting

Layer / File(s)	Summary
Apply header toolbar values packages/docs/fluent-editor/demos/header-list-container.vue, packages/docs/fluent-editor/demos/header-list.vue	Both demos add custom header handlers. The handlers pass numeric header levels to Quill and use false to clear header formatting.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

I’m a rabbit with a toolbar bright,
Headers become numbers, neat and right.
false clears the style away,
Quill formats text without delay.
Hop, hop, clean lists today!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main fix for failed title switching after using the quill-header-list plugin.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch wyp/list-0805

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

packages/docs/fluent-editor/demos/header-list-container.vue (1)

29-31: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Add a regression test for header selection and clearing.

The existing specs verify initial headings and header-list visibility, but they do not exercise this handler. Add tests in packages/docs/fluent-editor/demos/header-list.spec.ts and packages/docs/fluent-editor/demos/header-list-container.spec.ts that select a heading level and then select the reset option.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/docs/fluent-editor/demos/header-list-container.vue` around lines 29
- 31, Add regression coverage for the header(value) handler in both
header-list.spec.ts and header-list-container.spec.ts: select a heading level,
assert the editor applies that header, then select the reset option and assert
the header formatting is cleared. Reuse the existing demo test setup and
selectors without changing the handler behavior.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@packages/docs/fluent-editor/demos/header-list-container.vue`:
- Around line 29-31: Add regression coverage for the header(value) handler in
both header-list.spec.ts and header-list-container.spec.ts: select a heading
level, assert the editor applies that header, then select the reset option and
assert the header formatting is cleared. Reuse the existing demo test setup and
selectors without changing the handler behavior.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c03ac424-4fcc-4fd8-b388-41f8cea9be94

📥 Commits

Reviewing files that changed from the base of the PR and between 662006b and c96df9f.

📒 Files selected for processing (2)

• packages/docs/fluent-editor/demos/header-list-container.vue
• packages/docs/fluent-editor/demos/header-list.vue