diff options
author | Maxime Quandalle <maxime@quandalle.com> | 2016-01-03 23:22:08 +0100 |
---|---|---|
committer | Maxime Quandalle <maxime@quandalle.com> | 2016-01-03 23:40:08 +0100 |
commit | 46747b01fb3eb42901518a3fdb8a5f6d823465eb (patch) | |
tree | 73c203c7c47f6a74c4f4d7aa20854b2d401d2d2f /client/components/cards | |
parent | 8c01d51577e3748d2c1b38b247d6c2cc4e7dbb08 (diff) | |
download | wekan-46747b01fb3eb42901518a3fdb8a5f6d823465eb.tar.gz wekan-46747b01fb3eb42901518a3fdb8a5f6d823465eb.tar.bz2 wekan-46747b01fb3eb42901518a3fdb8a5f6d823465eb.zip |
Fix card infinite scrolling on card activities
I imagine blaze-component changed their Mixins API since I written
this code. We need some tests to avoid this kind of regressions when
updating dependencies!
Fixes #420
Diffstat (limited to 'client/components/cards')
-rw-r--r-- | client/components/cards/cardDetails.js | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/client/components/cards/cardDetails.js b/client/components/cards/cardDetails.js index bfe9f755..f6cd4140 100644 --- a/client/components/cards/cardDetails.js +++ b/client/components/cards/cardDetails.js @@ -8,8 +8,11 @@ BlazeComponent.extendComponent({ }, calculateNextPeak() { - const altitude = this.find('.js-card-details').scrollHeight; - this.callFirstWith(this, 'setNextPeak', altitude); + const cardElement = this.find('.js-card-details'); + if (cardElement) { + const altitude = cardElement.scrollHeight; + this.callFirstWith(this, 'setNextPeak', altitude); + } }, reachNextPeak() { @@ -21,6 +24,7 @@ BlazeComponent.extendComponent({ this.isLoaded = new ReactiveVar(false); this.parentComponent().showOverlay.set(true); this.parentComponent().mouseHasEnterCardDetails = false; + this.calculateNextPeak(); }, scrollParentContainer() { |