Fix #2581: Marquee auto restart issue

app/src/main/java/org/oppia/android/app/devoptions/markchapterscompleted/MarkChaptersCompletedFragmentPresenter.kt

@@ -78,6 +78,14 @@ class MarkChaptersCompletedFragmentPresenter @Inject constructor(
               )
           }
         }
+      } else if (binding.isAllChecked == true) {
+        binding.isAllChecked = false
+        getMarkChaptersCompletedViewModel().getItemList().forEach { viewModel ->
+          if (viewModel is ChapterSummaryViewModel) {
+            if (!viewModel.checkIfChapterIsCompleted())
+              chapterUnselected(viewModel.chapterIndex, viewModel.nextStoryIndex)
+          }
+        }
       }
     }
 

app/src/main/java/org/oppia/android/app/devoptions/markstoriescompleted/MarkStoriesCompletedFragmentPresenter.kt

@@ -64,8 +64,11 @@ class MarkStoriesCompletedFragmentPresenter @Inject constructor(
     }
 
     binding.markStoriesCompletedAllCheckBoxContainer.setOnClickListener {
-      if (binding.isAllChecked == null || binding.isAllChecked == false)
+      if (binding.isAllChecked == null || binding.isAllChecked == false) {
         binding.isAllChecked = true
+      } else if (binding.isAllChecked == true) {
+        binding.isAllChecked = false
+      }
     }
 
     binding.markStoriesCompletedAllCheckBox.setOnCheckedChangeListener { _, isChecked ->
@@ -74,6 +77,10 @@ class MarkStoriesCompletedFragmentPresenter @Inject constructor(
           if (!viewModel.isCompleted)
             storySelected(viewModel.storySummary.storyId)
         }
+      } else {
+        getMarkStoriesCompletedViewModel().getStorySummaryMap().values.forEach { viewModel ->
+          if (!viewModel.isCompleted) storyUnselected(viewModel.storySummary.storyId)
+        }
       }
       bindingAdapter.notifyDataSetChanged()
     }

app/src/main/java/org/oppia/android/app/devoptions/marktopicscompleted/MarkTopicsCompletedFragmentPresenter.kt

@@ -64,15 +64,18 @@ class MarkTopicsCompletedFragmentPresenter @Inject constructor(
     }
 
     binding.markTopicsCompletedAllCheckBoxContainer.setOnClickListener {
-      if (binding.isAllChecked == null || binding.isAllChecked == false)
-        binding.isAllChecked = true
+      binding.isAllChecked = !(binding.isAllChecked ?: false)
     }
 
     binding.markTopicsCompletedAllCheckBox.setOnCheckedChangeListener { _, isChecked ->
       if (isChecked) {
         getMarkTopicsCompletedViewModel().getTopicList().forEach { viewModel ->
           if (!viewModel.isCompleted) topicSelected(viewModel.topic.topicId)
         }
+      } else {
+        getMarkTopicsCompletedViewModel().getTopicList().forEach { viewModel ->
+          if (!viewModel.isCompleted) topicUnselected(viewModel.topic.topicId)
+        }
       }
       bindingAdapter.notifyDataSetChanged()
     }

app/src/main/java/org/oppia/android/app/story/StoryFragmentPresenter.kt

@@ -28,6 +28,7 @@ import org.oppia.android.app.story.storyitemviewmodel.StoryHeaderViewModel
 import org.oppia.android.app.story.storyitemviewmodel.StoryItemViewModel
 import org.oppia.android.app.topic.RouteToResumeLessonListener
 import org.oppia.android.app.translation.AppLanguageResourceHandler
+import org.oppia.android.app.utility.LifecycleSafeTimerFactory
 import org.oppia.android.databinding.StoryChapterViewBinding
 import org.oppia.android.databinding.StoryFragmentBinding
 import org.oppia.android.databinding.StoryHeaderViewBinding
@@ -49,6 +50,7 @@ class StoryFragmentPresenter @Inject constructor(
   private val explorationDataController: ExplorationDataController,
   @DefaultResourceBucketName private val resourceBucketName: String,
   @TopicHtmlParserEntityType private val entityType: String,
+  private val lifecycleSafeTimerFactory: LifecycleSafeTimerFactory,
   private val resourceHandler: AppLanguageResourceHandler
 ) {
   private val routeToExplorationListener = activity as RouteToExplorationListener
@@ -86,6 +88,12 @@ class StoryFragmentPresenter @Inject constructor(
       binding.storyToolbarTitle.isSelected = true
     }
 
+    lifecycleSafeTimerFactory.createTimer(25000).observe(fragment.viewLifecycleOwner) {
+      if (binding.storyToolbarTitle.isSelected) {
+        binding.storyToolbarTitle.isSelected = false
+      }
+    }
+
     linearLayoutManager = LinearLayoutManager(activity.applicationContext)
     linearSmoothScroller = createSmoothScroller()
 

app/src/main/java/org/oppia/android/app/topic/TopicFragmentPresenter.kt

@@ -12,6 +12,7 @@ import com.google.android.material.tabs.TabLayoutMediator
 import org.oppia.android.R
 import org.oppia.android.app.fragment.FragmentScope
 import org.oppia.android.app.translation.AppLanguageResourceHandler
+import org.oppia.android.app.utility.LifecycleSafeTimerFactory
 import org.oppia.android.app.viewmodel.ViewModelProvider
 import org.oppia.android.databinding.TopicFragmentBinding
 import org.oppia.android.domain.oppialogger.OppiaLogger
@@ -24,6 +25,7 @@ class TopicFragmentPresenter @Inject constructor(
   private val fragment: Fragment,
   private val viewModelProvider: ViewModelProvider<TopicViewModel>,
   private val oppiaLogger: OppiaLogger,
+  private val lifecycleSafeTimerFactory: LifecycleSafeTimerFactory,
   @EnablePracticeTab private val enablePracticeTab: Boolean,
   private val resourceHandler: AppLanguageResourceHandler
 ) {
@@ -61,6 +63,12 @@ class TopicFragmentPresenter @Inject constructor(
       binding.topicToolbarTitle.isSelected = true
     }
 
+    lifecycleSafeTimerFactory.createTimer(25000).observe(fragment.viewLifecycleOwner) {
+      if (binding.topicToolbarTitle.isSelected) {
+        binding.topicToolbarTitle.isSelected = false
+      }
+    }
+
     val viewModel = getTopicViewModel()
     viewModel.setInternalProfileId(internalProfileId)
     viewModel.setTopicId(topicId)