DevTools: Light blue highlight when hovering in Sources Navigator (issue 2431373006 by einbinder@chromium.org)

0 views
Skip to first unread message

einb...@chromium.org

unread,
Oct 21, 2016, 7:56:37 PM10/21/16
to lush...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org
Reviewers: lushnikov
CL: https://codereview.chromium.org/2431373006/

Message:
ptal

Description:
DevTools: Light blue highlight when hovering in Sources Navigator

BUG=none

Affected files (+5, -0 lines):
M third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css


Index: third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css b/third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css
index bec0a65d2ffadb181c8bd0996e8822adaf3a0f68..5a353013db411ced8e085e081fc3a2dc920edefc 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css
+++ b/third_party/WebKit/Source/devtools/front_end/sources/navigatorTree.css
@@ -69,6 +69,11 @@
min-height: 20px;
}

+.tree-outline li:hover:not(.selected) .selection{
+ display:block;
+ background-color: rgba(56, 121, 217, 0.1);
+}
+
.navigator-folder-tree-item .icon {
-webkit-mask-position: -64px -120px;
background-color: #555;


einb...@chromium.org

unread,
Oct 21, 2016, 8:02:16 PM10/21/16
to lush...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org
On 2016/10/21 at 23:56:37, einbinder wrote:
> ptal

http://imgur.com/UjLZoKc

https://codereview.chromium.org/2431373006/

lush...@chromium.org

unread,
Oct 21, 2016, 8:55:39 PM10/21/16
to einb...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org

einb...@chromium.org

unread,
Oct 21, 2016, 10:32:16 PM10/21/16
to lush...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org
On 2016/10/22 at 00:55:38, lushnikov wrote:
> nit: missing space

Done.

https://codereview.chromium.org/2431373006/

commit-bot@chromium.org via chromiumcodereview-hr.appspot.com

unread,
Oct 21, 2016, 10:33:05 PM10/21/16
to einb...@chromium.org, lush...@chromium.org, commi...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org

commit-bot@chromium.org via chromiumcodereview-hr.appspot.com

unread,
Oct 22, 2016, 1:37:58 AM10/22/16
to einb...@chromium.org, lush...@chromium.org, commi...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org
Committed patchset #3 (id:60001)

https://codereview.chromium.org/2431373006/

commit-bot@chromium.org via chromiumcodereview-hr.appspot.com

unread,
Oct 22, 2016, 2:12:26 AM10/22/16
to einb...@chromium.org, lush...@chromium.org, commi...@chromium.org, apavlo...@chromium.org, blink-...@chromium.org, caseq...@chromium.org, chromium...@chromium.org, devtools...@chromium.org, kozyatins...@chromium.org, lushnik...@chromium.org, pfeldma...@chromium.org, pfel...@chromium.org
Patchset 3 (id:??) landed as
https://crrev.com/c067174d429128e4d37e52bbd2abc9dd84de20d0
Cr-Commit-Position: refs/heads/master@{#426981}

https://codereview.chromium.org/2431373006/
Reply all
Reply to author
Forward
0 new messages