MartijnVisser opened a new pull request, #27569:
URL: https://github.com/apache/flink/pull/27569

   ## What is the purpose of the change
   
   Chrome 144 introduced `BoundaryEventDispatchTracksNodeRemoval`, which 
changes how boundary events (pointerover, mouseenter, etc.) are dispatched 
after DOM mutations. The dagre component used `appendChild` in 
`onNodeMouseEnter`/`onNodeMouseLeave` to reorder SVG elements for z-ordering on 
hover. After Chrome 144, these DOM mutations cause Chrome to reset the logical 
pointer target to the parent `<g class="graph">` element, preventing click 
events from ever reaching the node elements.
   
   This PR removes `appendChild` calls from mouse enter/leave handlers. These 
calls were a cosmetic optimization that brought hovered nodes to the front in 
SVG paint order. In dagre layouts, nodes do not overlap, so removing this 
reordering has no visible effect. Link focus highlighting on hover is preserved 
because it uses property changes, not DOM reordering.
   
   ## Brief change log
   
   * Removes `appendChild` calls from mouse enter/leave handlers
   
   ## Verifying this change
   
   1. Build  this PR via `./mvnw clean package -DskipTests -Djdk17 
-Pjava17-target`
   2. Start a cluster `./build-target/bin/start-cluster.sh`
   3. Submit a job `./build-target/bin/flink run 
./build-target/examples/streaming/WordCount.jar`
   4. Open the UI at http://localhost:8081 and verify on a Chrome 144+ browser 
that clicking now opens the details again
   
   ## Does this pull request potentially affect one of the following parts:
   
     - Dependencies (does it add or upgrade a dependency): (yes / **no**)
     - The public API, i.e., is any changed class annotated with 
`@Public(Evolving)`: (yes / **no**)
     - The serializers: (yes / **no** / don't know)
     - The runtime per-record code paths (performance sensitive): (yes / **no** 
/ don't know)
     - Anything that affects deployment or recovery: JobManager (and its 
components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / **no** / don't 
know)
     - The S3 file system connector: (yes / **no** / don't know)
   
   ## Documentation
   
     - Does this pull request introduce a new feature? (yes / **no**)
     - If yes, how is the feature documented? (**not applicable** / docs / 
JavaDocs / not documented)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to