northword / zotero-plugin-scaffold

A scaffold for Zotero plugin development
https://www.npmjs.com/package/zotero-plugin-scaffold
GNU Affero General Public License v3.0
8 stars 5 forks source link

Build: prefix fluent messages only those that existed in XHTML/HTML #14

Closed northword closed 4 months ago

northword commented 7 months ago

当前是把所有的 ftl 文件里的 message 都加了前缀,不过插件模板里的其实可以不加。

但这样似乎不好判断,因为似乎也可在代码中 insert fluent 片段?所以似乎还是全加了方便一些

(而且反正现在是和模板一致了的

EDIT 2024-6-23: 所有的 ftl message 都应添加前缀。

northword commented 4 months ago

我理了一下,可以看看对不对,以及需不需要把这些信息写在插件模板的自述里:

所有的 ftl message 都应有 namespace 前缀,以防止冲突,scaffold 会为所有 ftl 文件的每一条 message 都添加前缀。

对于插件模板,

如果需要在代码中直接写 message id,需要手动为 id 指定前缀。 如果在其他 html/xhtml 中使用,无需添加前缀但是需要手动引入。

github-actions[bot] commented 4 months ago

:rocket: This ticket has been resolved in v0.0.32. See v0.0.32 for release notes.

windingwind commented 4 months ago

如果需要在代码中直接写 message id,需要手动为 id 指定前缀。

这个指的是什么,我一时想不到有啥场景

windingwind commented 4 months ago

另外可能需要:

  1. 从zotero仓库chrome/locale/en-US/zotero/mozilla 拉取自带的ftl,并对其中的ftl字符串跳过加前缀
  2. 或允许用户自定义一些跳过ftl字符串加前缀的正则匹配,若命中则不加前缀且不警告
northword commented 4 months ago

如果需要在代码中直接写 message id,需要手动为 id 指定前缀。

这个指的是什么,我一时想不到有啥场景

这种:https://github.com/windingwind/zotero-plugin-template/blob/main/src%2Fmodules%2Fexamples.ts#L277

windingwind commented 4 months ago

这种感觉可以增加一个getLocaleID函数,我改下