fix: paper analysis

topic number and text answer layout
JingBh · Feb 12, 2022 · 9c9bbd2 · 9c9bbd2
1 parent a15347b
commit 9c9bbd2
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 23 deletions.
diff --git a/app/src/main/java/top/jingbh/zhixuehelper/ui/exam/PaperAnalysisActivity.kt b/app/src/main/java/top/jingbh/zhixuehelper/ui/exam/PaperAnalysisActivity.kt
@@ -125,19 +125,6 @@ class PaperAnalysisActivity : AppCompatActivity() {
             }
         }
 
-        /*lifecycleScope.launch {
-            repeatOnLifecycle(Lifecycle.State.STARTED) {
-                viewModel.uiState
-                    .map { it.selectedIndex }
-                    .filter { it >= 0 }
-                    .distinctUntilChanged()
-                    .collectLatest {
-                        indexAdapter.notifyItemChanged(it)
-                        binding.list.smoothScrollToPosition(it)
-                    }
-            }
-        }*/
-
         lifecycleScope.launch {
             viewModel.analysis
                 .collectLatest { analysis ->
@@ -214,14 +201,21 @@ class PaperAnalysisActivity : AppCompatActivity() {
         private val binding: ItemPaperAnalysisIndexBinding
     ) : RecyclerView.ViewHolder(binding.root) {
         fun bind(topic: ExamPaperTopic, selected: Boolean = false) {
-            binding.text.text = topic.id.toString()
-            binding.text.setTextColor(
-                when (topic.getCorrectness()) {
-                    ExamPaperTopic.Correctness.CORRECT -> colorCorrectRoles.accent
-                    ExamPaperTopic.Correctness.HALF_CORRECT -> colorHalfCorrectRoles.accent
-                    ExamPaperTopic.Correctness.WRONG -> colorWrongRoles.accent
+            binding.text.apply {
+                text = topic.title.let { title ->
+                    if (title.length > 2) {
+                        title.substring(0, 2)
+                    } else title
                 }
-            )
+
+                setTextColor(
+                    when (topic.getCorrectness()) {
+                        ExamPaperTopic.Correctness.CORRECT -> colorCorrectRoles.accent
+                        ExamPaperTopic.Correctness.HALF_CORRECT -> colorHalfCorrectRoles.accent
+                        ExamPaperTopic.Correctness.WRONG -> colorWrongRoles.accent
+                    }
+                )
+            }
 
             binding.root.isSelected = selected
             if (!selected) binding.root.setOnClickListener {

diff --git a/app/src/main/java/top/jingbh/zhixuehelper/ui/exam/PaperAnalysisTopicViewHolder.kt b/app/src/main/java/top/jingbh/zhixuehelper/ui/exam/PaperAnalysisTopicViewHolder.kt
@@ -28,7 +28,7 @@ class PaperAnalysisTopicViewHolder private constructor(
     private val context = binding.root.context
 
     fun bind(topic: ExamPaperTopic) {
-        binding.title.text = context.getString(R.string.paper_topic_title, topic.id)
+        binding.title.text = context.getString(R.string.paper_topic_title, topic.title)
 
         binding.userAnswerCard.apply {
             val answerLayout = when (topic.type) {

diff --git a/app/src/main/res/layout/layout_topic_text_answer.xml b/app/src/main/res/layout/layout_topic_text_answer.xml
@@ -18,13 +18,14 @@
 
     <com.google.android.material.textview.MaterialTextView
         android:id="@+id/answerText"
-        android:layout_width="wrap_content"
+        android:layout_width="0dp"
         android:layout_height="wrap_content"
-        android:layout_marginStart="16dp"
+        android:layout_marginHorizontal="16dp"
         android:layout_marginBottom="8dp"
         android:textAppearance="?attr/textAppearanceHeadlineMedium"
         app:layout_constraintBottom_toBottomOf="parent"
         app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
         app:layout_constraintTop_toBottomOf="@id/answerTitle"
         app:layout_constraintVertical_bias="0.0"
         tools:text="A" />