Skip to content

[Feature-6918][UI] DAG page interaction optimization#6919

Merged
davidzollo merged 8 commits intoapache:devfrom
cmss-bfs:issue-6918
Nov 19, 2021
Merged

[Feature-6918][UI] DAG page interaction optimization#6919
davidzollo merged 8 commits intoapache:devfrom
cmss-bfs:issue-6918

Conversation

@wangyizhi1
Copy link
Copy Markdown
Member

See #6918

  1. Add grid layout function.
    Kapture 2021-11-19 at 10 54 48

  2. Open pre-tasks settings for all tasks
    Kapture 2021-11-19 at 11 01 17

  3. Add node search function
    Kapture 2021-11-19 at 11 05 59

  4. Add DAG scale bar
    Kapture 2021-11-19 at 11 09 40

  5. Add process status tag
    image

  6. Optimize node feedback when creating connection
    Kapture 2021-11-19 at 11 15 46

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 19, 2021

Codecov Report

Merging #6919 (f07331e) into dev (595e484) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #6919      +/-   ##
============================================
- Coverage     33.07%   33.03%   -0.05%     
+ Complexity     1626     1625       -1     
============================================
  Files           433      433              
  Lines         14365    14365              
  Branches       1437     1437              
============================================
- Hits           4751     4745       -6     
- Misses         9158     9168      +10     
+ Partials        456      452       -4     
Impacted Files Coverage Δ
...he/dolphinscheduler/common/enums/SqoopJobType.java 0.00% <0.00%> (-88.89%) ⬇️
...inscheduler/common/task/sqoop/SqoopParameters.java 74.00% <0.00%> (ø)
...dolphinscheduler/remote/future/ResponseFuture.java 83.05% <0.00%> (+1.69%) ⬆️
...er/master/dispatch/host/assign/RandomSelector.java 83.33% <0.00%> (+5.55%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 595e484...f07331e. Read the comment docs.

@William-GuoWei
Copy link
Copy Markdown
Contributor

excellent design +1

display: flex;
justify-content: center;

/deep/ .el-slider__runway{
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.

It is recommended to replace /deep/ with ::v-deep.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done.

@@ -80,7 +80,6 @@ body{
//background: #f8fbfe;
}
.conditions-model {
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.

This will conflict with the pr of https://github.com/apache/dolphinscheduler/pull/6911/files, which requires local rebase.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done.

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@wangyizhi1 wangyizhi1 changed the title [Feature][UI] DAG page interaction optimization [Feature-6918][UI] DAG page interaction optimization Nov 19, 2021
@songjianet
Copy link
Copy Markdown
Member

LGTM

Copy link
Copy Markdown
Contributor

@davidzollo davidzollo left a comment

Choose a reason for hiding this comment

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

+1
great job
so amazing!

@davidzollo davidzollo linked an issue Nov 19, 2021 that may be closed by this pull request
3 tasks
@davidzollo davidzollo merged commit f2d242c into apache:dev Nov 19, 2021
wangyizhi1 added a commit to cmss-bfs/dolphinscheduler that referenced this pull request Dec 2, 2021
* [Feature] Add grid layout and optimize DAG style

* [Improvement] Add process definition status label and code optimization

* [Feature] add DAG node search bar

* [Feature] Add DAG scale bar

* [Improvement] Open pre-tasks settings for all tasks

* [Fix] Fix ut

* [Fix] Replace /deep/ with ::v-deep

* [Fix] Fix code smell
brave-lee pushed a commit that referenced this pull request Dec 3, 2021
…#7137)

* [Feature-6918][UI] DAG page interaction optimization (#6919)

* [Feature] Add grid layout and optimize DAG style

* [Improvement] Add process definition status label and code optimization

* [Feature] add DAG node search bar

* [Feature] Add DAG scale bar

* [Improvement] Open pre-tasks settings for all tasks

* [Fix] Fix ut

* [Fix] Replace /deep/ with ::v-deep

* [Fix] Fix code smell

* [Improvement-7092][UI] Optimize DAG to adapt to situations without locations (#7102)
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.

[Feature][UI] Some optimization suggestions for DAG page

5 participants