Skip to content

Conversation

@zimzaza4
Copy link

@zimzaza4 zimzaza4 commented Dec 6, 2025

626e924821514fc6179a46d9ba6f50ea 在右键账户按钮的时候,可以打开一个菜单,能够快速预览并选择账户

@zkitefly
Copy link
Member

zkitefly commented Dec 6, 2025

我觉得可以在 tooltip 上添加相关提示以提醒有相关功能?

@zimzaza4
Copy link
Author

zimzaza4 commented Dec 6, 2025

image 这个 tooltip 吗 怎么加好一点?

@Glavo
Copy link
Member

Glavo commented Dec 6, 2025

这个功能确实有用,但我一直没想好怎么做 UI 比较合适,这样一个孤立的弹出菜单看着太奇怪了。

单纯为了快速选择账户的话,可以把鼠标放到这个按钮上然后用滚轮切换。

@burningtnt
Copy link
Member

这个功能确实有用,但我一直没想好怎么做 UI 比较合适,这样一个孤立的弹出菜单看着太奇怪了。

单纯为了快速选择账户的话,可以把鼠标放到这个按钮上然后用滚轮切换。

滚轮切换对绝大部分用户操作起来有些奇怪。我认为可以同时提供弹出菜单类型和滚轮切换,与右下角【启动游戏】按钮的表现保持一致。

@zimzaza4
Copy link
Author

zimzaza4 commented Dec 8, 2025

image 这个怎么样

@Glavo
Copy link
Member

Glavo commented Dec 11, 2025

我感觉弹在这个按钮的右侧是比较符合逻辑的,但这个右箭头感觉挺丑的。

建议先把这个箭头删了,未来再研究怎么给用户提供指引。

import static org.jackhuang.hmcl.setting.Accounts.getAccountFactory;
import static org.jackhuang.hmcl.setting.Accounts.getLocalizedLoginTypeName;

public class AccountMenuItem extends AdvancedListItem {
Copy link
Member

Choose a reason for hiding this comment

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

你可以考虑给 AccountAdvancedListItem 引入一个新的构造器,而不是复制一遍 AccountAdvancedListItem

Copy link
Author

Choose a reason for hiding this comment

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

这样行嘛

@Mine-diamond
Copy link
Contributor

Mine-diamond commented Dec 12, 2025

发现一个bug,左侧为最新开发版,右侧为该PR版本,账号名称右侧显示成了"...",但是是有足够空间的

image

@zimzaza4 zimzaza4 changed the title 新增 AccountMenuItem 控件, 在 RootPage 中添加快速选择账户 修改AccountAdvancedListItem 控件, 在 RootPage 中添加快速选择账户 Dec 12, 2025
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.

5 participants