Update comments

Author: zsmb13

samplejava/src/main/java/com/example/chattutorial/ChannelActivity.java

@@ -48,7 +48,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
             throw new IllegalStateException("Specifying a channel id is required when starting ChannelActivity");
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         MessageListViewModelFactory factory = new MessageListViewModelFactory(cid);
         ViewModelProvider provider = new ViewModelProvider(this, factory);
         MessageListHeaderViewModel messageListHeaderViewModel = provider.get(MessageListHeaderViewModel.class);
@@ -62,7 +63,7 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
         MessageListViewModelBinding.bind(messageListViewModel, binding.messageListView, this);
         MessageInputViewModelBinding.bind(messageInputViewModel, binding.messageInputView, this);
 
-        // Step 3 - Let both message list header and message input know when we open a thread
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
         messageListViewModel.getMode().observe(this, mode -> {
             if (mode instanceof Thread) {
                 Message parentMessage = ((Thread) mode).getParentMessage();

samplejava/src/main/java/com/example/chattutorial/ChannelActivity2.java

@@ -48,7 +48,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
             throw new IllegalStateException("Specifying a channel id is required when starting ChannelActivity2");
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         MessageListViewModelFactory factory = new MessageListViewModelFactory(cid);
         ViewModelProvider provider = new ViewModelProvider(this, factory);
         MessageListHeaderViewModel messageListHeaderViewModel = provider.get(MessageListHeaderViewModel.class);
@@ -63,7 +64,7 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
         MessageListViewModelBinding.bind(messageListViewModel, binding.messageListView, this);
         MessageInputViewModelBinding.bind(messageInputViewModel, binding.messageInputView, this);
 
-        // Step 3 - Let both message list header and message input know when we open a thread
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
         messageListViewModel.getMode().observe(this, mode -> {
             if (mode instanceof Thread) {
                 Message parentMessage = ((Thread) mode).getParentMessage();

samplejava/src/main/java/com/example/chattutorial/ChannelActivity3.java

@@ -56,7 +56,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
             throw new IllegalStateException("Specifying a channel id is required when starting ChannelActivity3");
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         MessageListViewModelFactory factory = new MessageListViewModelFactory(cid);
         ViewModelProvider provider = new ViewModelProvider(this, factory);
         MessageListHeaderViewModel messageListHeaderViewModel = provider.get(MessageListHeaderViewModel.class);
@@ -71,7 +72,7 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
         MessageListViewModelBinding.bind(messageListViewModel, binding.messageListView, this);
         MessageInputViewModelBinding.bind(messageInputViewModel, binding.messageInputView, this);
 
-        // Step 3 - Let both message list header and message input know when we open a thread
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
         messageListViewModel.getMode().observe(this, mode -> {
             if (mode instanceof Thread) {
                 Message parentMessage = ((Thread) mode).getParentMessage();

samplejava/src/main/java/com/example/chattutorial/ChannelActivity4.java

@@ -57,7 +57,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
             throw new IllegalStateException("Specifying a channel id is required when starting ChannelActivity4");
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         MessageListViewModelFactory factory = new MessageListViewModelFactory(cid);
         ViewModelProvider provider = new ViewModelProvider(this, factory);
         MessageListHeaderViewModel messageListHeaderViewModel = provider.get(MessageListHeaderViewModel.class);

samplejava/src/main/java/com/example/chattutorial/MainActivity.java

@@ -30,7 +30,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
         ActivityMainBinding binding = ActivityMainBinding.inflate(getLayoutInflater());
         setContentView(binding.getRoot());
 
-        // Step 1 - Set up the client for API calls, the domain for offline storage and the UI components
+        // Step 1 - Set up the client for API calls, the domain for offline storage
+        //          and the UI components
         ChatClient client = new ChatClient.Builder("b67pax5b2wdq", getApplicationContext()).build();
         new ChatDomain.Builder(client, getApplicationContext()).build();
         new ChatUI.Builder(getApplicationContext()).build();
@@ -62,7 +63,8 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
         ChannelListViewModel channelsViewModel =
                 new ViewModelProvider(this, factory).get(ChannelListViewModel.class);
 
-        // Step 4 - Connect the ChannelListViewModel to the ChannelListView, loose coupling makes it easy to customize
+        // Step 4 - Connect the ChannelListViewModel to the ChannelListView, loose
+        //          coupling makes it easy to customize
         ChannelListViewModelBinding.bind(channelsViewModel, binding.channelListView, this);
         binding.channelListView.setChannelItemClickListener(channel -> {
             startActivity(ChannelActivity4.newIntent(this, channel));

samplekotlin/src/main/java/com/example/chattutorial/ChannelActivity.kt

@@ -34,7 +34,8 @@ class ChannelActivity : AppCompatActivity() {
             "Specifying a channel id is required when starting ChannelActivity"
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         val factory = MessageListViewModelFactory(cid)
         val messageListHeaderViewModel: MessageListHeaderViewModel by viewModels { factory }
         val messageListViewModel: MessageListViewModel by viewModels { factory }
@@ -47,8 +48,8 @@ class ChannelActivity : AppCompatActivity() {
         messageListViewModel.bindView(binding.messageListView, this)
         messageInputViewModel.bindView(binding.messageInputView, this)
 
-        // Step 3 - Let both message list header and message input know when we open a thread
-        // Note: the .observe support was added in kotlin 1.4, upgrade kotlin to support this syntax
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
+        // Note: the observe syntax used here requires Kotlin 1.4
         messageListViewModel.mode.observe(this) { mode ->
             when (mode) {
                 is Thread -> {

samplekotlin/src/main/java/com/example/chattutorial/ChannelActivity2.kt

@@ -34,7 +34,8 @@ class ChannelActivity2 : AppCompatActivity() {
             "Specifying a channel id is required when starting ChannelActivity2"
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         val factory = MessageListViewModelFactory(cid)
         val messageListHeaderViewModel: MessageListHeaderViewModel by viewModels { factory }
         val messageListViewModel: MessageListViewModel by viewModels { factory }
@@ -48,8 +49,8 @@ class ChannelActivity2 : AppCompatActivity() {
         messageListViewModel.bindView(binding.messageListView, this)
         messageInputViewModel.bindView(binding.messageInputView, this)
 
-        // Step 3 - Let both message list header and message input know when we open a thread
-        // Note: the .observe support was added in kotlin 1.4, upgrade kotlin to support this syntax
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
+        // Note: the observe syntax used here requires Kotlin 1.4
         messageListViewModel.mode.observe(this) { mode ->
             when (mode) {
                 is Thread -> {

samplekotlin/src/main/java/com/example/chattutorial/ChannelActivity3.kt

@@ -36,7 +36,8 @@ class ChannelActivity3 : AppCompatActivity() {
             "Specifying a channel id is required when starting ChannelActivity3"
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         val factory = MessageListViewModelFactory(cid)
         val messageListHeaderViewModel: MessageListHeaderViewModel by viewModels { factory }
         val messageListViewModel: MessageListViewModel by viewModels { factory }
@@ -50,8 +51,8 @@ class ChannelActivity3 : AppCompatActivity() {
         messageListViewModel.bindView(binding.messageListView, this)
         messageInputViewModel.bindView(binding.messageInputView, this)
 
-        // Step 3 - Let both message list header and message input know when we open a thread
-        // Note: the .observe support was added in kotlin 1.4, upgrade kotlin to support this syntax
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
+        // Note: the observe syntax used here requires Kotlin 1.4
         messageListViewModel.mode.observe(this) { mode ->
             when (mode) {
                 is Thread -> {

samplekotlin/src/main/java/com/example/chattutorial/ChannelActivity4.kt

@@ -39,7 +39,8 @@ class ChannelActivity4 : AppCompatActivity() {
             "Specifying a channel id is required when starting ChannelActivity"
         }
 
-        // Step 1 - Create 3 separate ViewModels for the views so it's easy to customize one of the components
+        // Step 1 - Create three separate ViewModels for the views so it's easy
+        //          to customize them individually
         val factory = MessageListViewModelFactory(cid)
         val messageListHeaderViewModel: MessageListHeaderViewModel by viewModels { factory }
         val messageListViewModel: MessageListViewModel by viewModels { factory }
@@ -53,8 +54,8 @@ class ChannelActivity4 : AppCompatActivity() {
         messageListViewModel.bindView(binding.messageListView, this)
         messageInputViewModel.bindView(binding.messageInputView, this)
 
-        // Step 3 - Let both message list header and message input know when we open a thread
-        // Note: the .observe support was added in kotlin 1.4, upgrade kotlin to support this syntax
+        // Step 3 - Let both MessageListHeaderView and MessageInputView know when we open a thread
+        // Note: the observe syntax used here requires Kotlin 1.4
         messageListViewModel.mode.observe(this) { mode ->
             when (mode) {
                 is Thread -> {

samplekotlin/src/main/java/com/example/chattutorial/MainActivity.kt

@@ -25,7 +25,8 @@ class MainActivity : AppCompatActivity() {
         binding = ActivityMainBinding.inflate(layoutInflater)
         setContentView(binding.root)
 
-        // Step 1 - Set up the client for API calls, the domain for offline storage and the UI components
+        // Step 1 - Set up the client for API calls, the domain for offline storage
+        //          and the UI components
         val client = ChatClient.Builder("b67pax5b2wdq", applicationContext).build()
         ChatDomain.Builder(client, applicationContext).build()
         ChatUI.Builder(applicationContext).build()
@@ -53,7 +54,9 @@ class MainActivity : AppCompatActivity() {
         val viewModelFactory = ChannelListViewModelFactory(filter, ChannelListViewModel.DEFAULT_SORT)
         val viewModel: ChannelListViewModel by viewModels { viewModelFactory }
 
-        // Step 4 - Connect the ChannelListViewModel to the ChannelListView, loose coupling makes it easy to customize
+        // Step 4 - Connect the ChannelListViewModel to the ChannelListView, loose
+        //          coupling makes it easy to customize
+        // Note: the listener syntax used here requires Kotlin 1.4
         viewModel.bindView(binding.channelListView, this)
         binding.channelListView.setChannelItemClickListener { channel ->
             startActivity(ChannelActivity4.newIntent(this, channel))