Fix #2682: NTP learn more secondary button actions. (#2687)

brave · Jul 2, 2020 · ce39001 · ce39001
1 parent 4d9a1c2
commit ce39001
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/Client/Frontend/Browser/New Tab Page/Notifications/NTPLearnMoreContentView.swift b/Client/Frontend/Browser/New Tab Page/Notifications/NTPLearnMoreContentView.swift
@@ -134,12 +134,13 @@ extension NTPLearnMoreViewController {
                 .forEach(stackView.addArrangedSubview(_:))
         }
 
-        private let learnMoreButton = secondaryButton(title: nil).then {
-            $0.addTarget(self, action: #selector(learnMoreButtonAction), for: .touchDown)
+        private lazy var learnMoreButton = secondaryButton(title: nil).then {
+            $0.addTarget(self, action: #selector(learnMoreButtonAction), for: .touchUpInside)
         }
 
-        private let hideSponsoredImageButton = secondaryButton(title: Strings.NTP.hideSponsoredImages).then {
-          $0.addTarget(self, action: #selector(hideSponsoredImagesAction), for: .touchDown)
+        private lazy var hideSponsoredImageButton =
+            secondaryButton(title: Strings.NTP.hideSponsoredImages).then {
+          $0.addTarget(self, action: #selector(hideSponsoredImagesAction), for: .touchUpInside)
         }
 
         private let config: NTPNotificationLearnMoreViewConfig