fix: Edges use zIndex from Connected Nodes#1761
Open
camielvs wants to merge 1 commit into02-05-feat_zindex_management_for_io_nodesfrom
Open
fix: Edges use zIndex from Connected Nodes#1761camielvs wants to merge 1 commit into02-05-feat_zindex_management_for_io_nodesfrom
camielvs wants to merge 1 commit into02-05-feat_zindex_management_for_io_nodesfrom
Conversation
3 tasks
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Feb 5, 2026
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
67775fc to
e673511
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
c61f27b to
2db9e39
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
e673511 to
90fab04
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
2db9e39 to
553d1eb
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
90fab04 to
f4f140b
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
3 times, most recently
from
f759585 to
bb361f6
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
13a4131 to
e87f47d
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
2 times, most recently
from
44a6e9a to
d0f2029
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
e87f47d to
b2ab23d
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
d0f2029 to
3960bb7
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
b2ab23d to
8960935
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
3960bb7 to
3cf77a7
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
2 times, most recently
from
6f3ae39 to
aaa8df1
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
3cf77a7 to
d9d84f4
Compare
camielvs
force-pushed
the
02-05-feat_zindex_management_for_io_nodes
branch
from
aaa8df1 to
dce220c
Compare
camielvs
force-pushed
the
02-05-fix_edges_use_zindex_from_connected_nodes
branch
from
d9d84f4 to
5d47149
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Edges will automatically use the maximum zIndex of the two connected nodes.
Related Issue and Pull requests
Closes https://github.com/Shopify/oasis-frontend/issues/483
Type of Change
Checklist
Screenshots (if applicable)
Before:
edge-bring-to-front-notfixed.mov (uploaded via Graphite)
After:
edge-bring-to-front-fixed.mov (uploaded via Graphite)
Test Instructions
Additional Comments