Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Commit

Permalink
Fix #8566: Remove ScreenTime temporarily. (#8568)
Browse files Browse the repository at this point in the history
Seems to cause UI freezes.
  • Loading branch information
iccub committed Dec 15, 2023
1 parent fd6b34b commit d6d0e17
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 @@ -371,7 +371,8 @@ public class BrowserViewController: UIViewController {
}

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

Expand Down Expand Up @@ -3244,7 +3245,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 d6d0e17

Please sign in to comment.