Skip to content

Commit 1529b24

Browse files
committed
chore(fc): make spectator footer a consistent unit when room is closed
Signed-off-by: Brandon McAnsh <[email protected]>
1 parent 374f788 commit 1529b24

File tree

2 files changed

+13
-12
lines changed

2 files changed

+13
-12
lines changed

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationChatInput.kt

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ import com.getcode.theme.CodeTheme
3434
import com.getcode.ui.components.chat.ChatInput
3535
import com.getcode.ui.theme.ButtonState
3636
import com.getcode.ui.theme.CodeButton
37+
import com.getcode.ui.utils.addIf
3738
import com.getcode.ui.utils.keyboardAsState
3839
import com.getcode.ui.utils.withTopBorder
3940
import com.getcode.util.resources.LocalResources
@@ -109,17 +110,21 @@ fun ConversationChatInput(
109110
}
110111

111112
is ChattableState.Spectator -> {
112-
Column {
113+
Column(
114+
modifier = Modifier.addIf(
115+
!state.isRoomOpen && state.isOpenCloseEnabled
116+
) {
117+
Modifier.background(CodeTheme.colors.secondary)
118+
}.navigationBarsPadding(),
119+
) {
113120
if (!state.isRoomOpen && state.isOpenCloseEnabled) {
114121
Text(
115122
modifier = Modifier
116123
.fillMaxWidth()
117-
.background(CodeTheme.colors.secondary)
118124
.padding(
119125
top = CodeTheme.dimens.grid.x1,
120-
bottom = CodeTheme.dimens.grid.x3
121-
)
122-
.navigationBarsPadding(),
126+
bottom = CodeTheme.dimens.grid.x2
127+
),
123128
textAlign = TextAlign.Center,
124129
text = stringResource(R.string.title_roomIsClosed),
125130
style = CodeTheme.typography.textSmall,
@@ -133,8 +138,7 @@ fun ConversationChatInput(
133138
.padding(
134139
start = CodeTheme.dimens.inset,
135140
end = CodeTheme.dimens.inset
136-
)
137-
.navigationBarsPadding(),
141+
),
138142
buttonState = ButtonState.Filled,
139143
text = stringResource(
140144
R.string.action_joinRoomWithCost,

flipchatApp/src/main/kotlin/xyz/flipchat/app/features/chat/conversation/ConversationViewModel.kt

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,7 @@ import com.getcode.model.chat.MessageContent
2020
import com.getcode.model.chat.MessageStatus
2121
import com.getcode.model.chat.Sender
2222
import com.getcode.model.uuid
23-
import com.getcode.models.ConfirmationState
2423
import com.getcode.navigation.RoomInfoArgs
25-
import com.getcode.network.BalanceController
26-
import com.getcode.network.BalanceDisplay
2724
import com.getcode.services.model.ExtendedMetadata
2825
import com.getcode.ui.components.chat.messagecontents.MessageControlAction
2926
import com.getcode.ui.components.chat.messagecontents.MessageControls
@@ -306,13 +303,13 @@ class ConversationViewModel @Inject constructor(
306303
val chattableState = if (selfMember != null) {
307304
val isMuted = selfMember.isMuted
308305
val isSpectator = !selfMember.isFullMember
309-
val isRoomClosed = !conversation.isOpen && !selfMember.isHost
306+
val isRoomClosedAsMember = !conversation.isOpen && !selfMember.isHost
310307

311308
when {
312309
isSpectator -> ChattableState.Spectator(
313310
Kin.fromQuarks(it.conversation.coverChargeQuarks ?: 0)
314311
)
315-
isRoomClosed -> ChattableState.DisabledByClosedRoom
312+
isRoomClosedAsMember -> ChattableState.DisabledByClosedRoom
316313
isMuted -> ChattableState.DisabledByMute
317314
else -> ChattableState.Enabled
318315
}

0 commit comments

Comments
 (0)