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

Unified Diff: Source/devtools/front_end/StylesSidebarPane.js

Issue 136613004: DevTools: [Styles] Remove the color format selector Cog menu (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/StylesSidebarPane.js
diff --git a/Source/devtools/front_end/StylesSidebarPane.js b/Source/devtools/front_end/StylesSidebarPane.js
index 9dec4a56e49197a18ef68c3c6de641319a2ef926..29631bbc100b88a649cbad5532ebdde1fb2990c8 100644
--- a/Source/devtools/front_end/StylesSidebarPane.js
+++ b/Source/devtools/front_end/StylesSidebarPane.js
@@ -37,38 +37,6 @@ WebInspector.StylesSidebarPane = function(computedStylePane, setPseudoClassCallb
{
WebInspector.SidebarPane.call(this, WebInspector.UIString("Styles"));
- this.settingsSelectElement = document.createElement("select");
- this.settingsSelectElement.className = "select-settings";
-
- var option = document.createElement("option");
- option.value = WebInspector.Color.Format.Original;
- option.label = WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "As authored" : "As Authored");
- this.settingsSelectElement.appendChild(option);
-
- option = document.createElement("option");
- option.value = WebInspector.Color.Format.HEX;
- option.label = WebInspector.UIString("Hex Colors");
- this.settingsSelectElement.appendChild(option);
-
- option = document.createElement("option");
- option.value = WebInspector.Color.Format.RGB;
- option.label = WebInspector.UIString("RGB Colors");
- this.settingsSelectElement.appendChild(option);
-
- option = document.createElement("option");
- option.value = WebInspector.Color.Format.HSL;
- option.label = WebInspector.UIString("HSL Colors");
- this.settingsSelectElement.appendChild(option);
-
- // Prevent section from collapsing.
- var muteEventListener = function(event) { event.consume(true); };
-
- this.settingsSelectElement.addEventListener("click", muteEventListener, true);
- this.settingsSelectElement.addEventListener("change", this._changeSetting.bind(this), false);
- this._updateColorFormatFilter();
-
- this.titleElement.appendChild(this.settingsSelectElement);
-
this._elementStateButton = document.createElement("button");
this._elementStateButton.className = "pane-title-button element-state";
this._elementStateButton.title = WebInspector.UIString("Toggle Element State");
@@ -747,7 +715,6 @@ WebInspector.StylesSidebarPane.prototype = {
_colorFormatSettingChanged: function(event)
{
- this._updateColorFormatFilter();
for (var pseudoId in this.sections) {
var sections = this.sections[pseudoId];
for (var i = 0; i < sections.length; ++i)
@@ -755,28 +722,6 @@ WebInspector.StylesSidebarPane.prototype = {
}
},
- _updateColorFormatFilter: function()
- {
- // Select the correct color format setting again, since it needs to be selected.
- var selectedIndex = 0;
- var value = WebInspector.settings.colorFormat.get();
- var options = this.settingsSelectElement.options;
- for (var i = 0; i < options.length; ++i) {
- if (options[i].value === value) {
- selectedIndex = i;
- break;
- }
- }
- this.settingsSelectElement.selectedIndex = selectedIndex;
- },
-
- _changeSetting: function(event)
- {
- var options = this.settingsSelectElement.options;
- var selectedOption = options[this.settingsSelectElement.selectedIndex];
- WebInspector.settings.colorFormat.set(selectedOption.value);
- },
-
_createNewRule: function(event)
{
event.consume();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698