Skip to content

Conversation

@suiyun39
Copy link
Collaborator

Checklist - zh-CN

  • 阅读 贡献指南
  • 把 issue 和 pr 相关联.
  • 确保代码与 master 分支保持同步.
  • 尽可能详细的描述所做的变更.

内容摘要

为辅助栏(右侧边栏)增加了背景设置支持,具体效果如下:

Snipaste_2025-06-13_18-52-13

@suiyun39 suiyun39 requested review from Copilot and shalldie June 13, 2025 11:40
@suiyun39 suiyun39 self-assigned this Jun 13, 2025
Copy link

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 adds support for configuring background images for the auxiliary bar. Key changes include:

  • Adding new configuration and integration for the auxiliary bar via AuxiliarybarPatchGenerator in PatchGenerator/index.ts.
  • Introducing a new module (PatchGenerator.auxiliarybar.ts) that extends the fullscreen patch generator for auxiliary bar styling.
  • Updating configuration schemas and documentation (nls files, package.json, and README files) to include auxiliary bar settings.

Reviewed Changes

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

Show a summary per file
File Description
src/background/PatchGenerator/index.ts Added import and integration of AuxiliarybarPatchGenerator.
src/background/PatchGenerator/PatchGenerator.auxiliarybar.ts Created the auxiliary bar patch generator module.
package.nls.zh.json, package.nls.json, package.nls.ja.json Added auxiliarybar description messages for localization.
package.json Defined auxiliarybar configuration schema.
README.zh-CN.md, README.md, README.ja-JP.md Updated documentation to include auxiliarybar feature details.
Comments suppressed due to low confidence (1)

src/background/PatchGenerator/index.ts:30

  • Consider adding explicit unit tests for the auxiliarybar background generation to ensure consistent behavior with the other sections.
new AuxiliarybarPatchGenerator(options.auxiliarybar).create(), // auxiliarybar

@suiyun39 suiyun39 linked an issue Jun 13, 2025 that may be closed by this pull request
@shalldie shalldie merged commit 58c7658 into shalldie:master Jul 29, 2025
1 check passed
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.

[Feature] Secondary Sidebar 右侧侧边栏支持

2 participants