Merge pull request #7 from CraveFood/fix-scroll-bug

Fixing scroll bug on operation text
CraveFood · Feb 25, 2019 · 000ee6d · 000ee6d
2 parents 94ab64c + 2bad0c7
commit 000ee6d
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 1 deletion.
diff --git a/cravecalculator/src/main/java/com/cravefood/cravecalculator/CalculatorFragment.kt b/cravecalculator/src/main/java/com/cravefood/cravecalculator/CalculatorFragment.kt
@@ -4,6 +4,7 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
+import android.widget.HorizontalScrollView
 import android.widget.Toast
 import androidx.fragment.app.Fragment
 import androidx.lifecycle.Observer
@@ -33,6 +34,7 @@ class CalculatorFragment : Fragment() {
 
         viewModel.currentExpressionObservable.observe(this, Observer {
             textViewOperation.text = it
+            focusOperationText()
         })
 
         viewModel.resultObservable.observe(this, Observer {
@@ -94,6 +96,13 @@ class CalculatorFragment : Fragment() {
         else activity?.onBackPressed()
     }
 
+    private fun focusOperationText() {
+        horizontalScrollView.postDelayed(
+            Runnable { horizontalScrollView.fullScroll(HorizontalScrollView.FOCUS_RIGHT) },
+            100L
+        )
+    }
+
     companion object {
         fun newInstance(
             cancelListener: (() -> Unit)? = null,

diff --git a/cravecalculator/src/main/res/layout/calculator_fragment.xml b/cravecalculator/src/main/res/layout/calculator_fragment.xml
@@ -22,6 +22,7 @@
       android:paddingStart="8dp">
 
     <HorizontalScrollView
+        android:id="@+id/horizontalScrollView"
         android:layout_width="match_parent"
         app:layout_constraintTop_toTopOf="parent"
         android:layout_marginTop="16dp"

diff --git a/cravecalculator/src/main/res/values/strings.xml b/cravecalculator/src/main/res/values/strings.xml
@@ -1,4 +1,6 @@
 <resources>
+  <public />
+
   <string name="app_name">cravecalculator</string>
 
   <string name="_point">.</string>

diff --git a/cravecalculator/src/main/res/values/styles.xml b/cravecalculator/src/main/res/values/styles.xml
@@ -61,7 +61,7 @@
   </style>
 
   <style name="TextViewOperation">
-    <item name="android:layout_width">match_parent</item>
+    <item name="android:layout_width">wrap_content</item>
     <item name="android:layout_height">wrap_content</item>
     <item name="android:textSize">24sp</item>
     <item name="android:layout_gravity">end</item>