Julian Descottes [:jdescottes] Reporter
	Description • 8 years ago

Regression from 1262439.

STRs:

- open inspector > ruleview
- create a property color: red
- open the color picker 
- click on the eyedropper icon
- pick a color on the page
- (color picker is automatically closed, picked color is set as property value)
- open the color picker
- try to update the color using the gradient background

ER: Colorpicker widget should still work
AR: Colorpicker widget is no longer working

	Julian Descottes [:jdescottes] Reporter
	Comment 1 • 8 years ago

The eyedropper gets hidden when canceled and when selecting a color.
The Tooltip should listen to both events to update its internal state flags.

Review commit: https://reviewboard.mozilla.org/r/66908/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/66908/

	Matteo Ferretti [:zer0] [:matteo]
	Comment 2 • 8 years ago

Comment on attachment 8774453 [details]
Bug 1289152 - switch eyedropperOpen flag in Tooltip when picking color;

https://reviewboard.mozilla.org/r/66908/#review64510

::: devtools/client/shared/widgets/Tooltip.js:868
(Diff revision 1)
>      this.hide();
>  
>      this.tooltip.emit("eyedropper-opened");
>    },
>  
> +  _onEyeDropperDone: function () {

We usually uses the prefixed `on` for function's name that are actually listener, bound to specific events; in that case it seems there is no direct correlation, so it's a bit misleading. I would probably call the function just `this._eyeDropperDone` – or anything else doesn't include `on` as prefix.

(I'm not going to block this bug just for the naming convention, since you're also in PTO, I writing this done just as reminder, maybe we could fix it in one of the eye dropper follow up bug)

	Pulsebot
	Comment 3 • 8 years ago

Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/fx-team/rev/80206c183389
Switch eyedropperOpen flag in Tooltip when picking color. r=zer0

	Carsten Book [:Tomcat]
	Comment 4 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/80206c183389