-
Notifications
You must be signed in to change notification settings - Fork 226
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
Consolidate Sample #226
Consolidate Sample #226
Conversation
Signed-off-by: shaishavgandhi05 <shaishgandhi@gmail.com>
Signed-off-by: shaishavgandhi05 <shaishgandhi@gmail.com>
@@ -20,4 +20,7 @@ | |||
<string name="title_activity_main">AutoDispose Demo</string> | |||
<string name="title_activity_arch_component">AutoDispose ArchComponents Demo</string> | |||
<string name="load_image">Load Image</string> | |||
<string name="main_activity_description">Sample Java Activity which uses Architecture Components lifecycle to manage subscription</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.
Tentative descriptions for each Activity. Welcome any feedback!
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.
How about Sample Java Activity demonstrating integration with Architecture Components Lifecycle
?
|
||
val mainActivityButton: Button = findViewById(R.id.mainActivityButton) | ||
mainActivityButton.setOnClickListener { | ||
startActivity(MainActivity::class.java) |
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.
Proposal to rename MainActivity
to JavaActivity
or something similar?
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.
Works for me!
Would you mind adding a screenshot(s) or screenrecording of the new UI for posterity? |
Nit: Can we remove the "AutoDispose" prefix in the button texts in favor of just a toolbar with |
Signed-off-by: shaishavgandhi05 <shaishgandhi@gmail.com>
thanks! |
Description: Consolidate sample to open up different activities. Have also included a brief description before the Button to describe the sample.