Skip to content
This repository was archived by the owner on Nov 27, 2023. It is now read-only.

Conversation

@mmelko
Copy link
Contributor

@mmelko mmelko commented Jan 20, 2023

In order to fix #1096 I had to revert #1010 and then tweak default Viewport configuration.

@mmelko mmelko requested a review from a team January 20, 2023 10:16
@mmelko mmelko changed the title fix(Visualisation): position after dropping the first node. fix(Visualization): position after dropping the first node. Jan 20, 2023
@sonarqubecloud
Copy link

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

@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #1139 (b0c53cf) into main (ef0db01) will increase coverage by 0.32%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1139      +/-   ##
==========================================
+ Coverage   48.43%   48.75%   +0.32%     
==========================================
  Files          51       51              
  Lines        1627     1610      -17     
  Branches      364      357       -7     
==========================================
- Hits          788      785       -3     
+ Misses        793      781      -12     
+ Partials       46       44       -2     
Impacted Files Coverage Δ
src/components/Visualization.tsx 40.74% <100.00%> (+3.14%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

const [isPanelExpanded, setIsPanelExpanded] = useState(false);
const [, setReactFlowInstance] = useState(null);
const reactFlowWrapperRef = useRef<HTMLDivElement>(null);
const reactFlowWrapper = useRef(null);
Copy link
Contributor

Choose a reason for hiding this comment

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

Why was this removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kahboom kahboom merged commit 23610d2 into kaoto-archive:main Jan 20, 2023
mmelko added a commit to mmelko/kaoto-ui that referenced this pull request Jan 23, 2023
…chive#1139)

* Revert "fix(Visualization): Initial node isn't always centered (kaoto-archive#1010)"

This reverts commit a408a13.

* fix(Visualisation): tweak centering of the initial node
fix kaoto-archive#1096
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Initial node drop changes position out of screen

2 participants