You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by so...@apache.org on 2022/03/24 06:43:08 UTC
[dolphinscheduler] branch dev updated: [BUG][UI Next]fix the problem about Batch delete not working (#9080)
This is an automated email from the ASF dual-hosted git repository.
songjian pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git
The following commit(s) were added to refs/heads/dev by this push:
new da2e85e [BUG][UI Next]fix the problem about Batch delete not working (#9080)
da2e85e is described below
commit da2e85e9ecb9fbf8916fd0f04a8f65ac3d80131c
Author: labbomb <73...@qq.com>
AuthorDate: Thu Mar 24 14:43:02 2022 +0800
[BUG][UI Next]fix the problem about Batch delete not working (#9080)
* fix the problem about Batch delete not working while click the border of the button
* try to fix e2e
Co-authored-by: Jiajie Zhong <zh...@hotmail.com>
---
.../pages/project/workflow/WorkflowDefinitionTab.java | 1 -
.../src/views/projects/workflow/definition/index.tsx | 16 ++++++++--------
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/dolphinscheduler-e2e/dolphinscheduler-e2e-case/src/test/java/org/apache/dolphinscheduler/e2e/pages/project/workflow/WorkflowDefinitionTab.java b/dolphinscheduler-e2e/dolphinscheduler-e2e-case/src/test/java/org/apache/dolphinscheduler/e2e/pages/project/workflow/WorkflowDefinitionTab.java
index 7040a53..9707dee 100644
--- a/dolphinscheduler-e2e/dolphinscheduler-e2e-case/src/test/java/org/apache/dolphinscheduler/e2e/pages/project/workflow/WorkflowDefinitionTab.java
+++ b/dolphinscheduler-e2e/dolphinscheduler-e2e-case/src/test/java/org/apache/dolphinscheduler/e2e/pages/project/workflow/WorkflowDefinitionTab.java
@@ -47,7 +47,6 @@ public final class WorkflowDefinitionTab extends NavBarPage implements ProjectDe
@FindBys({
@FindBy(className = "btn-delete-all"),
@FindBy(className = "n-button__content"),
- @FindBy(tagName = "span")
})
private WebElement buttonDeleteAll;
diff --git a/dolphinscheduler-ui-next/src/views/projects/workflow/definition/index.tsx b/dolphinscheduler-ui-next/src/views/projects/workflow/definition/index.tsx
index e1deedc..c3486da 100644
--- a/dolphinscheduler-ui-next/src/views/projects/workflow/definition/index.tsx
+++ b/dolphinscheduler-ui-next/src/views/projects/workflow/definition/index.tsx
@@ -176,19 +176,19 @@ export default defineComponent({
{{
default: () => t('project.workflow.delete'),
trigger: () => (
- <NButton
- tag='div'
- type='primary'
- disabled={this.checkedRowKeys.length <= 0}
- class='btn-delete-all'
- >
<NPopconfirm onPositiveClick={this.batchDeleteWorkflow}>
{{
default: () => t('project.workflow.delete_confirm'),
- trigger: () => t('project.workflow.delete')
+ trigger: () => (
+ <NButton
+ tag='div'
+ type='primary'
+ disabled={this.checkedRowKeys.length <= 0}
+ class='btn-delete-all'
+ >{t('project.workflow.delete')}</NButton>
+ )
}}
</NPopconfirm>
- </NButton>
)
}}
</NTooltip>