DevTools: Source color picker experiment. Make color swatches clickable

third_party/WebKit/Source/devtools/front_end/sources/CSSSourceFrame.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 21 matching lines @@
  * @constructor
  * @extends {WebInspector.UISourceCodeFrame}
  * @param {!WebInspector.UISourceCode} uiSourceCode
  */
 WebInspector.CSSSourceFrame = function(uiSourceCode)
 {
     WebInspector.UISourceCodeFrame.call(this, uiSourceCode);
     this.textEditor.setAutocompleteDelegate(new WebInspector.CSSSourceFrame.AutocompleteDelegate());
     this._registerShortcuts();
     this._colorBookmarks = [];
+    this._swatchPopoverHelper = new WebInspector.SwatchPopoverHelper();
+    this._muteColorProcessing = false;
 }
 
 /** @type {number} */
 WebInspector.CSSSourceFrame.UpdateTimeout = 200;
 
 WebInspector.CSSSourceFrame.prototype = {
     _registerShortcuts: function()
     {
         var shortcutKeys = WebInspector.ShortcutsScreen.SourcesPanelShortcuts;
         for (var i = 0; i < shortcutKeys.IncreaseCSSUnitByOne.length; ++i)
@@ skipping 93 matching lines @@
      * @param {?Array<!WebInspector.CSSSourceFrame.ColorPosition>} colorPositions
      */
     _putColorSwatchesInline: function(colorPositions)
     {
         this._clearColorBookmarks();
         if (!colorPositions)
             return;
 
         for (var i = 0; i < colorPositions.length; i++) {
             var colorPosition = colorPositions[i];
-
-            var swatch = WebInspector.ColorSwatch.create();
-            swatch.setColorText(colorPosition.color.asString(WebInspector.Color.Format.Original));
-            swatch.hideText(true);
-
-            var bookmark = this.textEditor.addBookmark(colorPosition.textRange.startLine, colorPosition.textRange.startColumn, swatch);
+            var swatch = new WebInspector.CSSSourceFrame.ColorSwatch(this, colorPosition);
+            var bookmark = this.textEditor.addBookmark(colorPosition.textRange.startLine, colorPosition.textRange.startColumn, swatch.element());
             this._colorBookmarks.push(bookmark);
         }
     },
 
     _clearColorBookmarks: function()
     {
         for (var i = 0; i < this._colorBookmarks.length; i++)
             this._colorBookmarks[i].clear();
         this._colorBookmarks = [];
     },
 
     /**
+     * @param {!WebInspector.CSSSourceFrame.ColorSwatch} swatch
+     */
+    _showSpectrum: function(swatch)
+    {
+        if (this._swatchPopoverHelper.isShowing()) {
+            this._swatchPopoverHelper.hide();
+            return;
+        }
+        this._currentSwatch = swatch;
+        this._spectrum = new WebInspector.Spectrum();
+        this._spectrum.setColor(swatch.element().color(), swatch.element().format());
+        this._spectrum.addEventListener(WebInspector.Spectrum.Events.SizeChanged, this._spectrumResized, this);
+        this._spectrum.addEventListener(WebInspector.Spectrum.Events.ColorChanged, this._spectrumChanged, this);
+        this._swatchPopoverHelper.show(this._spectrum, swatch.element().iconElement(), this._spectrumHidden.bind(this));
+    },
+
+    /**
+     * @param {!WebInspector.Event} event
+     */
+    _spectrumResized: function(event)
+    {
+        this._swatchPopoverHelper.reposition();
+    },
+
+    /**
+     * @param {!WebInspector.Event} event
+     */
+    _spectrumChanged: function(event)
+    {
+        this._muteColorProcessing = true;
+        var colorString = /** @type {string} */ (event.data);
+        this._currentSwatch.setColorText(colorString);
+        this._replaceColorText(this._currentSwatch.colorPosition(), colorString);
+    },
+
+    _spectrumHidden: function()
+    {
+        this._muteColorProcessing = false;
+        this._spectrum.removeEventListener(WebInspector.Spectrum.Events.SizeChanged, this._spectrumResized, this);
+        this._spectrum.removeEventListener(WebInspector.Spectrum.Events.ColorChanged, this._spectrumChanged, this);
+        delete this._spectrum;
+    },
+
+    /**
+     * @param {!WebInspector.CSSSourceFrame.ColorPosition} colorPosition
+     * @param {string} colorString
+     */
+    _replaceColorText: function(colorPosition, colorString)

[lushnikov, 2016/07/20 00:49:23]

you can inline this now

[flandy, 2016/07/20 01:56:18]

Done.

+    {
+        this._textEditor.editRange(colorPosition.textRange, colorString);
+        colorPosition.color = WebInspector.Color.parse(colorString);
+        colorPosition.textRange.endColumn = colorPosition.textRange.startColumn + colorString.length;
+    },
+
+    /**
      * @override
      */
     onTextEditorContentSet: function()
     {
         WebInspector.UISourceCodeFrame.prototype.onTextEditorContentSet.call(this);
-        if (Runtime.experiments.isEnabled("sourceColorPicker"))
+        if (!this._muteColorProcessing && Runtime.experiments.isEnabled("sourceColorPicker"))
             this._updateColorSwatches();
     },
 
     /**
      * @override
      * @param {!WebInspector.TextRange} oldRange
      * @param {!WebInspector.TextRange} newRange
      */
     onTextChanged: function(oldRange, newRange)
     {
         WebInspector.UISourceCodeFrame.prototype.onTextChanged.call(this, oldRange, newRange);
-        if (Runtime.experiments.isEnabled("sourceColorPicker")) {
+        if (!this._muteColorProcessing && Runtime.experiments.isEnabled("sourceColorPicker")) {
             if (this._updateTimeout)
                 clearTimeout(this._updateTimeout);
             this._updateTimeout = setTimeout(this._updateColorSwatches.bind(this), WebInspector.CSSSourceFrame.UpdateTimeout);
         }
     },
 
+    /**
+     * @override
+     * @param {number} lineNumber
+     */
+    scrollChanged: function(lineNumber)
+    {
+        WebInspector.UISourceCodeFrame.prototype.scrollChanged.call(this, lineNumber);
+        if (this._swatchPopoverHelper.isShowing())
+            this._swatchPopoverHelper.hide();
+    },
+
     __proto__: WebInspector.UISourceCodeFrame.prototype
 }
 
 /**
  * @constructor
  * @param {!WebInspector.Color} color
  * @param {number} lineNumber
  * @param {number} startColumn
  * @param {number} textLength
  */
 WebInspector.CSSSourceFrame.ColorPosition = function(color, lineNumber, startColumn, textLength)
 {
     this.color = color;
     this.textRange = new WebInspector.TextRange(lineNumber, startColumn, lineNumber, startColumn + textLength);
 }
 
 /**
  * @constructor
+ * @param {!WebInspector.CSSSourceFrame} cssSourceFrame
+ * @param {!WebInspector.CSSSourceFrame.ColorPosition} colorPosition
+ */
+WebInspector.CSSSourceFrame.ColorSwatch = function(cssSourceFrame, colorPosition)

[lushnikov, 2016/07/20 00:49:23]

And it's clear that this class by itself doesn't have much value - let's inline
the constructor in the _putColorSwatchesInline method:

var swatch = WebInspector.ColorSwatch.create();
swatch.setColorText(colorPosition.color.asString(WebInspector.Color.Format.Original))
swatch.iconElement().title = WebInspector.UIString("Open color picker.");
swatch.iconElement().addEventListener("click", this._showSpectrum.bind(this,
swatch, colorPosition), true);
swatch.hideText(true);

[flandy, 2016/07/20 01:56:18]

Done.

+{
+    this._cssSourceFrame = cssSourceFrame;
+    this._colorPosition = colorPosition;
+
+    this._swatch = WebInspector.ColorSwatch.create();
+    this._swatch.setColorText(colorPosition.color.asString(WebInspector.Color.Format.Original) || "white");

[lushnikov, 2016/07/20 00:49:23]

As you create color objects only with WebInspector.Color.parse, color object
will always
have original text.

So, you can safely cast colorPosition.color.asString(WI.Color.Format.Original)
to {!string}

[flandy, 2016/07/20 01:56:18]

The compiler no longer complains about this after inlining the constructor. 

+    this._swatch.iconElement().title = WebInspector.UIString("Open color picker.");
+    this._swatch.iconElement().addEventListener("click", this._handleClick.bind(this), true);
+    this._swatch.hideText(true);
+}
+
+WebInspector.CSSSourceFrame.ColorSwatch.prototype = {
+    /**
+     * @param {!Event} event
+     */
+    _handleClick: function(event)
+    {
+        this._cssSourceFrame._showSpectrum(this);
+        event.consume(true);
+    },
+
+    /**
+     * @return {!WebInspector.ColorSwatch}
+     */
+    element: function()
+    {
+        return this._swatch;
+    },
+
+    /**
+     * @return {!WebInspector.CSSSourceFrame.ColorPosition}
+     */
+    colorPosition: function()
+    {
+        return this._colorPosition;
+    },
+
+    /**
+     * @param {string} colorString
+     */
+    setColorText: function(colorString)
+    {
+        this._swatch.setColorText(colorString);
+    }
+}
+
+/**
+ * @constructor
  * @implements {WebInspector.TextEditorAutocompleteDelegate}
  */
 WebInspector.CSSSourceFrame.AutocompleteDelegate = function()
 {
     this._simpleDelegate = new WebInspector.SimpleAutocompleteDelegate(".-$");
 }
 
 WebInspector.CSSSourceFrame._backtrackDepth = 10;
 
 WebInspector.CSSSourceFrame.AutocompleteDelegate.prototype = {
@@ skipping 73 matching lines @@
         var propertyToken = this._backtrackPropertyToken(editor, prefixRange.startLine, prefixRange.startColumn - 1);
         if (!propertyToken)
             return this._simpleDelegate.wordsWithPrefix(editor, prefixRange, substituteRange);
 
         var line = editor.line(prefixRange.startLine);
         var tokenContent = line.substring(propertyToken.startColumn, propertyToken.endColumn);
         var keywords = WebInspector.CSSMetadata.keywordsForProperty(tokenContent);
         return keywords.startsWith(prefix);
     },
 }