DevTools: Changes View

third_party/WebKit/Source/devtools/front_end/changes/changesView.css

+/*
+ * Copyright (c) 2017 The Chromium Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+.insertion-point-main{
+    position: relative;
+    display: flex;
+    flex-direction: column;
+}
+
+.insertion-point-sidebar {
+    overflow: auto;
+}
+
+:focus.selected {
+    background-color: rgb(56, 121, 217);
+    color: #FFF;
+}
+
+.diff-area {

[lushnikov, 2017/03/28 03:32:10]

can we put it inside insertion-point-main?

[einbinder, 2017/03/28 23:51:04]

Yep

+    flex-grow: 1;
+    z-index: 0;
+    display: flex;
+}
+
+.CodeMirror-lines:not(:active) {
+    cursor: default !important;
+}
+
+.CodeMirror-line:hover {
+    cursor: default !important;
+    background-color: rgba(0,0,255,0.05);

[lushnikov, 2017/03/28 03:32:10]

nit: spaces are missing after commas

[einbinder, 2017/03/28 23:51:04]

Done.

+}
+
+.CodeMirror .CodeMirror-linebackground.spacer {
+    text-align: center;
+    color: rgba(0,0,0,0.5);
+    background-color: rgba(0,0,255,0.1)
+}
+
+.CodeMirror .equal > span > span {
+    opacity: .5;
+}
+
+.CodeMirror .CodeMirror-selectedtext:not(.CodeMirror-persist-highlight) {
+    opacity: 1.0;
+}
+
+.CodeMirror .CodeMirror-linebackground.addition, -theme-preserve {
+    background-color: hsla(144,55%,49%,.2);

[lushnikov, 2017/03/28 03:32:10]

ditto

[einbinder, 2017/03/28 23:51:04]

Done.

+}
+
+.CodeMirror .CodeMirror-linebackground.deletion, -theme-preserve {
+    background-color: rgba(255, 0, 0, .2);
+}
+
+.CodeMirror .addition .cm-inner-diff:not(.CodeMirror-selectedtext), -theme-preserve {
+    background-color: hsla(144,55%,49%,.3);

[lushnikov, 2017/03/28 03:32:10]

ditto

[einbinder, 2017/03/28 23:51:04]

Done.

+}
+
+.CodeMirror .deletion .cm-inner-diff:not(.CodeMirror-selectedtext), -theme-preserve {
+    background-color: rgba(255, 0, 0, .3);
+}
+
+.changes-toolbar {

[lushnikov, 2017/03/28 03:32:10]

I don't think you need this whole rule

[einbinder, 2017/03/28 23:51:04]

Yep

+    display: flex;
+    flex: 0 0 27px;
+    background-color: #f3f3f3;
+    border-top: 1px solid #dadada;
+    overflow: hidden;
+    z-index: 1;

[lushnikov, 2017/03/28 03:32:10]

do we need this?

[einbinder, 2017/03/28 23:51:04]

no

+}