From 47d1a8e5d6be49f2365c8da7fdd31e0b311568cc Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Wed, 4 May 2022 12:08:17 -0500
Subject: [PATCH] ActionBar: fix repost active state, fixes #936

---
 app/soapbox/components/status_action_bar.tsx  |  2 +-
 .../features/status/components/action-bar.tsx | 78 ++++++++-----------
 2 files changed, 35 insertions(+), 45 deletions(-)

diff --git a/app/soapbox/components/status_action_bar.tsx b/app/soapbox/components/status_action_bar.tsx
index 3737edfd8..d07fb94c0 100644
--- a/app/soapbox/components/status_action_bar.tsx
+++ b/app/soapbox/components/status_action_bar.tsx
@@ -636,7 +636,7 @@ class StatusActionBar extends ImmutablePureComponent<IStatusActionBar, IStatusAc
           count={replyCount}
         />
 
-        {features.quotePosts && me ? (
+        {(features.quotePosts && me) ? (
           <DropdownMenuContainer
             items={reblogMenu}
             disabled={!publicStatus}
diff --git a/app/soapbox/features/status/components/action-bar.tsx b/app/soapbox/features/status/components/action-bar.tsx
index fff6654b9..0a191b536 100644
--- a/app/soapbox/features/status/components/action-bar.tsx
+++ b/app/soapbox/features/status/components/action-bar.tsx
@@ -522,49 +522,29 @@ class ActionBar extends React.PureComponent<IActionBar, IActionBarState> {
 
     const reblog_disabled = (status.get('visibility') === 'direct' || status.get('visibility') === 'private');
 
-    let reblogButton;
-
-    if (me && features.quotePosts) {
-      const reblogMenu: Menu = [
-        {
-          text: intl.formatMessage(status.get('reblogged') ? messages.cancel_reblog_private : messages.reblog),
-          action: this.handleReblogClick,
-          icon: require('@tabler/icons/icons/repeat.svg'),
-        },
-        {
-          text: intl.formatMessage(messages.quotePost),
-          action: this.handleQuoteClick,
-          icon: require('@tabler/icons/icons/quote.svg'),
-        },
-      ];
-
-      reblogButton = (
-        <DropdownMenuContainer
-          items={reblogMenu}
-          disabled={!publicStatus}
-          active={status.get('reblogged')}
-          pressed={status.get('reblogged')}
-          title={!publicStatus ? intl.formatMessage(messages.cannot_reblog) : intl.formatMessage(messages.reblog)}
-          src={reblogIcon}
-          text={intl.formatMessage(messages.reblog)}
-          onShiftClick={this.handleReblogClick}
-        />
-      );
-    } else {
-      reblogButton = (
-        <IconButton
-          disabled={reblog_disabled}
-          className={classNames({
-            'text-gray-400 hover:text-gray-600': !status.get('reblogged'),
-            'text-success-600 hover:text-success-600': status.get('reblogged'),
-          })}
-          title={reblog_disabled ? intl.formatMessage(messages.cannot_reblog) : intl.formatMessage(messages.reblog)}
-          src={reblogIcon}
-          onClick={this.handleReblogClick}
-          text={intl.formatMessage(messages.reblog)}
-        />
-      );
-    }
+    const reblogMenu: Menu = [{
+      text: intl.formatMessage(status.reblogged ? messages.cancel_reblog_private : messages.reblog),
+      action: this.handleReblogClick,
+      icon: require('@tabler/icons/icons/repeat.svg'),
+    }, {
+      text: intl.formatMessage(messages.quotePost),
+      action: this.handleQuoteClick,
+      icon: require('@tabler/icons/icons/quote.svg'),
+    }];
+
+    const reblogButton = (
+      <IconButton
+        disabled={reblog_disabled}
+        className={classNames({
+          'text-gray-400 hover:text-gray-600': !status.reblogged,
+          'text-success-600 hover:text-success-600': status.reblogged,
+        })}
+        title={reblog_disabled ? intl.formatMessage(messages.cannot_reblog) : intl.formatMessage(messages.reblog)}
+        src={reblogIcon}
+        onClick={this.handleReblogClick}
+        text={intl.formatMessage(messages.reblog)}
+      />
+    );
 
     return (
       <HStack justifyContent='between'>
@@ -576,7 +556,17 @@ class ActionBar extends React.PureComponent<IActionBar, IActionBarState> {
           text={intl.formatMessage(messages.reply)}
         />
 
-        {reblogButton}
+        {(features.quotePosts && me) ? (
+          <DropdownMenuContainer
+            items={reblogMenu}
+            disabled={!publicStatus}
+            onShiftClick={this.handleReblogClick}
+          >
+            {reblogButton}
+          </DropdownMenuContainer>
+        ) : (
+          reblogButton
+        )}
 
         {features.emojiReacts ? (
           <EmojiButtonWrapper statusId={status.id}>