Skip to content

Add QMike0/siyuan-plugin-callout-enhance#1862

Open
QMike0 wants to merge 1 commit into
siyuan-note:mainfrom
QMike0:main
Open

Add QMike0/siyuan-plugin-callout-enhance#1862
QMike0 wants to merge 1 commit into
siyuan-note:mainfrom
QMike0:main

Conversation

@QMike0

@QMike0 QMike0 commented Jun 3, 2026

Copy link
Copy Markdown

优化callout块样式,并提供标题快速编辑、折叠/展开、补全菜单、类型菜单、外观自定义与类型管理等功能

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

Pull Request Check Report

Add 1 Plugin Repo

QMike0/siyuan-plugin-callout-enhance


@TCOTC

TCOTC commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

PR 修改的末尾多了一个空行

@TCOTC TCOTC self-requested a review June 5, 2026 04:55
@TCOTC

TCOTC commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

初步审核

以下是 AI 的审核结论,请开发者确认并修复之后回复,然后维护者会进行人工审核。

版本: v0.2.4 (commit: 1b811c0)

发现的问题

  • src/index.ts 缺少 uninstall() 方法:插件通过 saveData("callout-enhance-settings", ...) 持久化配置,但未在卸载时调用 removeData() 清理 petal 数据。请新增 async uninstall() { await this.removeData(STORAGE_NAME); }(或等价的 removeData() 调用),且不要将 removeData() 放在 onunload() 中。
  • README_zh_CN.md 第 76 行菜单名称错误:当前为「设置-关于-清理历史数据」,该条目描述的是 DEBUG 日志功能,应改为与英文 README 一致的「设置-关于-调试日志」(或「Debug 日志」)。
  • README_zh_CN.md 第 58 行存在笔误「思源思源笔记」,应改为「思源笔记」。
  • src/index.ts onunload()(约第 700 行)未移除 onload() 中为发布服务添加的 document.body.classList 类名 callout-enhance-publish-service,请在 onunload() 中调用 document.body.classList.remove(PUBLISH_BODY_CLASS)
  • Bazaar PR plugins.txt 变更末尾多了一行空行(维护者 TCOTC 已指出),请删除 QMike0/siyuan-plugin-callout-enhance 条目后的多余空行。

需要维护者人工验证

  • 安装插件后验证 Callout 外观增强、嵌套样式是否正常
  • 验证标题快速编辑、折叠/展开、补全菜单、类型菜单等交互功能
  • 验证设置面板:外观自定义(预设保存/加载/管理、实时预览)、Callout 类型管理(增删改排序)、清理历史数据、调试日志开关
  • 在 desktop、mobile、browser-desktop 前端检查 UI 布局与样式
  • 验证插件禁用/启用、更新重载后功能与样式是否正常,控制台无异常报错
  • 若使用发布服务,验证 disabledInPublish: false 下发布页 Callout 样式展示及交互限制是否符合预期

检查通过

  • CI 已通过:Release v0.2.4 存在 package.zip,必需文件与 plugin.json 字段齐全
  • plugin.jsonname 与仓库名一致,url 正确,keywords 不含 siyuandisplayName 不含品牌名,无多余 funding/i18n 字段
  • icon.png(约 5.4 KB)、preview.png(约 74 KB)体积符合要求
  • README 链接均为绝对 URL,未嵌入 preview.png
  • package.zip 路径分隔符均为 /,无 node_modules/.git 等多余文件
  • i18n/en_US.jsoni18n/zh_CN.json 已打包,语言内容基本对应
  • 仓库含 MIT LICENSE,版权年份 2026、持有人 QMike0
  • Release tag 与 default branch HEAD 均为 1b811c0package.zip 与仓库源码同步
  • 配置管理:loadData()onload() 时加载并缓存,saveData() 不在 onunload() 中调用
  • 运行时清理:listen() 统一注册/移除事件监听,MutationObserveronunload() 中断开,防抖 timer 已清理,动态样式表已移除
  • window.location.reload()、无 .plugin-sample 模板残留样式
  • 发布服务场景有 isPublishService() 分支处理,编辑类交互在发布模式下被屏蔽

@QMike0 QMike0 reopened this Jun 5, 2026
@QMike0

QMike0 commented Jun 5, 2026

Copy link
Copy Markdown
Author

已重新PR;根据问题进行了修复,见新发布的v0.2.5

辛苦了哈~

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