Skip to content

Commit

Permalink
DuckPlayer: 31. Native PoC - Limit to iPhone (#3925)
Browse files Browse the repository at this point in the history
Task/Issue URL: https://app.asana.com/0/0/1209317514439765/f
Tech Design URL:
CC:

**Description**:
- Limit the PoC to iPhone
  • Loading branch information
afterxleep authored Feb 4, 2025
1 parent 71a09c3 commit 492b6cc
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions DuckDuckGo/DuckPlayer/DuckPlayerSettings.swift
Original file line number Diff line number Diff line change
Expand Up @@ -191,12 +191,12 @@ final class DuckPlayerSettingsDefault: DuckPlayerSettings {

// Determines if we should use the native verion of DuckPlayer (Internal only)
var nativeUI: Bool {
return appSettings.duckPlayerNativeUI && internalUserDecider.isInternalUser
return appSettings.duckPlayerNativeUI && internalUserDecider.isInternalUser && UIDevice.current.userInterfaceIdiom == .phone
}

// Determines if we should use the native verion of DuckPlayer (Internal only)
var autoplay: Bool {
return appSettings.duckPlayerAutoplay && internalUserDecider.isInternalUser
return appSettings.duckPlayerAutoplay && internalUserDecider.isInternalUser && UIDevice.current.userInterfaceIdiom == .phone
}

/// Registers a publisher to listen for changes in the privacy configuration.
Expand Down
2 changes: 1 addition & 1 deletion DuckDuckGo/SettingsDuckPlayerView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ struct SettingsDuckPlayerView: View {
}

/// Experimental features for internal users
if viewModel.isInternalUser {
if viewModel.isInternalUser && UIDevice.current.userInterfaceIdiom == .phone {
Section("Experimental (Internal only)", content: {
SettingsCellView(label: "Use Native UI (Alpha)", accessory: .toggle(isOn: viewModel.duckPlayerNativeUI))
if viewModel.appSettings.duckPlayerNativeUI {
Expand Down

0 comments on commit 492b6cc

Please sign in to comment.