improve UX by preventing recyclerview auto scroll to TOP for quiz

joelcuy · joelcuy · commit b922a750a078 · 2021-12-25T03:28:33.000+08:00
diff --git a/app/src/main/java/com/example/nobulijava/activity/AdminQuizListActivity.java b/app/src/main/java/com/example/nobulijava/activity/AdminQuizListActivity.java
@@ -1,5 +1,6 @@
 package com.example.nobulijava.activity;
 
+import androidx.annotation.NonNull;
 import androidx.appcompat.app.AppCompatActivity;
 import androidx.recyclerview.widget.RecyclerView;
 
@@ -10,6 +11,8 @@
 import com.example.nobulijava.R;
 import com.example.nobulijava.adapters.QuizAdapter;
 import com.example.nobulijava.model.QuizObj;
+import com.google.android.gms.tasks.OnCompleteListener;
+import com.google.android.gms.tasks.Task;
 import com.google.android.material.floatingactionbutton.FloatingActionButton;
 import com.google.firebase.database.DataSnapshot;
 import com.google.firebase.database.DatabaseError;
@@ -27,6 +30,8 @@ public class AdminQuizListActivity extends AppCompatActivity {
     private DatabaseReference mDatabase;
 
     private ArrayList<QuizObj> quizObjArrayList;
+    QuizAdapter quizAdapter;
+
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
@@ -45,34 +50,26 @@ public void onClick(View v) {
                 startActivity(new Intent(AdminQuizListActivity.this, AdminQuizAddActivity.class));
             }
         });
+
+        quizObjArrayList = new ArrayList<>();
+        quizAdapter = new QuizAdapter(quizObjArrayList, AdminQuizListActivity.this);
+        recyclerViewQuizList.setAdapter(quizAdapter);
+
         mDatabase.child("Quiz").addValueEventListener(new ValueEventListener() {
             @Override
             public void onDataChange(DataSnapshot dataSnapshot) {
-                quizObjArrayList = new ArrayList<>();
-                for (DataSnapshot quizSnapshot: dataSnapshot.getChildren()) {
+                quizObjArrayList.clear();
+                for (DataSnapshot quizSnapshot : dataSnapshot.getChildren()) {
                     QuizObj quizObj = quizSnapshot.getValue(QuizObj.class);
                     quizObj.setQuizID(quizSnapshot.getKey());
                     quizObjArrayList.add(quizObj);
-
                 }
-                QuizAdapter quizAdapter = new QuizAdapter(quizObjArrayList, AdminQuizListActivity.this);
-                recyclerViewQuizList.setAdapter(quizAdapter);
-                //TODO when delete item, recyclerview scrolls back to top
+                quizAdapter.notifyDataSetChanged();
             }
 
             @Override
             public void onCancelled(DatabaseError databaseError) {
             }
         });
-
-
-
-
-
-
-
-
     }
-
-
 }
diff --git a/app/src/main/res/layout/template_quiz.xml b/app/src/main/res/layout/template_quiz.xml
@@ -20,8 +20,8 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginStart="8dp"
+            android:layout_marginTop="16dp"
             android:text="1."
-            app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="parent" />
 
@@ -31,9 +31,7 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginTop="16dp"
-            android:layout_marginBottom="16dp"
             android:text="Question"
-            app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintStart_toEndOf="@+id/textView_quizList_quizNo"
             app:layout_constraintTop_toTopOf="parent" />
 
@@ -42,10 +40,20 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
+            android:layout_marginTop="16dp"
             android:text="Answer"
-            app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintStart_toEndOf="@+id/textView_quizList_question"
             app:layout_constraintTop_toTopOf="parent" />
+
+        <ImageView
+            android:id="@+id/imageView_quizList_image"
+            android:layout_width="0dp"
+            android:layout_height="100dp"
+            android:layout_marginTop="16dp"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintEnd_toEndOf="parent"
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintTop_toBottomOf="@+id/textView_quizList_question" />
     </androidx.constraintlayout.widget.ConstraintLayout>
 
 
