Fix modlog filter semantic label and instance explorer community subs…

…cription button (#1191)
thunder-app · Mar 11, 2024 · 411994c · 411994c
1 parent d9275a5
commit 411994c
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 24 deletions.
diff --git a/lib/community/widgets/community_list_entry.dart b/lib/community/widgets/community_list_entry.dart
@@ -70,29 +70,31 @@ class CommunityListEntry extends StatelessWidget {
             Icon(Icons.star_rounded, size: 15),
           ]
         ]),
-        trailing: IconButton(
-          onPressed: () {
-            SubscribedType? subscriptionStatus = getCurrentSubscriptionStatus?.call(isUserLoggedIn, communityView, currentSubscriptions);
-            onSubscribeIconPressed?.call(isUserLoggedIn, context, communityView);
-            showSnackbar(subscriptionStatus == SubscribedType.notSubscribed ? l10n.addedCommunityToSubscriptions : l10n.removedCommunityFromSubscriptions);
-            context.read<AccountBloc>().add(GetAccountSubscriptions());
-          },
-          icon: Icon(
-            switch (getCurrentSubscriptionStatus?.call(isUserLoggedIn, communityView, currentSubscriptions)) {
-              SubscribedType.notSubscribed => Icons.add_circle_outline_rounded,
-              SubscribedType.pending => Icons.pending_outlined,
-              SubscribedType.subscribed => Icons.remove_circle_outline_rounded,
-              _ => null,
-            },
-          ),
-          tooltip: switch (getCurrentSubscriptionStatus?.call(isUserLoggedIn, communityView, currentSubscriptions)) {
-            SubscribedType.notSubscribed => l10n.subscribe,
-            SubscribedType.pending => l10n.unsubscribePending,
-            SubscribedType.subscribed => l10n.unsubscribe,
-            _ => null,
-          },
-          visualDensity: VisualDensity.compact,
-        ),
+        trailing: getCurrentSubscriptionStatus == null
+            ? null
+            : IconButton(
+                onPressed: () {
+                  SubscribedType? subscriptionStatus = getCurrentSubscriptionStatus!(isUserLoggedIn, communityView, currentSubscriptions);
+                  onSubscribeIconPressed?.call(isUserLoggedIn, context, communityView);
+                  showSnackbar(subscriptionStatus == SubscribedType.notSubscribed ? l10n.addedCommunityToSubscriptions : l10n.removedCommunityFromSubscriptions);
+                  context.read<AccountBloc>().add(GetAccountSubscriptions());
+                },
+                icon: Icon(
+                  switch (getCurrentSubscriptionStatus!(isUserLoggedIn, communityView, currentSubscriptions)) {
+                    SubscribedType.notSubscribed => Icons.add_circle_outline_rounded,
+                    SubscribedType.pending => Icons.pending_outlined,
+                    SubscribedType.subscribed => Icons.remove_circle_outline_rounded,
+                    _ => null,
+                  },
+                ),
+                tooltip: switch (getCurrentSubscriptionStatus!(isUserLoggedIn, communityView, currentSubscriptions)) {
+                  SubscribedType.notSubscribed => l10n.subscribe,
+                  SubscribedType.pending => l10n.unsubscribePending,
+                  SubscribedType.subscribed => l10n.unsubscribe,
+                  _ => null,
+                },
+                visualDensity: VisualDensity.compact,
+              ),
         onTap: () async {
           int? communityId = communityView.community.id;
           if (resolutionInstance != null) {

diff --git a/lib/modlog/widgets/modlog_feed_page_app_bar.dart b/lib/modlog/widgets/modlog_feed_page_app_bar.dart
@@ -48,7 +48,10 @@ class ModlogFeedPageAppBar extends StatelessWidget {
       ),
       actions: <Widget>[
         IconButton(
-          icon: const Icon(Icons.filter_alt_rounded),
+          icon: Icon(
+            Icons.filter_alt_rounded,
+            semanticLabel: l10n.filters,
+          ),
           onPressed: () {
             HapticFeedback.mediumImpact();