Skip to content

animations added to fragment transaction for better performance #3

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

Closed
wants to merge 2 commits into from

Conversation

amindadgar
Copy link

No description provided.

Copy link
Owner

@anantcodes anantcodes left a comment

Choose a reason for hiding this comment

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

Use a different animation.
Thank you

@@ -71,23 +71,29 @@ class ControlActivity : AppCompatActivity() {
R.id.language->{
supportFragmentManager.beginTransaction().replace(R.id.frame,
LanguagesFragment()
).commit()
)
.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_FADE)
Copy link
Owner

Choose a reason for hiding this comment

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

Kindly use some other animation.

supportActionBar?.title="Programming languages"
drawerLayout.closeDrawers()
}

R.id.dev->{
supportFragmentManager.beginTransaction().replace(R.id.frame,
DevelopmentFragment()
).commit()
)
.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_FADE)
Copy link
Owner

Choose a reason for hiding this comment

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

Kindly use some other animation.

@anantcodes
Copy link
Owner

@amindadgar Kindly send different PR for different issues to maintain the ease in code review process.

@amindadgar
Copy link
Author

sorry for my late answer, I didn't see the notification.
I'll do it in a day.

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

Successfully merging this pull request may close these issues.

2 participants