Merge branch 'status-menu' into 'develop'

Don't show 'Bookmark' item menu if not authenticated

See merge request soapbox-pub/soapbox-fe!1033

app/soapbox/components/status_action_bar.js

@@ -342,6 +342,10 @@ class StatusActionBar extends ImmutablePureComponent {
       // });
     }
 
+    if (!me) {
+      return menu;
+    }
+
     if (features.bookmarks) {
       menu.push({
         text: intl.formatMessage(status.get('bookmarked') ? messages.unbookmark : messages.bookmark),
@@ -350,10 +354,6 @@ class StatusActionBar extends ImmutablePureComponent {
       });
     }
 
-    if (!me) {
-      return menu;
-    }
-
     menu.push(null);
 
     if (ownAccount || withDismiss) {

app/soapbox/features/status/components/action_bar.js

@@ -345,15 +345,15 @@ class ActionBar extends React.PureComponent {
       // });
     }
 
-    if (features.bookmarks) {
-      menu.push({
-        text: intl.formatMessage(status.get('bookmarked') ? messages.unbookmark : messages.bookmark),
-        action: this.handleBookmarkClick,
-        icon: require(status.get('bookmarked') ? '@tabler/icons/icons/bookmark-off.svg' : '@tabler/icons/icons/bookmark.svg'),
-      });
-    }
-
     if (me) {
+      if (features.bookmarks) {
+        menu.push({
+          text: intl.formatMessage(status.get('bookmarked') ? messages.unbookmark : messages.bookmark),
+          action: this.handleBookmarkClick,
+          icon: require(status.get('bookmarked') ? '@tabler/icons/icons/bookmark-off.svg' : '@tabler/icons/icons/bookmark.svg'),
+        });
+      }
+
       menu.push(null);
 
       if (ownAccount) {