Skip to content

Commit bd8dc0c

Browse files
committed
Fix bug when values do not have associated components.
1 parent 5097bf3 commit bd8dc0c

1 file changed

Lines changed: 13 additions & 11 deletions

File tree

src/ui/Palette.js

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -45,17 +45,19 @@ var Palette = this.Palette = Base.extend(Callback, /** @lends Palette# */{
4545
// directly link the value to the component and observe change.
4646
this._values = Base.each(values, function(value, name) {
4747
var component = components[name];
48-
Base.define(values, name, {
49-
enumerable: true,
50-
configurable: true,
51-
writable: true,
52-
get: function() {
53-
return component._value;
54-
},
55-
set: function(val) {
56-
component.setValue(val);
57-
}
58-
});
48+
if (component) {
49+
Base.define(values, name, {
50+
enumerable: true,
51+
configurable: true,
52+
writable: true,
53+
get: function() {
54+
return component._value;
55+
},
56+
set: function(val) {
57+
component.setValue(val);
58+
}
59+
});
60+
}
5961
});
6062
if (window.paper)
6163
paper.palettes.push(this);

0 commit comments

Comments
 (0)