Title: [207165] trunk/Source/WebInspectorUI
- Revision
- 207165
- Author
- [email protected]
- Date
- 2016-10-11 14:24:35 -0700 (Tue, 11 Oct 2016)
Log Message
Web Inspector: Remove line highlight on primary execution line while stepping because it is distracting
https://bugs.webkit.org/show_bug.cgi?id=163294
<rdar://problem/28721176>
Patch by Joseph Pecoraro <[email protected]> on 2016-10-11
Reviewed by Timothy Hatcher.
* UserInterface/Views/TextEditor.js:
(WebInspector.TextEditor.prototype._updateExecutionLine):
When setting the primary execution line, remove default line highlights.
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (207164 => 207165)
--- trunk/Source/WebInspectorUI/ChangeLog 2016-10-11 20:51:53 UTC (rev 207164)
+++ trunk/Source/WebInspectorUI/ChangeLog 2016-10-11 21:24:35 UTC (rev 207165)
@@ -1,3 +1,15 @@
+2016-10-11 Joseph Pecoraro <[email protected]>
+
+ Web Inspector: Remove line highlight on primary execution line while stepping because it is distracting
+ https://bugs.webkit.org/show_bug.cgi?id=163294
+ <rdar://problem/28721176>
+
+ Reviewed by Timothy Hatcher.
+
+ * UserInterface/Views/TextEditor.js:
+ (WebInspector.TextEditor.prototype._updateExecutionLine):
+ When setting the primary execution line, remove default line highlights.
+
2016-10-10 Matt Baker <[email protected]>
Web Inspector: Revealed line not highlighted in TextEditor while debugger paused
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js (207164 => 207165)
--- trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js 2016-10-11 20:51:53 UTC (rev 207164)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js 2016-10-11 21:24:35 UTC (rev 207165)
@@ -1220,6 +1220,7 @@
if (this._executionLineHandle) {
this._codeMirror.addLineClass(this._executionLineHandle, "wrap", WebInspector.TextEditor.ExecutionLineStyleClassName);
this._codeMirror.addLineClass(this._executionLineHandle, "wrap", "primary");
+ this._codeMirror.removeLineClass(this._executionLineHandle, "wrap", WebInspector.TextEditor.HighlightedStyleClassName);
}
});
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
