Changeset 186877 in webkit

Timestamp:

Jul 15, 2015, 5:33:33 PM (10 years ago)

Author:

Matt Baker

Message:

Web Inspector: Resizing TimelineRuler selection on both sides doesn't work with snapping enabled
​https://bugs.webkit.org/show_bug.cgi?id=146970

Reviewed by Brian Burg.

• UserInterface/Views/TimelineRuler.js:

(WebInspector.TimelineRuler.prototype._handleSelectionHandleMouseMove):
Use snapped current time value when setting selection bounds.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/TimelineRuler.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2015-07-15  Matt Baker  <mattbaker@apple.com>
+
+        Web Inspector: Resizing TimelineRuler selection on both sides doesn't work with snapping enabled
+        https://bugs.webkit.org/show_bug.cgi?id=146970
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Views/TimelineRuler.js:
+        (WebInspector.TimelineRuler.prototype._handleSelectionHandleMouseMove):
+        Use snapped current time value when setting selection bounds.
+
 2015-07-14  Nikita Vasilyev  <nvasilyev@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js

@@ -783 +783 @@
         var currentMousePosition = event.pageX - this._element.totalOffsetLeft;
         var currentTime = this.startTime + (currentMousePosition * this.secondsPerPixel);
+        if (this.snapInterval)
+            currentTime = this._snapValue(currentTime);
 
         if (event.altKey && !event.ctrlKey && !event.metaKey && !event.shiftKey) {