Skip to content

Commit

Permalink
Fix brave/brave-ios#8566: Remove ScreenTime temporarily. (brave/brave…
Browse files Browse the repository at this point in the history
…-ios#8568)

Seems to cause UI freezes.
  • Loading branch information
iccub authored Dec 15, 2023
1 parent b107a8a commit 7521d6a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Sources/Brave/Frontend/Browser/BrowserViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -378,7 +378,8 @@ public class BrowserViewController: UIViewController {
}

if Preferences.Privacy.screenTimeEnabled.value {
screenTimeViewController = STWebpageController()
// Enable once fixed, ref #8566
//screenTimeViewController = STWebpageController()
}
}

Expand Down Expand Up @@ -3302,7 +3303,8 @@ extension BrowserViewController: PreferencesObserver {
recordAdsUsageType()
case Preferences.Privacy.screenTimeEnabled.key:
if Preferences.Privacy.screenTimeEnabled.value {
screenTimeViewController = STWebpageController()
// Enable once fixed, ref #8566
//screenTimeViewController = STWebpageController()
if let tab = tabManager.selectedTab {
recordScreenTimeUsage(for: tab)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,11 +69,14 @@ struct OtherPrivacySettingsSectionView: View {
subtitle: String.localizedStringWithFormat(Strings.googleSafeBrowsingUsingWebKitDescription, URL.brave.safeBrowsingHelp.absoluteString),
option: Preferences.Shields.googleSafeBrowsing
)
// Enable once fixed, ref #8566
/*
OptionToggleView(
title: Strings.screenTimeSetting,
subtitle: String.localizedStringWithFormat(Strings.screenTimeSettingDescription, URL.brave.screenTimeHelp.absoluteString),
option: Preferences.Privacy.screenTimeEnabled
)
*/
ShieldToggleView(
title: Strings.P3A.settingTitle,
subtitle: Strings.P3A.settingSubtitle,
Expand Down

0 comments on commit 7521d6a

Please sign in to comment.