This is an automated email from the ASF dual-hosted git repository.
martijnvisser pushed a commit to branch release-1.20
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.20 by this push:
new 01fbbfb8229 [FLINK-39017][web] Fix task click handler in job graph for
Chrome 144+
01fbbfb8229 is described below
commit 01fbbfb8229a9da20903654a54639fc8c918884c
Author: Martijn Visser <[email protected]>
AuthorDate: Tue Feb 10 15:34:03 2026 +0100
[FLINK-39017][web] Fix task click handler in job graph for Chrome 144+
Chrome 144 introduced BoundaryEventDispatchTracksNodeRemoval which
changes how pointer/mouse boundary events are dispatched after DOM
mutations. The onNodeMouseEnter handler used appendChild to reorder
SVG elements for z-ordering, which caused Chrome to reset the logical
pointer target to the parent graph element. This prevented click
events from reaching the node-group elements.
Remove appendChild DOM mutations from onNodeMouseEnter and
onNodeMouseLeave. Link focus styling on hover is preserved via
property changes without DOM reordering.
Co-Authored-By: Claude Opus 4.6 <[email protected]>
(cherry picked from commit f5e7558e64d519afed6f98165676a1d26f70628e)
---
.../src/app/components/dagre/dagre.component.ts | 15 ---------------
1 file changed, 15 deletions(-)
diff --git
a/flink-runtime-web/web-dashboard/src/app/components/dagre/dagre.component.ts
b/flink-runtime-web/web-dashboard/src/app/components/dagre/dagre.component.ts
index 471a883b414..4480997a193 100644
---
a/flink-runtime-web/web-dashboard/src/app/components/dagre/dagre.component.ts
+++
b/flink-runtime-web/web-dashboard/src/app/components/dagre/dagre.component.ts
@@ -349,7 +349,6 @@ export class DagreComponent extends NzGraph {
* @param $event
*/
onNodeMouseEnter($event: MouseEvent): void {
- this.graphElement.nativeElement.appendChild($event.target);
this.layoutLinks.forEach(l => {
if (l.id.split('-').indexOf(($event.target as HTMLElement).id) !== -1) {
l.options.focused = true;
@@ -366,20 +365,6 @@ export class DagreComponent extends NzGraph {
l.options.focused = this.focusedLinkIds.indexOf(l.id) !== -1;
});
-
this.graphElement.nativeElement.appendChild(this.overlayElement.nativeElement);
-
-
this.graphElement.nativeElement.querySelectorAll(`.link-group`).forEach((e:
Element) => {
- if (this.focusedLinkIds.indexOf(e.id) !== -1) {
- this.graphElement.nativeElement.appendChild(e);
- }
- });
-
-
this.graphElement.nativeElement.querySelectorAll(`.node-group`).forEach((e:
Element) => {
- if ([this.selectedNodeId, ...this.circleNodeIds].indexOf(e.id) !== -1) {
- this.graphElement.nativeElement.appendChild(e);
- }
- });
-
this.cd.detectChanges();
}
