From c672f7a6ae80f14280ddf47d9cf0b8ba722a08a6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?marcin=20miko=C5=82ajczak?= <git@mkljczk.pl>
Date: Wed, 9 Feb 2022 23:11:45 +0100
Subject: [PATCH] Don't show 'Bookmark' item menu if not authenticated
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: marcin mikołajczak <git@mkljczk.pl>
---
 app/soapbox/components/status_action_bar.js      |  8 ++++----
 .../features/status/components/action_bar.js     | 16 ++++++++--------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/app/soapbox/components/status_action_bar.js b/app/soapbox/components/status_action_bar.js
index 429b092ca..79018e680 100644
--- a/app/soapbox/components/status_action_bar.js
+++ b/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) {
diff --git a/app/soapbox/features/status/components/action_bar.js b/app/soapbox/features/status/components/action_bar.js
index b34711632..967702a1a 100644
--- a/app/soapbox/features/status/components/action_bar.js
+++ b/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) {