Skip to content
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

CommandHelp #38

Closed
wants to merge 7 commits into from
Closed

CommandHelp #38

wants to merge 7 commits into from

Conversation

Apleax
Copy link
Contributor

@Apleax Apleax commented Nov 8, 2024

把指向指令列表URL的改成/dominion help

@Apleax Apleax closed this Nov 8, 2024
@Apleax Apleax reopened this Nov 8, 2024
@Apleax Apleax closed this Nov 8, 2024
@Apleax Apleax reopened this Nov 8, 2024
@ColdeZhang
Copy link
Owner

能提供一张修改过后的对应页面tui截图吗

@ColdeZhang
Copy link
Owner

图裂了 看不到

.append(Translation.TUI_CommandHelp_DominionManageDescription);

Line memberManage = Line.create()
.append(Button.create(Translation.TUI_CommandHelp_MemberManageButton).setExecuteCommand("").build())
Copy link
Owner

Choose a reason for hiding this comment

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

帮助菜单的按钮没有关联实际功能? 我看ExecuteCommand都是空的。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

还没有,一直没写,现在刚准备开始

@@ -59,25 +72,47 @@ public static void show(CommandSender sender, String[] args) {
Line reload_config = Line.create()
.append(Button.create(Translation.TUI_Menu_ReloadConfigButton).setExecuteCommand("/dominion reload_config").build())
.append(Translation.TUI_Menu_ReloadConfigDescription);

//以下用于将上方的组件添加到菜单中

Copy link
Owner

Choose a reason for hiding this comment

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

尽可能避免在提交合并的代码中包含类似过多的冗余注释 :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK

@Apleax Apleax closed this Nov 28, 2024
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.

2 participants