WГ
Чтоб использовало bootstrap css,а не самописное
Size: a a a
2019 October 22
v
ну так переопредели кллассы
v
в чем проблема
YK
Да, вероятно. Стоит заменить takeWhile на filter.
шикарно! Проще вышло!
YK
шикарно! Проще вышло!
setUpStartStopEvents(): Observable<void> {
const autoModeToggled$ = this.autoModeSlideToggle.change.pipe(
switchMapTo(interval(1000)),
filter(() => this.autoModeSlideToggle.checked),
);
const newSimulationClicks$ = fromEvent(this.newSimulationButton.nativeElement, 'click');
return merge<void>(newSimulationClicks$, autoModeToggled$);
}YK
Да, вероятно. Стоит заменить takeWhile на filter.
Кода получилось меньше. А не страшно, что мы внутри фильтра смотрим на булевскую переменную вне потока?
V
Кода получилось меньше. А не страшно, что мы внутри фильтра смотрим на булевскую переменную вне потока?
Не страшно.
V
Это нормальная практика
YK
Знал ведь, что с партишином слишком хитро выходит, спасибо!
V
Кстати, можно же ещё проще
V
interval(1000).pipe(
filter(() => !!toggle.value)
)
.subscribe(() => yourCalculations())
filter(() => !!toggle.value)
)
.subscribe(() => yourCalculations())
V
Правда, тогда будет задержка первого вычисления
V
Что может быть неприемлемо для UX
V
Но тут уж от ситуации зависит.
YK
В данном случае приемлемо
YK
Но тут ведь получается, что интервал как бы вечный
V
Повесить unsubscribe на component onDestroy
YK
Ну да, у меня все равно все на async пайпах
YK
Повесить unsubscribe на component onDestroy
Я тут скорее имею в виду, что интервал будет запускаться при старте компонента, а не при нажатии на галочку