Skip to content

fix(dashboard): fix dark mode icons and platform template order - #10061

Open
momo-OwO-qwq wants to merge 2 commits into
AstrBotDevs:masterfrom
momo-OwO-qwq:master
Open

fix(dashboard): fix dark mode icons and platform template order#10061
momo-OwO-qwq wants to merge 2 commits into
AstrBotDevs:masterfrom
momo-OwO-qwq:master

Conversation

@momo-OwO-qwq

@momo-OwO-qwq momo-OwO-qwq commented Sep 12, 2026

Copy link
Copy Markdown

Modifications / 改动点

  1. 修复暗色主题下部分图标显示为黑色、无法辨识的问题
  2. 调整「创建机器人」中消息平台类别的排列顺序,将同一产品线的适配器归到一起
  • This is NOT a breaking change. / 这不是一个破坏性变更。

Screenshots or Test Results / 运行截图或测试结果

image image

Checklist / 检查清单

  • 😊 If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
    / 如果 PR 中有新加入的功能,已经通过 Issue / 邮件等方式和作者讨论过。

  • 👀 My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
    / 我的更改经过了良好的测试,并已在上方提供了“验证步骤”和“运行截图”

  • 🤓 I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
    / 我确保没有引入新依赖库,或者引入了新依赖库的同时将其添加到 requirements.txtpyproject.toml 文件相应位置。

  • 😮 My changes do not introduce malicious code.
    / 我的更改没有引入恶意代码。

Summary by Sourcery

Fix dashboard platform icon appearance in dark mode and restore the intended ordering of default platform templates.

Bug Fixes:

  • Fix dark-mode rendering for Mattermost and Matrix platform icons.
  • Correct the default platform template order so WeChat Official Account and Lark appear in their intended positions.
  • Classify additional provider icons as monochrome for consistent display.

Some monochrome icons still rendered black on dark surfaces:

* microsoft (Edge TTS), fishaudio (FishAudio TTS), vllm (vLLM Rerank),
  huggingface (TEI Rerank), coze and deerflow were missing from the
  monochrome icon list.
* The Mattermost and Matrix platform logos are pure black SVGs. They are now
  imported with ?no-inline so the dark theme CSS can match the emitted asset
  URL instead of an inlined data URI.
* The new platform logo selectors are anchored to a path segment, otherwise
  img[src*="matrix"] would also match inline SVG data URIs that contain
  gradientTransform="matrix(...)".
The WeChat adapters were split apart by Lark and DingTalk in the create bot
platform selector. Reorder the CONFIG_METADATA_2 platform templates so adapters
of the same product line stay together, keeping the remaining order unchanged.

* place weixin_official_account right after weixin_oc

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Approved.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

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.

1 participant