Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Fix #4710 the button scaling problem in largest text, and change button name" #4987

Merged
merged 12 commits into from
Jun 15, 2023
Merged
Next Next commit
"Fix the buttom scaling problem in largest text, and change button name"
  • Loading branch information
XichengSpencer committed May 17, 2023
commit 90507270c7e6521790ed5a9df6e9de0d40cd1cc7
10 changes: 6 additions & 4 deletions app/src/main/res/layout/resume_lesson_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,9 @@
<androidx.constraintlayout.widget.ConstraintLayout
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_marginStart="32dp"
android:layout_marginStart="28dp"
android:layout_marginTop="32dp"
android:layout_marginEnd="32dp"
android:layout_marginEnd="28dp"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
Expand All @@ -83,7 +83,8 @@
<Button
android:id="@+id/resume_lesson_start_over_button"
style="@style/StartOverLessonButton"
android:layout_marginEnd="12dp"
android:layout_marginEnd="8dp"
app:layout_constraintHorizontal_weight="1"
android:text="@string/start_over_lesson_button"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toStartOf="@+id/resume_lesson_continue_button"
Expand All @@ -93,7 +94,8 @@
<Button
android:id="@+id/resume_lesson_continue_button"
style="@style/ContinueLessonButton"
android:layout_marginStart="12dp"
app:layout_constraintHorizontal_weight="1"
android:layout_marginStart="8dp"
android:text="@string/resume_lesson_button"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintEnd_toEndOf="parent"
Expand Down
4 changes: 2 additions & 2 deletions app/src/main/res/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -554,8 +554,8 @@
<string name="help_activity_copyright_licenses_list">copyright licenses list</string>
<!-- ResumeLessonActivity -->
<string name="resume_lesson_activity_title">Resume Lesson</string>
<string name="resume_lesson_button">Continue</string>
<string name="start_over_lesson_button">Start Over</string>
<string name="resume_lesson_button">Resume</string>
<string name="start_over_lesson_button">Reset</string>
<!-- Home screen greeting fragments. -->
<string name="home_screen_good_morning_greeting_fragment">Good morning,</string>
<string name="home_screen_good_afternoon_greeting_fragment">Good afternoon,</string>
Expand Down
11 changes: 3 additions & 8 deletions app/src/main/res/values/styles.xml
Original file line number Diff line number Diff line change
Expand Up @@ -411,11 +411,8 @@
</style>
<!-- Start Over Lesson Button -->
<style name="SecondaryButton" parent="TextAppearance.AppCompat.Widget.Button">
<item name="android:layout_width">144dp</item>
<item name="android:layout_width">0dp</item>
adhiamboperes marked this conversation as resolved.
Show resolved Hide resolved
<item name="android:layout_height">wrap_content</item>
<item name="android:paddingEnd">12dp</item>
<item name="android:paddingStart">12dp</item>
<item name="android:minWidth">144dp</item>
<item name="android:background">@drawable/secondary_button_background</item>
<item name="android:fontFamily">sans-serif-medium</item>
<item name="android:drawablePadding">4dp</item>
Expand All @@ -429,12 +426,9 @@
</style>
<!-- Continue Lesson Button -->
<style name="ContinueLessonButton" parent="TextAppearance.AppCompat.Widget.Button">
<item name="android:layout_width">144dp</item>
<item name="android:layout_width">0dp</item>
adhiamboperes marked this conversation as resolved.
Show resolved Hide resolved
<item name="android:layout_height">wrap_content</item>
<item name="android:drawablePadding">4dp</item>
<item name="android:paddingEnd">12dp</item>
<item name="android:paddingStart">12dp</item>
<item name="android:minWidth">144dp</item>
adhiamboperes marked this conversation as resolved.
Show resolved Hide resolved
<item name="android:background">@drawable/state_button_primary_background</item>
<item name="drawableEndCompat">@drawable/ic_arrow_right_alt_24dp</item>
<item name="drawableTint">@color/component_color_shared_white_background_color</item>
Expand All @@ -443,6 +437,7 @@
<item name="android:textAllCaps">true</item>
<item name="android:textColor">@color/component_color_shared_secondary_4_text_color</item>
<item name="android:textSize">14sp</item>

</style>

<style name="BorderlessMaterialButton" parent="Widget.AppCompat.Button.Borderless" />
Expand Down