Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOP-24183] Highlight column lineage on click #30

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

dolfinus
Copy link
Member

@dolfinus dolfinus commented Mar 4, 2025

Clicking on column lineage edge selects edge + both connected columns:
Снимок экрана_20250304_163033

Clicking on column selects connected edges and related column:
Снимок экрана_20250304_163044

Double clicking on edge or column selects all related edges and nodes, recursively. Almost like double clicking on regular edge or dataset/job node, but only related columns are highlighted.

Non-selected edge is gray and thin, but selected edge is light-blue and a bit thicker, to highlight it visually. Label has the same light-blue color.

Current solution is custom state for selection which is updated by node/edge clicking events, and then ReactFlow nodes/edges .selected value is updated to match this state. This is because ReactFlow doesn't know anything about columns, so we cannot get selection from its internal state.

@dolfinus dolfinus self-assigned this Mar 4, 2025
@dolfinus dolfinus force-pushed the feature/DOP-24183 branch from 774307f to f1d56ca Compare March 4, 2025 13:57
@dolfinus dolfinus requested a review from Zabilsya March 4, 2025 13:58
@dolfinus dolfinus marked this pull request as ready for review March 4, 2025 13:58
@dolfinus dolfinus force-pushed the feature/DOP-24183 branch 6 times, most recently from c94f4ec to cdc193d Compare March 4, 2025 17:18
@dolfinus dolfinus force-pushed the feature/DOP-24183 branch from cdc193d to 5e6f79b Compare March 5, 2025 10:27
@dolfinus dolfinus merged commit 263eb1e into develop Mar 5, 2025
3 checks passed
@dolfinus dolfinus deleted the feature/DOP-24183 branch March 5, 2025 10:54
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.

2 participants