diff --git a/app/soapbox/components/badge.tsx b/app/soapbox/components/badge.tsx index 7ca11c863..2eb2d221d 100644 --- a/app/soapbox/components/badge.tsx +++ b/app/soapbox/components/badge.tsx @@ -15,7 +15,7 @@ const Badge: React.FC = ({ title, slug }) => ( 'bg-yellow-500': slug === 'donor', 'bg-black': slug === 'admin', 'bg-cyan-600': slug === 'moderator', - 'bg-gray-100 text-gray-800': slug === 'bot', + 'bg-gray-100 text-gray-900': slug === 'bot', 'bg-white bg-opacity-75 text-gray-900': slug === 'opaque', })} > 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 - {features.quotePosts && me ? ( + {(features.quotePosts && me) ? ( = { mention: require('@tabler/icons/icons/at.svg'), favourite: require('@tabler/icons/icons/heart.svg'), reblog: require('@tabler/icons/icons/repeat.svg'), - status: require('@tabler/icons/icons/home.svg'), + status: require('@tabler/icons/icons/bell-ringing.svg'), poll: require('@tabler/icons/icons/chart-bar.svg'), move: require('@tabler/icons/icons/briefcase.svg'), 'pleroma:chat_mention': require('@tabler/icons/icons/messages.svg'), 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 { const reblog_disabled = (status.get('visibility') === 'direct' || status.get('visibility') === 'private'); - let reblogButton; + 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'), + }]; - 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 = ( - - ); - } else { - reblogButton = ( - - ); - } + const reblogButton = ( + + ); return ( @@ -576,7 +556,17 @@ class ActionBar extends React.PureComponent { text={intl.formatMessage(messages.reply)} /> - {reblogButton} + {(features.quotePosts && me) ? ( + + {reblogButton} + + ) : ( + reblogButton + )} {features.emojiReacts ? ( diff --git a/app/soapbox/pages/profile_page.tsx b/app/soapbox/pages/profile_page.tsx index 5cf8bee21..e1691e14b 100644 --- a/app/soapbox/pages/profile_page.tsx +++ b/app/soapbox/pages/profile_page.tsx @@ -27,23 +27,22 @@ interface IProfilePage { }, } +/** Page to display a user's profile. */ const ProfilePage: React.FC = ({ params, children }) => { const history = useHistory(); + const username = params?.username || ''; - const { accountId, account, accountUsername, realAccount } = useAppSelector(state => { - const username = params?.username || ''; + const { accountId, account, realAccount } = useAppSelector(state => { const { accounts } = state; const accountFetchError = (((state.accounts.getIn([-1, 'username']) || '') as string).toLowerCase() === username.toLowerCase()); let accountId: string | -1 | null = -1; let account = null; - let accountUsername = username; if (accountFetchError) { accountId = null; } else { account = findAccountByUsername(state, username); accountId = account ? account.id : -1; - accountUsername = account ? account.acct : ''; } let realAccount; @@ -57,7 +56,6 @@ const ProfilePage: React.FC = ({ params, children }) => { return { account: typeof accountId === 'string' ? getAccount(state, accountId) : account, accountId, - accountUsername, realAccount, }; }); @@ -66,24 +64,30 @@ const ProfilePage: React.FC = ({ params, children }) => { const features = useFeatures(); const { displayFqn } = useSoapboxConfig(); + // Redirect from a user ID if (realAccount) { return ; } + // Fix case of username + if (account && account.acct !== username) { + return ; + } + const tabItems = [ { text: , - to: `/@${accountUsername}`, + to: `/@${username}`, name: 'profile', }, { text: , - to: `/@${accountUsername}/with_replies`, + to: `/@${username}/with_replies`, name: 'replies', }, { text: , - to: `/@${accountUsername}/media`, + to: `/@${username}/media`, name: 'media', }, ]; @@ -100,14 +104,14 @@ const ProfilePage: React.FC = ({ params, children }) => { } let activeItem; - const pathname = history.location.pathname.replace(`@${accountUsername}/`, ''); + const pathname = history.location.pathname.replace(`@${username}/`, ''); if (pathname.includes('with_replies')) { activeItem = 'replies'; } else if (pathname.includes('media')) { activeItem = 'media'; } else if (pathname.includes('favorites')) { activeItem = 'likes'; - } else if (pathname === `/@${accountUsername}`) { + } else { activeItem = 'profile'; } @@ -117,13 +121,13 @@ const ProfilePage: React.FC = ({ params, children }) => {
{/* @ts-ignore */} - + - {Component => } + {Component => } - {account && activeItem && ( + {account && ( )}