Skip to content

feat(MyMsgInput):支持执行代码#5782

Open
LinQingYuu wants to merge 26 commits into
Meloong-Git:mainfrom
LuoYun-Team:my-msg-input-action
Open

feat(MyMsgInput):支持执行代码#5782
LinQingYuu wants to merge 26 commits into
Meloong-Git:mainfrom
LuoYun-Team:my-msg-input-action

Conversation

@LinQingYuu

@LinQingYuu LinQingYuu commented Mar 4, 2025

Copy link
Copy Markdown
Collaborator

PR 改动

  • 为 MyMsgInput 增加 Button3
  • 现在支持为 Button 附加 Action,使其支持执行特定代码(相当于支持点击时复制更新命令了)
  • Button2 按下后不会自动关闭(为了和自动复制更新命令配合使用)
  • 取消按钮改为 Button3 避免取消被放在中间产生奇怪的观感

拿 VSCode 写的,如果有啥 Bug 或者对部分不太满意请踢我一脚

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 · 优化 🟨 中 优先度:中 labels Mar 4, 2025
@LTCatt

LTCatt commented Mar 4, 2025

Copy link
Copy Markdown
Member

umm 取消按钮改为 Button3 的话……
还没看代码,不过现有的有两个按钮的 MyMsgInput 应该是可以按 Esc 取消的,那 Esc 取消逻辑那儿需要改么?

@LinQingYuu

Copy link
Copy Markdown
Collaborator Author

umm 取消按钮改为 Button3 的话…… 还没看代码,不过现有的有两个按钮的 MyMsgInput 应该是可以按 Esc 取消的,那 Esc 取消逻辑那儿需要改么?

🤔 原来 PCL 还支持快捷键的?
最新的提交应该做掉了,龙猫找个时间看看是不是预期的实现?

@Open-KFC

Copy link
Copy Markdown
Collaborator

@shimoranla 你这个 PR 好像炸了

@LinQingYuu

Copy link
Copy Markdown
Collaborator Author

@shimoranla 你这个 PR 好像炸了

什么东西炸了?截个图我看看

@Open-KFC

Open-KFC commented May 25, 2025

Copy link
Copy Markdown
Collaborator

要不还是 rebase -i upstream/main 删一下错的提交吧(

@LinQingYuu

Copy link
Copy Markdown
Collaborator Author

要不还是 rebase -i upsgtream/main 删一下错的提交吧(

这玩意好像会炸分支导致 PR 被关掉,但是我懒得开新的(

@Open-KFC

Copy link
Copy Markdown
Collaborator

我用了好多次没问题的呀,我合并上游更改用的都是 rebase(

@LTCatt LTCatt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

看了看逻辑,如果要加按钮 3 的话,得处理把按钮 2 作为取消键或者把 3 作为取消键的两种情况,MyMsgInput.xaml.vb 里不太能这样改(

@LTCatt LTCatt added 等待提交者 需要提交者补充信息或进行操作 and removed 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 labels May 30, 2025
@LinQingYuu

Copy link
Copy Markdown
Collaborator Author

现在弄了版新的实现,如果 Button2 的文本是取消,那么无论 Button3 是个啥都不会展示,这样似乎能处这种情况

龙猫再看看?

@LinQingYuu LinQingYuu requested a review from LTCatt May 30, 2025 16:12
@MoYuan-CN MoYuan-CN added 等待确认 已经过社区确认,等待开发者确认 and removed 等待提交者 需要提交者补充信息或进行操作 labels May 31, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 等待确认 已经过社区确认,等待开发者确认 labels Jun 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

· 优化 处理中 开发者正在调查或处理该项 🟨 中 优先度:中

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants