Merge pull request #397 from SourcePointUSA/DIA-1566_single_button_focus_for_ccpa

andresilveirah · web-flow · commit 245796443ab3 · 2022-12-09T13:33:42.000+01:00
[DIA-1566] Now if CCPA message has less than 2 buttons focus won't be set
diff --git a/ConsentViewController/Classes/Views/tvOS/NativePrivacyManager/CCPA/SPCCPANativePrivacyManagerViewController.swift b/ConsentViewController/Classes/Views/tvOS/NativePrivacyManager/CCPA/SPCCPANativePrivacyManagerViewController.swift
@@ -95,6 +95,9 @@ import Foundation
 
     func setFocusGuidesForButtons() {
         let visibleButtons: [UIView] = actionsContainer.arrangedSubviews.filter({!$0.isHidden})
+        if(visibleButtons.count <= 1) {
+            return;
+        }
         for i in 0...visibleButtons.count-2 {
             addFocusGuide(from: visibleButtons[i], to: visibleButtons[i+1], direction: .bottomTop)
         }

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,9 @@ import Foundation`
`95`	`95`
`96`	`96`	`func setFocusGuidesForButtons() {`
`97`	`97`	`let visibleButtons: [UIView] = actionsContainer.arrangedSubviews.filter({!$0.isHidden})`
	`98`	`+ if(visibleButtons.count <= 1) {`
	`99`	`+ return;`
	`100`	`+ }`
`98`	`101`	`for i in 0...visibleButtons.count-2 {`
`99`	`102`	`addFocusGuide(from: visibleButtons[i], to: visibleButtons[i+1], direction: .bottomTop)`
`100`	`103`	`}`