Title: [164483] trunk/Source/WebInspectorUI
Revision
164483
Author
grao...@webkit.org
Date
2014-02-21 07:53:42 -0800 (Fri, 21 Feb 2014)

Log Message

Web Inspector: scrollbar may appear when selecting a stop in gradient editor
https://bugs.webkit.org/show_bug.cgi?id=129149

Reviewed by Timothy Hatcher.

Ensure the angle input is not focused as we update the size of the gradient editor's
popover content since, if it were, it'd make a scrollbar appear as we animate the
popover's frame to fit its new content.

* UserInterface/CodeMirrorGradientEditingController.js:
(WebInspector.CodeMirrorGradientEditingController.prototype.gradientSliderStopWasSelected):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (164482 => 164483)


--- trunk/Source/WebInspectorUI/ChangeLog	2014-02-21 15:29:49 UTC (rev 164482)
+++ trunk/Source/WebInspectorUI/ChangeLog	2014-02-21 15:53:42 UTC (rev 164483)
@@ -1,3 +1,17 @@
+2014-02-21  Antoine Quint  <grao...@webkit.org>
+
+        Web Inspector: scrollbar may appear when selecting a stop in gradient editor
+        https://bugs.webkit.org/show_bug.cgi?id=129149
+
+        Reviewed by Timothy Hatcher.
+
+        Ensure the angle input is not focused as we update the size of the gradient editor's
+        popover content since, if it were, it'd make a scrollbar appear as we animate the
+        popover's frame to fit its new content.
+
+        * UserInterface/CodeMirrorGradientEditingController.js:
+        (WebInspector.CodeMirrorGradientEditingController.prototype.gradientSliderStopWasSelected):
+
 2014-02-20  Antoine Quint  <grao...@webkit.org>
 
         Web Inspector: rich editing of CSS gradients

Modified: trunk/Source/WebInspectorUI/UserInterface/CodeMirrorGradientEditingController.js (164482 => 164483)


--- trunk/Source/WebInspectorUI/UserInterface/CodeMirrorGradientEditingController.js	2014-02-21 15:29:49 UTC (rev 164482)
+++ trunk/Source/WebInspectorUI/UserInterface/CodeMirrorGradientEditingController.js	2014-02-21 15:53:42 UTC (rev 164483)
@@ -159,6 +159,10 @@
             this._colorPicker.element.remove();
             this._container.classList.remove(WebInspector.CodeMirrorGradientEditingController.EditsColorClassName);
         }
+
+        // Ensure the angle input is not focused since, if it were, it'd make a scrollbar appear as we
+        // animate the popover's frame to fit its new content.
+        this._angleInput.blur();
         
         this.popover.update();
     },
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to