DON-1092 – Explain blurredSinceLastMajorScroll resets

thebiggive · Nov 27, 2024 · 848618c · 848618c
1 parent 7ec20f2
commit 848618c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/app/explore/explore.component.ts b/src/app/explore/explore.component.ts
@@ -132,6 +132,7 @@ export class ExploreComponent implements OnDestroy, OnInit {
   onScroll() {
     const scrollPositionY = this.scroller.getScrollPosition()[1];
     if (scrollPositionY < this.smallestSignificantScrollPx) {
+      // If we're now near the top, reset any previous input blurring as it might be helpful to blur again.
       this.blurredSinceLastMajorScroll = false;
       return;
     }

diff --git a/src/app/meta-campaign/meta-campaign.component.ts b/src/app/meta-campaign/meta-campaign.component.ts
@@ -209,7 +209,9 @@ export class MetaCampaignComponent implements AfterViewChecked, OnDestroy, OnIni
   onScroll() {
     const scrollPositionY = this.scroller.getScrollPosition()[1];
     if (scrollPositionY < this.smallestSignificantScrollPx) {
+      // If we're now near the top, reset any previous input blurring as it might be helpful to blur again.
       this.blurredSinceLastMajorScroll = false;
+
       // On return with internal app nav, automatic position seems to be [0,59]
       // or so as of Nov '22. So we want only larger scrolls to be picked up as
       // donor intervention and to turn off auto-scroll + trigger loading of