Skip to content

refactor: align model list buttons, use lucide icons #8803

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

alephpiece
Copy link
Collaborator

@alephpiece alephpiece commented Aug 3, 2025

What this PR does

  • 模型列表按钮对齐。原本有滚动条会挤占一部分空间,导致按钮上下对不齐。
  • 一些 icon 迁移到 lucide 保持风格一致。
    • “编辑模型”改用 lucide pen 图标,更符合“编辑”这个含义。
    • lucide penline 改用 pen,更加简洁。
    • 模型列表、Api key 列表的 MinusOutlined 和 PlusOutlined 改用 lucide minus 和 plus。
    • 管理模型列表的“添加列表中的模型”和“删除列表中的模型”改用 lucide icons。
  • ProviderSetting 中几个设置按钮对齐
BEFORE AFTER
模型列表 before after
管理模型,添加所有模型 image image
管理模型,移除所有模型 image image
Api key 列表 image image
ProviderSetting 界面按钮对齐 image image

Fixes #

Why we need it and why it was done in this way

The following tradeoffs were made:

The following alternatives were considered:

Links to places where the discussion took place:

Breaking changes

If this PR introduces breaking changes, please describe the changes and the impact on users.

Special notes for your reviewer

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note


@alephpiece alephpiece merged commit 84604a1 into CherryHQ:main Aug 3, 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.

3 participants