alireza0:s-ui 的 web/html 是构建产物,前端改动应提交到 frontend 仓库

在关闭一个主题 PR 时说明,该 PR 的全部文件都在 web/html 下,而这一目录在 .gitignore 中,是 frontend 子模块执行 npm run build 的输出,由 Dockerfile 与发布流程复制到位,仓库不跟踪其中任何内容。因此这里没有可审查的源码改动——主题只以压缩后的 bundle 形式存在(单个文件一行就有 25 万字符),无法看出它实际做了什么;即使合并,下一次发布构建也会重新生成 web/html 并全部覆盖。他表示若想改面板外观,应改 frontend 仓库里的 Vue 组件与 Vuetify 主题,并认为做一轮 Material Design 3 是合理想法、愿意看一下,随后关闭了该 PR。

作者原文

Hi @YiranrumengQAQ, thanks for the interest — but this can't be merged as it is.

Every file in this PR is under web/html, which is in .gitignore. It isn't
source: it's the output of npm run build in the frontend/ submodule, copied
into place by the Dockerfile and the release workflow. Nothing under it is
tracked in the repository.

That means two things. There's no source change here to review — the theme
exists only as a minified bundle (one line of df02d90cfd573e6a.js is 254,000
characters), so there's no way to see what it actually does. And even if it were
merged, the next release build would regenerate web/html from source and
overwrite all of it, so it would have no effect.

If you'd like to work on the panel's look, the place for it is the frontend
repository, in the Vue components and the Vuetify theme — a change there is
reviewable and survives a rebuild. A Material Design 3 pass is a reasonable idea
and I'd take a look at it.

Closing this one. Thanks for understanding.