Title: [294773] trunk/Source/WebInspectorUI/UserInterface/Views/ScreenshotsTimelineOverviewGraph.js
Revision
294773
Author
[email protected]
Date
2022-05-24 15:42:44 -0700 (Tue, 24 May 2022)

Log Message

Web Inspector: Timelines Tab: Screenshots: clicking on another screenshot doesn't select it if there's already a selected screenshot
https://bugs.webkit.org/show_bug.cgi?id=240884

Reviewed by Patrick Angle.

* Source/WebInspectorUI/UserInterface/Views/ScreenshotsTimelineOverviewGraph.js:
(WI.ScreenshotsTimelineOverviewGraph.prototype.layout):

Canonical link: https://commits.webkit.org/250938@main

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScreenshotsTimelineOverviewGraph.js (294772 => 294773)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ScreenshotsTimelineOverviewGraph.js	2022-05-24 22:34:24 UTC (rev 294772)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScreenshotsTimelineOverviewGraph.js	2022-05-24 22:42:44 UTC (rev 294773)
@@ -72,8 +72,10 @@
 
             imageElement.style.left = (record.startTime - this.startTime) / secondsPerPixel + "px";
             imageElement.addEventListener("click", (event) => {
+                // Ensure that the container "click" listener added by `WI.TimelineOverview` isn't called.
+                event.__timelineRecordClickEventHandled = true;
+
                 this.selectedRecord = record;
-                event.stopPropagation();
             });
         }
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to