-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding WeeklyWorkoutActivity. #13
Conversation
|
||
|
||
|
||
// val itemList2: ArrayList<YourItem2> = ArrayList() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused comments please remove
//import android.view.View | ||
//import android.view.ViewGroup | ||
//import android.widget.TextView | ||
//import androidx.recyclerview.widget.RecyclerView |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why commenting?
@@ -0,0 +1,3 @@ | |||
package com.modarb.android.ui.workout.models | |||
|
|||
//data class YourItem2(val textView: String) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the purpose of this class ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
items type for the second timeline
app/src/main/res/values/strings.xml
Outdated
@@ -95,5 +95,6 @@ | |||
<string name="Block1_your_personalized_plan">Block 1 : Your personalized plan</string> | |||
<string name="Week1_foundation">Week 1 : Foundation</string> | |||
<string name="Week1_description">Start easy in the first week to let your body get used to the workout. It sets the baseline for your progress in the weeks ahead.</string> | |||
<string name="Day1_4_Full_body">Day 1 / 4 -Full-body</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use tools instead of creating a string for an initial view that will be replaced by an API response
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work but please handle the changes.
Timeline need to edit.