From 785d8390e8c0161a8bde91c7159ee4174b4b8ad3 Mon Sep 17 00:00:00 2001
From: Dani Palou <dani@moodle.com>
Date: Thu, 21 Apr 2022 08:55:26 +0200
Subject: [PATCH] MOBILE-3833 course: Fix conflict with automatic scrolls

---
 .../course/components/course-format/course-format.ts   | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/core/features/course/components/course-format/course-format.ts b/src/core/features/course/components/course-format/course-format.ts
index ca8f8b29c..b6bada1f1 100644
--- a/src/core/features/course/components/course-format/course-format.ts
+++ b/src/core/features/course/components/course-format/course-format.ts
@@ -509,12 +509,14 @@ export class CoreCourseFormatComponent implements OnInit, OnChanges, OnDestroy {
         const moduleIdToScroll = this.moduleId && previousValue === undefined ? this.moduleId : moduleId;
         if (moduleIdToScroll) {
             this.scrollToModule(moduleIdToScroll);
-        } else {
-            this.content.scrollToTop(0);
         }
 
-        if (!previousValue || previousValue.id != newSection.id) {
-            // First load or section changed, add log in Moodle.
+        if (!previousValue || previousValue.id !== newSection.id) {
+            // First load or section changed.
+            if (!moduleIdToScroll) {
+                this.content.scrollToTop(0);
+            }
+
             CoreUtils.ignoreErrors(
                 CoreCourse.logView(this.course.id, newSection.section, undefined, this.course.fullname),
             );