From 015ee0a8e02718e57fb1f4736365ad6cd5fd2faa Mon Sep 17 00:00:00 2001 From: Stuart Breckenridge Date: Thu, 19 Nov 2020 12:02:08 +0800 Subject: [PATCH 1/3] fixes button state on account help view --- .../Accounts/AddAccountHelpView.swift | 31 +++++++------------ 1 file changed, 11 insertions(+), 20 deletions(-) diff --git a/Mac/Preferences/Accounts/AddAccountHelpView.swift b/Mac/Preferences/Accounts/AddAccountHelpView.swift index 373fe6edb..48dbab52d 100644 --- a/Mac/Preferences/Accounts/AddAccountHelpView.swift +++ b/Mac/Preferences/Accounts/AddAccountHelpView.swift @@ -14,33 +14,24 @@ struct AddAccountHelpView: View { let accountTypes: [AccountType] = AddAccountSections.allOrdered.sectionContent var delegate: AccountsPreferencesAddAccountDelegate? var helpText: String - @State private var hoveringId: String? = nil @State private var iCloudUnavailableError: Bool = false var body: some View { VStack { HStack { ForEach(accountTypes, id: \.self) { account in - account.image() - .resizable() - .frame(width: 20, height: 20, alignment: .center) - .onTapGesture { - if account == .cloudKit && AccountManager.shared.accounts.contains(where: { $0.type == .cloudKit }) { - iCloudUnavailableError = true - } else { - delegate?.presentSheetForAccount(account) - } - hoveringId = nil + Button(action: { + if account == .cloudKit && AccountManager.shared.accounts.contains(where: { $0.type == .cloudKit }) { + iCloudUnavailableError = true + } else { + delegate?.presentSheetForAccount(account) } - .onHover(perform: { hovering in - if hovering { - hoveringId = account.localizedAccountName() - } else { - hoveringId = nil - } - }) - .scaleEffect(hoveringId == account.localizedAccountName() ? 1.2 : 1) - .shadow(radius: hoveringId == account.localizedAccountName() ? 0.8 : 0) + }, label: { + account.image() + .resizable() + .frame(width: 20, height: 20, alignment: .center) + }) + .buttonStyle(PlainButtonStyle()) } } From f4fe19ae90edaab3170aea0be252fc9834471d64 Mon Sep 17 00:00:00 2001 From: Stuart Breckenridge Date: Thu, 19 Nov 2020 12:04:43 +0800 Subject: [PATCH 2/3] Fixes button state on Extension Help View --- .../EnableExtensionPointHelpView.swift | 25 +++++++------------ 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift b/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift index a9f9f24de..e4b44473a 100644 --- a/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift +++ b/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift @@ -26,22 +26,15 @@ struct EnableExtensionPointHelpView: View { VStack { HStack { ForEach(0.. Date: Thu, 19 Nov 2020 12:05:20 +0800 Subject: [PATCH 3/3] removes hovering id --- .../ExtensionPoints/EnableExtensionPointHelpView.swift | 3 --- 1 file changed, 3 deletions(-) diff --git a/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift b/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift index e4b44473a..58595877f 100644 --- a/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift +++ b/Mac/Preferences/ExtensionPoints/EnableExtensionPointHelpView.swift @@ -20,15 +20,12 @@ struct EnableExtensionPointHelpView: View { var helpText: String weak var preferencesController: ExtensionPointPreferencesViewController? - @State private var hoveringId: String? - var body: some View { VStack { HStack { ForEach(0..