Skip to content

Node Dependency Highlighting#1081

Open
1AhmedYasser wants to merge 3 commits into
buerokratt:devfrom
1AhmedYasser:Node-Dependency-Highlighting
Open

Node Dependency Highlighting#1081
1AhmedYasser wants to merge 3 commits into
buerokratt:devfrom
1AhmedYasser:Node-Dependency-Highlighting

Conversation

@1AhmedYasser

Copy link
Copy Markdown
Collaborator

@1AhmedYasser 1AhmedYasser requested a review from trevorling June 18, 2026 15:32
return arrayMove(elements, oldIndex, newIndex);
}

type HighlightData = { isHighlighted?: boolean; isDimmed?: boolean };

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

readonly isHighlighted?: boolean; readonly isDimmed?: boolean

style,
markerEnd,
data,
}: EdgeProps) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

EdgeProps & {
  readonly highlightData?: HighlightData;
}

markerEnd,
data,
}: EdgeProps) {
const highlightData = data as HighlightData | undefined;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Component props type change -> no need for the casting :)

@sonarqubecloud

Copy link
Copy Markdown

@1AhmedYasser 1AhmedYasser requested a review from trevorling June 19, 2026 11:29
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