Title: [214906] trunk/Source/WebInspectorUI
Revision
214906
Author
[email protected]
Date
2017-04-04 15:23:46 -0700 (Tue, 04 Apr 2017)

Log Message

Web Inspector: RTL: layout issues in Type Profiler popovers
https://bugs.webkit.org/show_bug.cgi?id=170467

Reviewed by Timothy Hatcher.

Flip some margins and padding. Remove the -1px leading margin for
the prototype disclosure button, as it is unnecessary and looks
(more) wrong when in RTL.

* UserInterface/Views/TypeTreeElement.css:
(.item.type-tree-element > .disclosure-button):
(.item.type-tree-element.prototype):
(body[dir=ltr] .item.type-tree-element.prototype):
(body[dir=rtl] .item.type-tree-element.prototype):
* UserInterface/Views/TypeTreeView.css:
(.tree-outline.type):
(body[dir=ltr] .tree-outline.type):
(body[dir=rtl] .tree-outline.type):
(.tree-outline.type li .empty-message):
(body[dir=ltr] .tree-outline.type li .empty-message):
(body[dir=rtl] .tree-outline.type li .empty-message):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (214905 => 214906)


--- trunk/Source/WebInspectorUI/ChangeLog	2017-04-04 22:23:37 UTC (rev 214905)
+++ trunk/Source/WebInspectorUI/ChangeLog	2017-04-04 22:23:46 UTC (rev 214906)
@@ -1,3 +1,27 @@
+2017-04-04  Brian Burg  <[email protected]>
+
+        Web Inspector: RTL: layout issues in Type Profiler popovers
+        https://bugs.webkit.org/show_bug.cgi?id=170467
+
+        Reviewed by Timothy Hatcher.
+
+        Flip some margins and padding. Remove the -1px leading margin for
+        the prototype disclosure button, as it is unnecessary and looks
+        (more) wrong when in RTL.
+
+        * UserInterface/Views/TypeTreeElement.css:
+        (.item.type-tree-element > .disclosure-button):
+        (.item.type-tree-element.prototype):
+        (body[dir=ltr] .item.type-tree-element.prototype):
+        (body[dir=rtl] .item.type-tree-element.prototype):
+        * UserInterface/Views/TypeTreeView.css:
+        (.tree-outline.type):
+        (body[dir=ltr] .tree-outline.type):
+        (body[dir=rtl] .tree-outline.type):
+        (.tree-outline.type li .empty-message):
+        (body[dir=ltr] .tree-outline.type li .empty-message):
+        (body[dir=rtl] .tree-outline.type li .empty-message):
+
 2017-04-04  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: Include more Network information in Resource Details Sidebar

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css (214905 => 214906)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css	2017-04-04 22:23:37 UTC (rev 214905)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeElement.css	2017-04-04 22:23:46 UTC (rev 214906)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015 Apple Inc. All rights reserved.
+ * Copyright (C) 2015-2017 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -46,7 +46,6 @@
 
     width: 15px;
 
-    margin-left: -1px;
     border: 0;
     background: none;
     -webkit-appearance: none;
@@ -75,10 +74,24 @@
     background-color: hsla(0, 0%, 0%, 0.03);
     border-radius: 3px;
 
-    padding: 0 10px 1px 0;
-    margin: 3px 0 2px -1px;
+    padding-bottom: 1px;
+    margin-top: 3px;
+    margin-bottom: 2px;
+
+    --type-tree-element-prototype-padding-end: 10px;
+    --type-tree-element-prototype-margin-start: -1px;
 }
 
+body[dir=ltr] .item.type-tree-element.prototype {
+    padding-right: var(--type-tree-element-prototype-padding-end);
+    margin-left: var(--type-tree-element-prototype-margin-start);
+}
+
+body[dir=rtl] .item.type-tree-element.prototype {
+    padding-left: var(--type-tree-element-prototype-padding-end);
+    margin-right: var(--type-tree-element-prototype-margin-start);
+}
+
 .item.type-tree-element.prototype:hover,
 .item.type-tree-element.prototype:focus {
     border-color: hsla(0, 0%, 0%, 0.1);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css (214905 => 214906)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css	2017-04-04 22:23:37 UTC (rev 214905)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TypeTreeView.css	2017-04-04 22:23:46 UTC (rev 214906)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015 Apple Inc. All rights reserved.
+ * Copyright (C) 2015-2017 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -34,12 +34,23 @@
 
 .tree-outline.type {
     margin: 0;
-    padding: 0 6px 2px;
+    padding-bottom: 2px;
+    min-height: 18px;
+
     list-style: none;
-    min-height: 18px;
     outline: none;
+
+    --type-tree-outline-padding-end: 6px;
 }
 
+body[dir=ltr] .tree-outline.type {
+    padding-right: var(--type-tree-outline-padding-end);
+}
+
+body[dir=rtl] .tree-outline.type {
+    padding-left: var(--type-tree-outline-padding-end);
+}
+
 .tree-outline.type li {
     padding: 0;
     white-space: nowrap;
@@ -62,5 +73,14 @@
 
 .tree-outline.type li .empty-message {
     color: hsl(0, 0%, 60%);
-    margin-left: 13px;
+
+    --type-tree-outline-empty-message-margin-start: 13px;
 }
+
+body[dir=ltr] .tree-outline.type li .empty-message {
+    margin-left: var(--type-tree-outline-empty-message-margin-start);
+}
+
+body[dir=rtl] .tree-outline.type li .empty-message {
+    margin-right: var(--type-tree-outline-empty-message-margin-start);
+}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to