Skip to content

Commit

Permalink
refactor(md-select): try to fix that nasty multiple select bug
Browse files Browse the repository at this point in the history
  • Loading branch information
Thanood committed Jun 1, 2016
1 parent d77109d commit 12e6904
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/select/select.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ export class MdSelect {
}
attached() {
this.subscriptions.push(this.bindingEngine.propertyObserver(this.element, 'value').subscribe(this.handleChangeFromViewModel));
// this.subscriptions.push(this.bindingEngine.propertyObserver(this.element, 'selectedOptions').subscribe(this.notifyBindingEngine.bind(this)));
// $(this.element).material_select(() => {
// this.log.warn('materialize callback', $(this.element).val());
// this.handleChangeFromNativeSelect();
Expand All @@ -42,6 +43,10 @@ export class MdSelect {
});
}

notifyBindingEngine() {
this.log.debug('selectedOptions changed', arguments);
}

handleChangeFromNativeSelect() {
if (!this._suspendUpdate) {
this.log.debug('handleChangeFromNativeSelect', this.element.value, $(this.element).val());
Expand Down

0 comments on commit 12e6904

Please sign in to comment.