letro/letro-ios
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases 2 Wiki Activity

Combine DM call actions into one menu item

Browse Source 
This helps give more space to the room header. Specially in the scenario when threads is enabled.
This commit is contained in:
amshak
2026-04-23 11:31:15 +01:00
committed by Stefan Ceriu
parent ce50e4c5f8
commit eb1cfb68d5
1 changed files with 25 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
ElementX/Sources/Screens/RoomScreen/View/RoomCallControlsToolbar.swift
View File

@@ -24,21 +24,38 @@ struct RoomCallControlsToolbar: ToolbarContent {
} else {
if viewState.isDirectOneToOneRoom {
ToolbarItem(placement: .primaryAction) {
Button { onCallTap(true) } label: {
Menu {
Button(action: { onCallTap(true) }) {
Label("Audio call", systemImage: "phone.fill")
}
.accessibilityIdentifier(A11yIdentifiers.roomScreen.startVoiceCall)
.disabled(!viewState.canJoinCall)
Button(action: { onCallTap(false) }) {
Label("Video call", systemImage: "video.fill")
}
.accessibilityIdentifier(A11yIdentifiers.roomScreen.startVideoCall)
.disabled(!viewState.canJoinCall)
} label: {
CompoundIcon(\.voiceCallSolid)
.frame(width: 28, height: 28)
.contentShape(Rectangle())
}
.menuIndicator(.hidden)
.buttonStyle(.plain)
.accessibilityLabel(L10n.a11yStartVoiceCall)
.accessibilityIdentifier(A11yIdentifiers.roomScreen.startVoiceCall)
.disabled(!viewState.canJoinCall)
}
}
ToolbarItem(placement: .primaryAction) {
Button { onCallTap(false) } label: {
CompoundIcon(\.videoCallSolid)
} else {
ToolbarItem(placement: .primaryAction) {
Button { onCallTap(false) } label: {
CompoundIcon(\.videoCallSolid)
}
.accessibilityLabel(L10n.a11yStartVideoCall)
.accessibilityIdentifier(A11yIdentifiers.roomScreen.startVideoCall)
.disabled(!viewState.canJoinCall)
}
.accessibilityLabel(L10n.a11yStartVideoCall)
.accessibilityIdentifier(A11yIdentifiers.roomScreen.startVideoCall)
.disabled(!viewState.canJoinCall)
}
}
}