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.

Sign up for GitHub

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

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

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
9050727
"Fix the buttom scaling problem in largest text, and change button name"
XichengSpencer May 17, 2023
8cc523d
Merge branch 'develop' into #4710
XichengSpencer May 17, 2023
281ce30
"Move the Resume button style to view level"
XichengSpencer May 25, 2023
a9114ad
Merge remote-tracking branch 'origin/#4710' into #4710
XichengSpencer May 25, 2023
d57ecb2
"Center the button with icon by using material button"
XichengSpencer May 31, 2023
e58b523
"Remove the drawable padding attribute since it seems useless"
XichengSpencer May 31, 2023
77f8769
"Change the button wrapper layout to flexbox layout and center the bu…
XichengSpencer Jun 6, 2023
ac47851
"Also delete unused button style"
XichengSpencer Jun 6, 2023
779217b
Merge branch 'develop' into #4710
adhiamboperes Jun 6, 2023
931650a
Merge branch 'develop' into #4710
XichengSpencer Jun 7, 2023
5e00bea
Merge branch 'develop' into #4710
adhiamboperes Jun 9, 2023
b2d57f3
Merge branch 'develop' into #4710
adhiamboperes Jun 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions app/src/main/res/layout/resume_lesson_fragment.xml
View file
Open in desktop
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
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -553,8 +553,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
View file
Open in desktop
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