diff options
author | Valerio Virgillito | 2012-03-26 15:35:59 -0700 |
---|---|---|
committer | Valerio Virgillito | 2012-03-26 15:35:59 -0700 |
commit | 309dde5a8c4599cef6a1052c1ff9ee1ad8ec5858 (patch) | |
tree | af3b114d6a83ae62dfcead8bd61bc62efe0c9516 /js | |
parent | 63f0cab872879f72021692f064b044a0cde2459a (diff) | |
parent | 23b89609d7fc26f3523b4a8e354fa8c8208cbdbf (diff) | |
download | ninja-309dde5a8c4599cef6a1052c1ff9ee1ad8ec5858.tar.gz |
Merge pull request #142 from joseeight/Color
Color Bug Fix
Diffstat (limited to 'js')
-rwxr-xr-x | js/controllers/color-controller.js | 5 | ||||
-rwxr-xr-x | js/panels/color/colorpopup-manager.js | 2 |
2 files changed, 3 insertions, 4 deletions
diff --git a/js/controllers/color-controller.js b/js/controllers/color-controller.js index 53573baf..8e000b19 100755 --- a/js/controllers/color-controller.js +++ b/js/controllers/color-controller.js | |||
@@ -198,7 +198,6 @@ exports.ColorController = Montage.create(Component, { | |||
198 | handleChange: { | 198 | handleChange: { |
199 | enumerable: true, | 199 | enumerable: true, |
200 | value: function (e) { | 200 | value: function (e) { |
201 | |||
202 | // | 201 | // |
203 | var actionEvent, color, input = e._event.input, panelMode, mode = e._event.mode; | 202 | var actionEvent, color, input = e._event.input, panelMode, mode = e._event.mode; |
204 | if (this.colorView) { | 203 | if (this.colorView) { |
@@ -228,14 +227,14 @@ exports.ColorController = Montage.create(Component, { | |||
228 | // | 227 | // |
229 | this.fill = color; | 228 | this.fill = color; |
230 | // | 229 | // |
231 | if(e._event.wasSetByCode) return; | 230 | if(e._event.wasSetByCode && mode !== 'nocolor') return; |
232 | // | 231 | // |
233 | this.setColor(mode, color, true); | 232 | this.setColor(mode, color, true); |
234 | } else if (input === 'stroke') { | 233 | } else if (input === 'stroke') { |
235 | // | 234 | // |
236 | this.stroke = color; | 235 | this.stroke = color; |
237 | // | 236 | // |
238 | if(e._event.wasSetByCode) return; | 237 | if(e._event.wasSetByCode && mode !== 'nocolor') return; |
239 | 238 | ||
240 | this.setColor(mode, color, false); | 239 | this.setColor(mode, color, false); |
241 | } | 240 | } |
diff --git a/js/panels/color/colorpopup-manager.js b/js/panels/color/colorpopup-manager.js index a4c9a8c2..d6d94807 100755 --- a/js/panels/color/colorpopup-manager.js +++ b/js/panels/color/colorpopup-manager.js | |||
@@ -68,7 +68,7 @@ exports.ColorPopupManager = Montage.create(Component, { | |||
68 | this.application.ninja.colorController.colorPopupManager.hideColorPopup(); | 68 | this.application.ninja.colorController.colorPopupManager.hideColorPopup(); |
69 | }.bind(this)); | 69 | }.bind(this)); |
70 | // | 70 | // |
71 | document.addEventListener('click', function (e) { | 71 | document.addEventListener('mousedown', function (e) { |
72 | // | 72 | // |
73 | if (e._event.srcElement.id === 'stageCanvas' || e._event.srcElement.id === 'mainContainer' || e._event.srcElement.id === 'drawingCanvas') { | 73 | if (e._event.srcElement.id === 'stageCanvas' || e._event.srcElement.id === 'mainContainer' || e._event.srcElement.id === 'drawingCanvas') { |
74 | this.application.ninja.colorController.colorPopupManager.hideColorPopup(); | 74 | this.application.ninja.colorController.colorPopupManager.hideColorPopup(); |