Test feedback (#30)

First little part:
- tappable area
- avatar padding
- simulator warning

Author: pblazej

README.md

@@ -61,6 +61,10 @@ If your agent publishes a [virtual avatar](https://docs.livekit.io/agents/integr
 
 In a production environment, you will be responsible for developing a solution to [generate tokens for your users](https://docs.livekit.io/home/server/generating-tokens/) which is integrated with your authentication solution. You should disable your sandbox token server and modify `TokenService.swift` to use your own token server.
 
+## Running on Simulator
+
+To use this template with video (or screen sharing) input, you need to run the app on a physical device. Testing on the Simulator will still support voice and text modes, as well as virtual avatars.
+
 ## Submitting to the App Store
 
 `LiveKitWebRTC.xcframework` binary framework, which is part of the LiveKit Swift SDK, does not contain DSYMs. Submitting the app to the App Store will result in a following warning:

VoiceAgent/ControlBar/ControlBar.swift

@@ -77,7 +77,7 @@ struct ControlBar: View {
 
     @ViewBuilder
     private func audioControls() -> some View {
-        HStack(spacing: 2 * .grid) {
+        HStack(spacing: .zero) {
             Spacer()
             AsyncButton(action: viewModel.toggleMicrophone) {
                 HStack(spacing: .grid) {
@@ -89,6 +89,8 @@ struct ControlBar: View {
                         .id(viewModel.audioTrack?.id)
                 }
                 .frame(height: Constants.buttonHeight)
+                .padding(.horizontal, 2 * .grid)
+                .contentShape(Rectangle())
             }
             #if os(macOS)
             separator()
@@ -102,12 +104,14 @@ struct ControlBar: View {
 
     @ViewBuilder
     private func videoControls() -> some View {
-        HStack(spacing: 2 * .grid) {
+        HStack(spacing: .zero) {
             Spacer()
             AsyncButton(action: viewModel.toggleCamera) {
                 Image(systemName: viewModel.isCameraEnabled ? "video.fill" : "video.slash.fill")
                     .transition(.symbolEffect)
                     .frame(height: Constants.buttonHeight)
+                    .padding(.horizontal, 2 * .grid)
+                    .contentShape(Rectangle())
             }
             #if os(macOS)
             separator()
@@ -125,6 +129,7 @@ struct ControlBar: View {
         AsyncButton(action: viewModel.toggleScreenShare) {
             Image(systemName: "arrow.up.square.fill")
                 .frame(width: Constants.buttonWidth, height: Constants.buttonHeight)
+                .contentShape(Rectangle())
         }
         .buttonStyle(
             ControlBarButtonStyle(
@@ -142,6 +147,7 @@ struct ControlBar: View {
         AsyncButton(action: viewModel.toggleTextInput) {
             Image(systemName: "ellipsis.message.fill")
                 .frame(width: Constants.buttonWidth, height: Constants.buttonHeight)
+                .contentShape(Rectangle())
         }
         .buttonStyle(
             ControlBarButtonStyle(
@@ -159,6 +165,7 @@ struct ControlBar: View {
         AsyncButton(action: viewModel.disconnect) {
             Image(systemName: "phone.down.fill")
                 .frame(width: Constants.buttonWidth, height: Constants.buttonHeight)
+                .contentShape(Rectangle())
         }
         .buttonStyle(
             ControlBarButtonStyle(

VoiceAgent/ControlBar/Devices/AudioDeviceSelector.swift

@@ -23,6 +23,7 @@ struct AudioDeviceSelector: View {
             Image(systemName: "chevron.down")
                 .frame(height: 11 * .grid)
                 .font(.system(size: 12, weight: .semibold))
+                .contentShape(Rectangle())
         }
     }
 }

VoiceAgent/ControlBar/Devices/VideoDeviceSelector.swift

@@ -24,6 +24,7 @@ struct VideoDeviceSelector: View {
             Image(systemName: "chevron.down")
                 .frame(height: 11 * .grid)
                 .font(.system(size: 12, weight: .semibold))
+                .contentShape(Rectangle())
         }
     }
 }

VoiceAgent/Localizable.xcstrings

@@ -29,7 +29,7 @@
         "en" : {
           "stringUnit" : {
             "state" : "translated",
-            "value" : "Running on Simulator, functionality may be limited."
+            "value" : "Running on Simulator, camera and screen sharing will be unavailable."
           }
         }
       }

VoiceAgent/Participant/AgentParticipantView.swift

@@ -16,6 +16,7 @@ struct AgentParticipantView: View {
                 SwiftUIVideoView(avatarCameraTrack)
                     .clipShape(RoundedRectangle(cornerRadius: .cornerRadiusPerPlatform))
                     .aspectRatio(avatarCameraTrack.aspectRatio, contentMode: .fit)
+                    .padding(.horizontal, avatarCameraTrack.aspectRatio == 1 ? 4 * .grid : .zero)
                     .shadow(radius: 20, y: 10)
                     .mask(
                         GeometryReader { proxy in