Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2063)

Unified Diff: Source/devtools/front_end/elements/ElementsPanel.js

Issue 649633005: DevTools: fix nowrap style in the elements panel. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/elements/ElementsPanel.js
diff --git a/Source/devtools/front_end/elements/ElementsPanel.js b/Source/devtools/front_end/elements/ElementsPanel.js
index 9de1525ebea8bd79516b4d546772620e5db7b832..280a12e7539c52d0ad7120c105a930333b98c1e9 100644
--- a/Source/devtools/front_end/elements/ElementsPanel.js
+++ b/Source/devtools/front_end/elements/ElementsPanel.js
@@ -50,8 +50,9 @@ WebInspector.ElementsPanel = function()
this.contentElement = stackElement.createChild("div");
this.contentElement.id = "elements-content";
- if (!WebInspector.settings.domWordWrap.get())
- this.contentElement.classList.add("nowrap");
+ // FIXME: crbug.com/425984
+ if (WebInspector.settings.domWordWrap.get())
+ this.contentElement.classList.add("elements-wrap");
WebInspector.settings.domWordWrap.addChangeListener(this._domWordWrapSettingChanged.bind(this));
this._splitView.sidebarElement().addEventListener("contextmenu", this._sidebarContextMenuEventFired.bind(this), false);
@@ -129,6 +130,7 @@ WebInspector.ElementsPanel.prototype = {
targetAdded: function(target)
{
var treeOutline = new WebInspector.ElementsTreeOutline(target, true, true, this._setPseudoClassForNode.bind(this));
+ treeOutline.setWordWrap(WebInspector.settings.domWordWrap.get());
treeOutline.wireToDOMModel();
treeOutline.addEventListener(WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged, this._selectedNodeChanged, this);
treeOutline.addEventListener(WebInspector.ElementsTreeOutline.Events.NodePicked, this._onNodePicked, this);
@@ -443,10 +445,10 @@ WebInspector.ElementsPanel.prototype = {
_domWordWrapSettingChanged: function(event)
{
- if (event.data)
- this.contentElement.classList.remove("nowrap");
- else
- this.contentElement.classList.add("nowrap");
+ // FIXME: crbug.com/425984
+ this.contentElement.classList.toggle("elements-wrap", event.data);
+ for (var i = 0; i < this._treeOutlines.length; ++i)
+ this._treeOutlines[i].setWordWrap(/** @type {boolean} */ (event.data));
var selectedNode = this.selectedDOMNode();
if (!selectedNode)
« no previous file with comments | « Source/devtools/front_end/common/Settings.js ('k') | Source/devtools/front_end/elements/ElementsTreeOutline.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698