This is an automated email from the ASF dual-hosted git repository.
martijnvisser pushed a commit to branch release-2.0
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-2.0 by this push:
new c61ec3a9ae4 [FLINK-39017][web] Fix task click handler in job graph for
Chrome 144+
c61ec3a9ae4 is described below
commit c61ec3a9ae49840bcffb289fad4373f16fbb6a8a
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 c6f40a1273e..86425874ebe 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
@@ -360,7 +360,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;
@@ -377,20 +376,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();
}