fix: 修复工作流节点收缩后无法发布 #1582 #1626
Conversation
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| @@ -51,7 +51,7 @@ | |||
| </div> | |||
| </div> | |||
| <el-collapse-transition> | |||
| <div @mousedown.stop @keydown.stop @click.stop v-if="showNode" class="mt-16"> | |||
| <div @mousedown.stop @keydown.stop @click.stop v-show="showNode" class="mt-16"> | |||
| <el-alert | |||
| v-if="node_status != 200" | |||
| class="mb-16" | |||
There was a problem hiding this comment.
这个代码中有一些潜在问题和需要改进的地方：
-
没有正确地使用
v-model,v-on应该用到类组件（Vue3 版本）而不是普通方法。 -
在
.mousedown.stop,.keydown.stop, 和.click.stop事件上应用了@mousedown.stop @keydown.stop @click.stop, 这些是 Vue 的原生监听器，而不需要额外的指令。正确的写法应该是使用nativeListener:
<nativeListener />这样可以更易于在不同环境中使用，例如本地开发环境或生产运行时环境。
此外，代码中有重复的部分，请尽量避免。
fix: 修复工作流节点收缩后无法发布 #1582