diff --git a/app/src/main/java/com/morlunk/mumbleclient/channel/ChannelListAdapter.java b/app/src/main/java/com/morlunk/mumbleclient/channel/ChannelListAdapter.java index bf649af4..7459266f 100644 --- a/app/src/main/java/com/morlunk/mumbleclient/channel/ChannelListAdapter.java +++ b/app/src/main/java/com/morlunk/mumbleclient/channel/ChannelListAdapter.java @@ -124,14 +124,6 @@ public void onBindViewHolder(RecyclerView.ViewHolder viewHolder, int position) { if (node.isChannel()) { final IChannel channel = node.getChannel(); final ChannelViewHolder cvh = (ChannelViewHolder) viewHolder; - cvh.itemView.setOnClickListener(new View.OnClickListener() { - @Override - public void onClick(View v) { - if (mChannelClickListener != null) { - mChannelClickListener.onChannelClick(channel); - } - } - }); final boolean expandUsable = channel.getSubchannels().size() > 0 || channel.getSubchannelUserCount() > 0; @@ -185,11 +177,14 @@ public void onClick(View v) { cvh.mChannelHolder.getPaddingRight(), cvh.mChannelHolder.getPaddingBottom()); - cvh.mJoinButton.setOnClickListener(new View.OnClickListener() { + // Actions. + + cvh.mChatChannelButton.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { - if (mService.isConnected()) - mService.getSession().joinChannel(channel.getId()); + if (mChannelClickListener != null) { + mChannelClickListener.onChannelClick(channel); + } } }); @@ -201,9 +196,18 @@ public void onClick(View v) { } }); + cvh.itemView.setOnClickListener(new View.OnClickListener() { + @Override + public void onClick(View v) { + if (mService.isConnected()) + mService.getSession().joinChannel(channel.getId()); + } + + }); cvh.itemView.setOnLongClickListener(new View.OnLongClickListener() { @Override public boolean onLongClick(View v) { + //ToDo: olzzon Long Click join Channel - Setting Secure Channel Shift cvh.mMoreButton.performClick(); return true; } @@ -516,7 +520,7 @@ private static class ChannelViewHolder extends RecyclerView.ViewHolder { public ImageView mChannelExpandToggle; public TextView mChannelName; public TextView mChannelUserCount; - public ImageView mJoinButton; + public ImageView mChatChannelButton; public ImageView mMoreButton; public ChannelViewHolder(View itemView) { @@ -525,7 +529,7 @@ public ChannelViewHolder(View itemView) { mChannelExpandToggle = (ImageView) itemView.findViewById(R.id.channel_row_expand); mChannelName = (TextView) itemView.findViewById(R.id.channel_row_name); mChannelUserCount = (TextView) itemView.findViewById(R.id.channel_row_count); - mJoinButton = (ImageView) itemView.findViewById(R.id.channel_row_join); + mChatChannelButton = (ImageView) itemView.findViewById(R.id.channel_row_join); mMoreButton = (ImageView) itemView.findViewById(R.id.channel_row_more); } } diff --git a/app/src/main/res/layout/channel_row.xml b/app/src/main/res/layout/channel_row.xml index 8b6fad49..55c00392 100644 --- a/app/src/main/res/layout/channel_row.xml +++ b/app/src/main/res/layout/channel_row.xml @@ -41,24 +41,27 @@ android:gravity="center_vertical|left" android:minHeight="48dp" android:padding="4dp" - android:text="Channel name\nLine 2\nLine 3\nLine 4"/> + android:text="Channel name\nLine 2\nLine 3\nLine 4" + android:textAppearance="@style/TextAppearance.AppCompat.Body2" + android:textSize="18sp" + /> + android:textStyle="bold" /> + android:background="?attr/selectableItemBackgroundBorderless" + android:src="@android:drawable/sym_action_chat" + android:tint="?android:attr/textColorSecondary" />