Skip to content

Conversation

@jjatinggoyal
Copy link

No description provided.

public void onBindViewHolder(@NonNull ViewHolder holder, final int i) {
final ModelItems item = items.get(i);


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete the unnecessary spaces. Applies to all files

private ViewPager viewPager;
private MyPageAdapter adapter;
@Override
protected void onCreate(Bundle savedInstanceState) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NavigationDrawer functionality is missing as per the problem statement. Please check. You can include the diff categories in NavigationDrawer instead of TabbedLayout


@Override
public Fragment getItem(int i) {
// Log.e("pra"," "+i);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete the commented code. Applies to all files

}


public void moveNext(View view) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to hide the next button when you are in the last page of the view pager. Also, when user clicks on the next button in the last page of the viewpager, then the user must be taken out of the viewpager and can be directed to an empty activity,

viewPager.setCurrentItem(2);
}

public void moveBack(View view){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to hide the back button when you are in the first page of the view pager

}

public void moveSkip(View view) {
viewPager.setCurrentItem(2);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skip functionality is incorrect. OnClick of skip button must make the user skip the viewpager and display the first activity that will be displayed after clicking the Next button on the last page in the viewpager

import android.os.Bundle;
import android.os.Handler;

public class SplashActivity extends Activity {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This activity is not required. The ViewPager is supposed to be the splash screen

testImplementation 'junit:junit:4.+'
androidTestImplementation 'androidx.test.ext:junit:1.1.2'
androidTestImplementation 'androidx.test.espresso:espresso-core:3.3.0'
implementation 'com.journeyapps:zxing-android-embedded:4.1.0'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please include a README file in the project and explain the purpose of usage of the lib

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants