Title: [196187] trunk/Source/WebInspectorUI
Revision
196187
Author
commit-qu...@webkit.org
Date
2016-02-05 13:38:44 -0800 (Fri, 05 Feb 2016)

Log Message

Web Inspector: Navigation bar in sidebars should always fit on a single line
https://bugs.webkit.org/show_bug.cgi?id=153412
<rdar://problem/24318706>

Patch by Devin Rousso <dcrousso+web...@gmail.com> on 2016-02-05
Reviewed by Timothy Hatcher.

This happened because the allowed maximum width of the sidebar was greater
than the minimum width derived from the currently visible sidebar.

* UserInterface/Base/Utilities.js:
(Number.constrain):
Added logic to reverse the values of min and max if max is less than min.

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (196186 => 196187)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-02-05 21:34:27 UTC (rev 196186)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-02-05 21:38:44 UTC (rev 196187)
@@ -1,3 +1,18 @@
+2016-02-05  Devin Rousso  <dcrousso+web...@gmail.com>
+
+        Web Inspector: Navigation bar in sidebars should always fit on a single line
+        https://bugs.webkit.org/show_bug.cgi?id=153412
+        <rdar://problem/24318706>
+
+        Reviewed by Timothy Hatcher.
+
+        This happened because the allowed maximum width of the sidebar was greater
+        than the minimum width derived from the currently visible sidebar.
+
+        * UserInterface/Base/Utilities.js:
+        (Number.constrain):
+        Added logic to reverse the values of min and max if max is less than min.
+
 2016-02-05  Timothy Hatcher  <timo...@apple.com>
 
         Web Inspector: Don't wrap labels in Breakpoint Editor popover

Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js (196186 => 196187)


--- trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js	2016-02-05 21:34:27 UTC (rev 196186)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js	2016-02-05 21:38:44 UTC (rev 196187)
@@ -852,6 +852,11 @@
 {
     value: function(num, min, max)
     {
+        if (max < min) {
+            let temp = max;
+            max = min;
+            min = temp;
+        }
         if (num < min)
             num = min;
         else if (num > max)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to