try to make the color picker show the current color

pull/1/head
ansuz 8 years ago
parent 96dac80bd5
commit e2981c0223

@ -52,10 +52,12 @@ define([
$width.on('change', updateBrushWidth); $width.on('change', updateBrushWidth);
var pickColor = function (cb) { var pickColor = function (current, cb) {
// TODO find out why initial color is not being set
// http://jsfiddle.net/j3hZB/
var $picker = $('<input>', { var $picker = $('<input>', {
type: 'color', type: 'color',
value: module.color || '#000' value: '#FFFFFF',
}) })
.css({ .css({
display: 'none', display: 'none',
@ -65,6 +67,7 @@ define([
cb(color); cb(color);
}); });
setTimeout(function () { setTimeout(function () {
$picker.val(current);
$picker.click(); $picker.click();
}); });
}; };
@ -87,11 +90,13 @@ define([
.css({ .css({
'background-color': color, 'background-color': color,
}) })
// FIXME double click doesn't seem to work in chromium currently
.dblclick(function () { .dblclick(function () {
pickColor(function (c) { pickColor($color.css('background-color'), function (c) {
$color.css({ $color.css({
'background-color': c, 'background-color': c,
}); });
setColor(c);
}); });
// TODO commit chosen color to pad metadata: // TODO commit chosen color to pad metadata:
// json.metadata.palette[i] // json.metadata.palette[i]
@ -205,9 +210,8 @@ define([
title: "choose a color", title: "choose a color",
'class': "fa fa-square rightside-button", 'class': "fa fa-square rightside-button",
}) })
.text(' ')
.on('click', function () { .on('click', function () {
pickColor(function (color) { pickColor($color.css('background-color'), function (color) {
setColor(color); setColor(color);
}) })
}); });

Loading…
Cancel
Save