Merge pull request #30 from nicoespeon/debounce-datepicker

Don't update on every datepicker keystroke. Close #28
nicoespeon · Jun 12, 2016 · af4b6a1 · af4b6a1
2 parents c9d9df5 + c1ee148
commit af4b6a1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/app/application.js b/app/application.js
@@ -183,7 +183,7 @@ function main({ DOMAboveChart, DOMBelowChart, TrelloFetch, TrelloMissingInfo, St
     displayedLists$: trelloDisplayedLists$,
     dates$: selectedDates$,
     props$: Observable.of({
-      label: 'Get actions',
+      label: 'Refresh data',
       classNames: ['btn waves-effect waves-light trello-green'],
     }),
     previewTomorrow$: previewTomorrow.checked$,

diff --git a/app/components/LabeledDatePicker/LabeledDatePicker.js b/app/components/LabeledDatePicker/LabeledDatePicker.js
@@ -6,7 +6,9 @@ function LabeledDatePicker({ DOM, props$, value$ }) {
   const newSelected$ = DOM
     .select('.datepicker')
     .events('change')
-    .map(ev => ev.target.value);
+    .debounce(500)
+    .map(ev => ev.target.value)
+    .distinctUntilChanged();
 
   const selected$ = Observable.merge(
     value$,