Merge pull request #5891 from woocommerce/issue/5887--highlighted-state-of-the-settings-button-in-nav-bar-in-the-hub-menu

Hub Menu: highlighted state of the settings button

Author: pmusolino

WooCommerce/Classes/ViewRelated/Hub Menu/HubMenu.swift

@@ -116,22 +116,23 @@ struct HubMenu: View {
                 }
                 Spacer()
                 VStack {
-                    ZStack {
-                        Circle()
-                            .fill(Color(UIColor(light: .white,
-                                                dark: .secondaryButtonBackground)))
-                            .frame(width: settingsSize,
-                                   height: settingsSize)
-                        if let cogImage = UIImage.cogImage.imageWithTintColor(.accent) {
-                            Image(uiImage: cogImage)
-                                .resizable()
-                                .frame(width: settingsIconSize,
-                                       height: settingsIconSize)
-                        }
-                    }
-                    .onTapGesture {
+                    Button {
                         ServiceLocator.analytics.track(.hubMenuSettingsTapped)
                         showSettings = true
+                    } label: {
+                        ZStack {
+                            Circle()
+                                .fill(Color(UIColor(light: .white,
+                                                    dark: .secondaryButtonBackground)))
+                                .frame(width: settingsSize,
+                                       height: settingsSize)
+                            if let cogImage = UIImage.cogImage.imageWithTintColor(.accent) {
+                                Image(uiImage: cogImage)
+                                    .resizable()
+                                    .frame(width: settingsIconSize,
+                                           height: settingsIconSize)
+                            }
+                        }
                     }
                     Spacer()
                 }