-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
…d colors for backgrounds fixed!
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 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) |
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.
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) |
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.
Kindly use some other animation.
@amindadgar Kindly send different PR for different issues to maintain the ease in code review process. |
sorry for my late answer, I didn't see the notification. |
No description provided.