From 71391bcbce99c69fe78917b979d0159af911a91b Mon Sep 17 00:00:00 2001
From: Justin <dice.warrant-02@icloud.com>
Date: Fri, 13 May 2022 15:08:11 -0400
Subject: [PATCH 01/11] Improve UX of mobile landing page

---
 .../public_layout/components/header.tsx       | 25 +++++++++++--------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/app/soapbox/features/public_layout/components/header.tsx b/app/soapbox/features/public_layout/components/header.tsx
index 098994613..2153dff05 100644
--- a/app/soapbox/features/public_layout/components/header.tsx
+++ b/app/soapbox/features/public_layout/components/header.tsx
@@ -76,24 +76,27 @@ const Header = () => {
       <nav className='max-w-7xl mx-auto px-4 sm:px-6 lg:px-8' aria-label='Header'>
         <div className='w-full py-6 flex items-center justify-between border-b border-indigo-500 lg:border-none'>
           <div className='flex items-center sm:justify-center relative w-36'>
-            <div className='hidden sm:block absolute z-0 -top-24 -left-6'>
+            <div className='hidden md:block absolute z-0 -top-24 -left-6'>
               <Sonar />
             </div>
-            <Link to='/' className='z-10'>
-              <SiteLogo alt='Logo' className='h-6 w-auto cursor-pointer' />
-              <span className='hidden'>{intl.formatMessage(messages.home)}</span>
-            </Link>
-          </div>
-          <div className='ml-10 flex space-x-6 items-center relative z-10'>
+
             <IconButton
               title='Open Menu'
               src={require('@tabler/icons/icons/menu-2.svg')}
               onClick={open}
-              className='md:hidden bg-transparent text-gray-400 hover:text-gray-600'
+              className='md:hidden mr-4 bg-transparent text-gray-400 hover:text-gray-600'
             />
 
-            <div className='hidden md:flex items-center space-x-6'>
-              <HStack space={6} alignItems='center'>
+            <Link to='/' className='z-10'>
+              <SiteLogo alt='Logo' className='h-6 w-auto cursor-pointer' />
+              <span className='hidden'>{intl.formatMessage(messages.home)}</span>
+            </Link>
+
+          </div>
+
+          <div className='ml-10 flex space-x-6 items-center relative z-10'>
+            <HStack alignItems='center'>
+              <HStack space={6} alignItems='center' className='hidden md:flex md:mr-6'>
                 {links.get('help') && (
                   <a
                     href={links.get('help')}
@@ -119,7 +122,7 @@ const Header = () => {
                   </Button>
                 )}
               </HStack>
-            </div>
+            </HStack>
 
             <Form className='hidden xl:flex space-x-2 items-center' onSubmit={handleSubmit}>
               <Input

From 1307b436b1bda304be42b6a600334b1cfea3801a Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Fri, 13 May 2022 20:23:03 -0500
Subject: [PATCH 02/11] Focus the active item in the thread

---
 app/soapbox/components/scrollable_list.tsx | 15 ++++++++----
 app/soapbox/features/status/index.tsx      | 27 +++++++++++-----------
 2 files changed, 24 insertions(+), 18 deletions(-)

diff --git a/app/soapbox/components/scrollable_list.tsx b/app/soapbox/components/scrollable_list.tsx
index 672520fa5..cc4b55867 100644
--- a/app/soapbox/components/scrollable_list.tsx
+++ b/app/soapbox/components/scrollable_list.tsx
@@ -1,5 +1,5 @@
 import React from 'react';
-import { Virtuoso, Components } from 'react-virtuoso';
+import { Virtuoso, Components, VirtuosoProps, VirtuosoHandle } from 'react-virtuoso';
 
 import PullToRefresh from 'soapbox/components/pull-to-refresh';
 import { useSettings } from 'soapbox/hooks';
@@ -25,7 +25,7 @@ const List: Components<Context>['List'] = React.forwardRef((props, ref) => {
   return <div ref={ref} className={context?.listClassName} {...rest} />;
 });
 
-interface IScrollableList {
+interface IScrollableList extends VirtuosoProps<any, any> {
   scrollKey?: string,
   onLoadMore?: () => void,
   isLoading?: boolean,
@@ -45,7 +45,7 @@ interface IScrollableList {
 }
 
 /** Legacy ScrollableList with Virtuoso for backwards-compatibility */
-const ScrollableList: React.FC<IScrollableList> = ({
+const ScrollableList = React.forwardRef<VirtuosoHandle, IScrollableList>(({
   prepend = null,
   alwaysPrepend,
   children,
@@ -61,7 +61,9 @@ const ScrollableList: React.FC<IScrollableList> = ({
   hasMore,
   placeholderComponent: Placeholder,
   placeholderCount = 0,
-}) => {
+  initialTopMostItemIndex = 0,
+  scrollerRef,
+}, ref) => {
   const settings = useSettings();
   const autoloadMore = settings.get('autoloadMore');
 
@@ -126,6 +128,7 @@ const ScrollableList: React.FC<IScrollableList> = ({
   /** Render the actual Virtuoso list */
   const renderFeed = (): JSX.Element => (
     <Virtuoso
+      ref={ref}
       useWindowScroll
       className={className}
       data={data}
@@ -133,6 +136,7 @@ const ScrollableList: React.FC<IScrollableList> = ({
       endReached={handleEndReached}
       isScrolling={isScrolling => isScrolling && onScroll && onScroll()}
       itemContent={renderItem}
+      initialTopMostItemIndex={showLoading ? 0 : initialTopMostItemIndex}
       context={{
         listClassName: className,
         itemClassName,
@@ -145,6 +149,7 @@ const ScrollableList: React.FC<IScrollableList> = ({
         Item,
         Footer: loadMore,
       }}
+      scrollerRef={scrollerRef}
     />
   );
 
@@ -162,6 +167,6 @@ const ScrollableList: React.FC<IScrollableList> = ({
       {renderBody()}
     </PullToRefresh>
   );
-};
+});
 
 export default ScrollableList;
diff --git a/app/soapbox/features/status/index.tsx b/app/soapbox/features/status/index.tsx
index 25247ac6f..16c398df7 100644
--- a/app/soapbox/features/status/index.tsx
+++ b/app/soapbox/features/status/index.tsx
@@ -65,6 +65,7 @@ import ThreadStatus from './components/thread-status';
 
 import type { AxiosError } from 'axios';
 import type { History } from 'history';
+import type { VirtuosoHandle } from 'react-virtuoso';
 import type { AnyAction } from 'redux';
 import type { ThunkDispatch } from 'redux-thunk';
 import type { RootState } from 'soapbox/store';
@@ -210,6 +211,7 @@ class Status extends ImmutablePureComponent<IStatus, IStatusState> {
 
   node: HTMLDivElement | null = null;
   status: HTMLDivElement | null = null;
+  scroller: VirtuosoHandle | null = null;
   _scrolledIntoView: boolean = false;
 
   fetchData = async() => {
@@ -615,11 +617,10 @@ class Status extends ImmutablePureComponent<IStatus, IStatusState> {
   }
 
   componentDidUpdate(prevProps: IStatus, prevState: IStatusState) {
-    const { params, status, displayMedia } = this.props;
-    const { ancestorsIds } = prevProps;
+    const { params, status, displayMedia, ancestorsIds } = this.props;
+    const { isLoaded } = this.state;
 
     if (params.statusId !== prevProps.params.statusId) {
-      this._scrolledIntoView = false;
       this.fetchData();
     }
 
@@ -627,17 +628,11 @@ class Status extends ImmutablePureComponent<IStatus, IStatusState> {
       this.setState({ showMedia: defaultMediaVisibility(status, displayMedia), loadedStatusId: status.id });
     }
 
-    if (this._scrolledIntoView) {
-      return;
-    }
-
-    if (prevProps.status && ancestorsIds && ancestorsIds.size > 0 && this.node) {
-      const element = this.node.querySelector('.detailed-status');
-
-      window.requestAnimationFrame(() => {
-        element?.scrollIntoView(true);
+    if (params.statusId !== prevProps.params.statusId || status?.id !== prevProps.status?.id || ancestorsIds.size > prevProps.ancestorsIds.size || isLoaded !== prevState.isLoaded) {
+      this.scroller?.scrollToIndex({
+        index: this.props.ancestorsIds.size,
+        offset: -80,
       });
-      this._scrolledIntoView = true;
     }
   }
 
@@ -671,6 +666,10 @@ class Status extends ImmutablePureComponent<IStatus, IStatusState> {
     }));
   }
 
+  setScrollerRef = (c: VirtuosoHandle) => {
+    this.scroller = c;
+  }
+
   render() {
     const { me, status, ancestorsIds, descendantsIds, intl } = this.props;
 
@@ -788,10 +787,12 @@ class Status extends ImmutablePureComponent<IStatus, IStatusState> {
         <Stack space={2}>
           <div ref={this.setRef} className='thread'>
             <ScrollableList
+              ref={this.setScrollerRef}
               onRefresh={this.handleRefresh}
               hasMore={!!this.state.next}
               onLoadMore={this.handleLoadMore}
               placeholderComponent={() => <PlaceholderStatus thread />}
+              initialTopMostItemIndex={ancestorsIds.size}
             >
               {children}
             </ScrollableList>

From ac610f98d06809b3a5987ec1296d2090de73015a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?marcin=20miko=C5=82ajczak?= <git@mkljczk.pl>
Date: Sat, 14 May 2022 10:37:06 +0200
Subject: [PATCH 03/11] Reverse auth sessions list
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/features/auth_token_list/index.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/soapbox/features/auth_token_list/index.tsx b/app/soapbox/features/auth_token_list/index.tsx
index 8014def9f..4563e4b34 100644
--- a/app/soapbox/features/auth_token_list/index.tsx
+++ b/app/soapbox/features/auth_token_list/index.tsx
@@ -55,7 +55,7 @@ const AuthToken: React.FC<IAuthToken> = ({ token }) => {
 const AuthTokenList: React.FC = () =>{
   const dispatch = useAppDispatch();
   const intl = useIntl();
-  const tokens = useAppSelector(state => state.security.get('tokens'));
+  const tokens = useAppSelector(state => state.security.get('tokens').reverse());
 
   useEffect(() => {
     dispatch(fetchOAuthTokens());

From 3d898957e3dfa55981e2186d0daf7b6059cc868b Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 11:21:12 -0500
Subject: [PATCH 04/11] ActionButton: put remote follow behind feature
 detection

---
 .../features/ui/components/action-button.tsx  | 41 +++++++++++++------
 app/soapbox/utils/features.ts                 |  6 +++
 2 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/app/soapbox/features/ui/components/action-button.tsx b/app/soapbox/features/ui/components/action-button.tsx
index e79c22567..c710f43d9 100644
--- a/app/soapbox/features/ui/components/action-button.tsx
+++ b/app/soapbox/features/ui/components/action-button.tsx
@@ -105,10 +105,9 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
     );
   };
 
-  const empty = <></>;
-
-  if (!me) {
-    // Remote follow
+  /** Render a remote follow button, depending on features. */
+  const renderRemoteFollow = (): JSX.Element | null => {
+    // Remote follow through the API.
     if (features.remoteInteractionsAPI) {
       return (
         <Button
@@ -117,18 +116,36 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
           text={intl.formatMessage(messages.follow)}
         />
       );
+    // Pleroma's classic remote follow form.
+    } else if (features.pleromaRemoteFollow) {
+      return (
+        <form method='POST' action='/main/ostatus'>
+          <input type='hidden' name='nickname' value={account.acct} />
+          <input type='hidden' name='profile' value='' />
+          <Button text={intl.formatMessage(messages.remote_follow)} type='submit' />
+        </form>
+      );
     }
 
-    return (
-      <form method='POST' action='/main/ostatus'>
-        <input type='hidden' name='nickname' value={account.get('acct')} />
-        <input type='hidden' name='profile' value='' />
-        <Button text={intl.formatMessage(messages.remote_follow)} type='submit' />
-      </form>
-    );
+    return null;
+  };
+
+  /** Render remote follow if federating, otherwise hide the button. */
+  const renderLoggedOut = (): JSX.Element | null => {
+    if (features.federating) {
+      return renderRemoteFollow();
+    }
+
+    return null;
+  };
+
+  const empty = <></>;
+
+  if (!me) {
+    return renderLoggedOut();
   }
 
-  if (me !== account.get('id')) {
+  if (me !== account.id) {
     const isFollowing = account.getIn(['relationship', 'following']);
     const blockedBy = account.getIn(['relationship', 'blocked_by']) as boolean;
 
diff --git a/app/soapbox/utils/features.ts b/app/soapbox/utils/features.ts
index 9f60b5a05..1e4d76266 100644
--- a/app/soapbox/utils/features.ts
+++ b/app/soapbox/utils/features.ts
@@ -356,6 +356,12 @@ const getInstanceFeatures = (instance: Instance) => {
      */
     paginatedContext: v.software === TRUTHSOCIAL,
 
+    /**
+     * Displays a form to follow a user when logged out.
+     * @see POST /main/ostatus
+     */
+    pleromaRemoteFollow: v.software === PLEROMA,
+
     /**
      * Can add polls to statuses.
      * @see POST /api/v1/statuses

From 9e09823f807e1480639e0a59267f2473180eebf0 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 11:44:31 -0500
Subject: [PATCH 05/11] ActionButton: remove most calls to .get, fix
 relationship loading bug

---
 .../features/ui/components/action-button.tsx  | 28 +++++++++----------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/app/soapbox/features/ui/components/action-button.tsx b/app/soapbox/features/ui/components/action-button.tsx
index c710f43d9..7d5967417 100644
--- a/app/soapbox/features/ui/components/action-button.tsx
+++ b/app/soapbox/features/ui/components/action-button.tsx
@@ -30,13 +30,13 @@ const messages = defineMessages({
   unmute: { id: 'account.unmute', defaultMessage: 'Unmute @{name}' },
 });
 
-interface iActionButton {
+interface IActionButton {
   account: AccountEntity
   actionType?: 'muting' | 'blocking'
   small?: boolean
 }
 
-const ActionButton = ({ account, actionType, small }: iActionButton) => {
+const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) => {
   const dispatch = useDispatch();
   const features = useFeatures();
   const intl = useIntl();
@@ -45,40 +45,40 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
 
   const handleFollow = () => {
     if (account.getIn(['relationship', 'following']) || account.getIn(['relationship', 'requested'])) {
-      dispatch(unfollowAccount(account.get('id')));
+      dispatch(unfollowAccount(account.id));
     } else {
-      dispatch(followAccount(account.get('id')));
+      dispatch(followAccount(account.id));
     }
   };
 
   const handleBlock = () => {
     if (account.getIn(['relationship', 'blocking'])) {
-      dispatch(unblockAccount(account.get('id')));
+      dispatch(unblockAccount(account.id));
     } else {
-      dispatch(blockAccount(account.get('id')));
+      dispatch(blockAccount(account.id));
     }
   };
 
   const handleMute = () => {
     if (account.getIn(['relationship', 'muting'])) {
-      dispatch(unmuteAccount(account.get('id')));
+      dispatch(unmuteAccount(account.id));
     } else {
-      dispatch(muteAccount(account.get('id')));
+      dispatch(muteAccount(account.id));
     }
   };
 
   const handleRemoteFollow = () => {
     dispatch(openModal('UNAUTHORIZED', {
       action: 'FOLLOW',
-      account: account.get('id'),
-      ap_id: account.get('url'),
+      account: account.id,
+      ap_id: account.url,
     }));
   };
 
   const mutingAction = () => {
     const isMuted = account.getIn(['relationship', 'muting']);
     const messageKey = isMuted ? messages.unmute : messages.mute;
-    const text = intl.formatMessage(messageKey, { name: account.get('username') });
+    const text = intl.formatMessage(messageKey, { name: account.username });
 
     return (
       <Button
@@ -93,7 +93,7 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
   const blockingAction = () => {
     const isBlocked = account.getIn(['relationship', 'blocking']);
     const messageKey = isBlocked ? messages.unblock : messages.block;
-    const text = intl.formatMessage(messageKey, { name: account.get('username') });
+    const text = intl.formatMessage(messageKey, { name: account.username });
 
     return (
       <Button
@@ -157,7 +157,7 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
       }
     }
 
-    if (!account.get('relationship')) {
+    if (account.relationship.isEmpty()) {
       // Wait until the relationship is loaded
       return empty;
     } else if (account.getIn(['relationship', 'requested'])) {
@@ -193,7 +193,7 @@ const ActionButton = ({ account, actionType, small }: iActionButton) => {
         <Button
           theme='danger'
           size='sm'
-          text={intl.formatMessage(messages.unblock, { name: account.get('username') })}
+          text={intl.formatMessage(messages.unblock, { name: account.username })}
           onClick={handleBlock}
         />
       );

From 6fa6e5b3b4d0c1da230f397d3798aa13df891f2f Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 11:48:57 -0500
Subject: [PATCH 06/11] ActionButton: add jsdoc comments

---
 .../features/ui/components/action-button.tsx       | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/app/soapbox/features/ui/components/action-button.tsx b/app/soapbox/features/ui/components/action-button.tsx
index 7d5967417..acf06e244 100644
--- a/app/soapbox/features/ui/components/action-button.tsx
+++ b/app/soapbox/features/ui/components/action-button.tsx
@@ -31,11 +31,19 @@ const messages = defineMessages({
 });
 
 interface IActionButton {
+  /** Target account for the action. */
   account: AccountEntity
+  /** Type of action to prioritize, eg on Blocks and Mutes pages. */
   actionType?: 'muting' | 'blocking'
+  /** Displays shorter text on the "Awaiting approval" button. */
   small?: boolean
 }
 
+/**
+ * Circumstantial action button (usually "Follow") to display on accounts.
+ * May say "Unblock" or something else, depending on the relationship and
+ * `actionType` prop.
+ */
 const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) => {
   const dispatch = useDispatch();
   const features = useFeatures();
@@ -75,6 +83,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
     }));
   };
 
+  /** Handles actionType='muting' */
   const mutingAction = () => {
     const isMuted = account.getIn(['relationship', 'muting']);
     const messageKey = isMuted ? messages.unmute : messages.mute;
@@ -90,6 +99,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
     );
   };
 
+  /** Handles actionType='blocking' */
   const blockingAction = () => {
     const isBlocked = account.getIn(['relationship', 'blocking']);
     const messageKey = isBlocked ? messages.unblock : messages.block;
@@ -106,7 +116,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
   };
 
   /** Render a remote follow button, depending on features. */
-  const renderRemoteFollow = (): JSX.Element | null => {
+  const renderRemoteFollow = () => {
     // Remote follow through the API.
     if (features.remoteInteractionsAPI) {
       return (
@@ -131,7 +141,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
   };
 
   /** Render remote follow if federating, otherwise hide the button. */
-  const renderLoggedOut = (): JSX.Element | null => {
+  const renderLoggedOut = () => {
     if (features.federating) {
       return renderRemoteFollow();
     }

From 22d37a2f10d10d6c829aa10ce0a8ff4ebceffd1e Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 11:49:28 -0500
Subject: [PATCH 07/11] ActionButton: remove `empty` variable(?)

---
 app/soapbox/features/ui/components/action-button.tsx | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/app/soapbox/features/ui/components/action-button.tsx b/app/soapbox/features/ui/components/action-button.tsx
index acf06e244..51348ee1d 100644
--- a/app/soapbox/features/ui/components/action-button.tsx
+++ b/app/soapbox/features/ui/components/action-button.tsx
@@ -149,8 +149,6 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
     return null;
   };
 
-  const empty = <></>;
-
   if (!me) {
     return renderLoggedOut();
   }
@@ -169,7 +167,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
 
     if (account.relationship.isEmpty()) {
       // Wait until the relationship is loaded
-      return empty;
+      return null;
     } else if (account.getIn(['relationship', 'requested'])) {
       // Awaiting acceptance
       return (
@@ -220,7 +218,7 @@ const ActionButton: React.FC<IActionButton> = ({ account, actionType, small }) =
     );
   }
 
-  return empty;
+  return null;
 };
 
 export default ActionButton;

From 165d6d197e14d56b1d96fd2610cbe6ad0d24cc84 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 12:01:21 -0500
Subject: [PATCH 08/11] Rip out identity_proofs

---
 app/soapbox/actions/identity_proofs.js        | 30 -----------------
 .../features/account/components/header.js     |  1 -
 .../account_timeline/components/header.js     |  4 +--
 .../containers/header_container.js            |  2 --
 .../features/account_timeline/index.js        |  7 ++--
 .../__tests__/identity_proofs-test.js         | 32 -------------------
 app/soapbox/reducers/identity_proofs.js       | 26 ---------------
 app/soapbox/reducers/index.ts                 |  2 --
 docs/store.md                                 |  2 --
 9 files changed, 3 insertions(+), 103 deletions(-)
 delete mode 100644 app/soapbox/actions/identity_proofs.js
 delete mode 100644 app/soapbox/reducers/__tests__/identity_proofs-test.js
 delete mode 100644 app/soapbox/reducers/identity_proofs.js

diff --git a/app/soapbox/actions/identity_proofs.js b/app/soapbox/actions/identity_proofs.js
deleted file mode 100644
index a14455a64..000000000
--- a/app/soapbox/actions/identity_proofs.js
+++ /dev/null
@@ -1,30 +0,0 @@
-import api from '../api';
-
-export const IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST = 'IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST';
-export const IDENTITY_PROOFS_ACCOUNT_FETCH_SUCCESS = 'IDENTITY_PROOFS_ACCOUNT_FETCH_SUCCESS';
-export const IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL    = 'IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL';
-
-export const fetchAccountIdentityProofs = accountId => (dispatch, getState) => {
-  dispatch(fetchAccountIdentityProofsRequest(accountId));
-
-  api(getState).get(`/api/v1/accounts/${accountId}/identity_proofs`)
-    .then(({ data }) => dispatch(fetchAccountIdentityProofsSuccess(accountId, data)))
-    .catch(err => dispatch(fetchAccountIdentityProofsFail(accountId, err)));
-};
-
-export const fetchAccountIdentityProofsRequest = id => ({
-  type: IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST,
-  id,
-});
-
-export const fetchAccountIdentityProofsSuccess = (accountId, identity_proofs) => ({
-  type: IDENTITY_PROOFS_ACCOUNT_FETCH_SUCCESS,
-  accountId,
-  identity_proofs,
-});
-
-export const fetchAccountIdentityProofsFail = (accountId, error) => ({
-  type: IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL,
-  accountId,
-  error,
-});
diff --git a/app/soapbox/features/account/components/header.js b/app/soapbox/features/account/components/header.js
index b16892cb8..36a0b7b7e 100644
--- a/app/soapbox/features/account/components/header.js
+++ b/app/soapbox/features/account/components/header.js
@@ -86,7 +86,6 @@ class Header extends ImmutablePureComponent {
   static propTypes = {
     account: ImmutablePropTypes.record,
     meaccount: ImmutablePropTypes.record,
-    identity_props: ImmutablePropTypes.list,
     intl: PropTypes.object.isRequired,
     username: PropTypes.string,
     features: PropTypes.object,
diff --git a/app/soapbox/features/account_timeline/components/header.js b/app/soapbox/features/account_timeline/components/header.js
index 740225878..b969e0b61 100644
--- a/app/soapbox/features/account_timeline/components/header.js
+++ b/app/soapbox/features/account_timeline/components/header.js
@@ -13,7 +13,6 @@ class Header extends ImmutablePureComponent {
 
   static propTypes = {
     account: ImmutablePropTypes.record,
-    identity_proofs: ImmutablePropTypes.list,
     onFollow: PropTypes.func.isRequired,
     onBlock: PropTypes.func.isRequired,
     onMention: PropTypes.func.isRequired,
@@ -143,7 +142,7 @@ class Header extends ImmutablePureComponent {
   }
 
   render() {
-    const { account, identity_proofs } = this.props;
+    const { account } = this.props;
     const moved = (account) ? account.get('moved') : false;
 
     return (
@@ -152,7 +151,6 @@ class Header extends ImmutablePureComponent {
 
         <InnerHeader
           account={account}
-          identity_proofs={identity_proofs}
           onFollow={this.handleFollow}
           onBlock={this.handleBlock}
           onMention={this.handleMention}
diff --git a/app/soapbox/features/account_timeline/containers/header_container.js b/app/soapbox/features/account_timeline/containers/header_container.js
index 4fbfa6d01..43ceceb2f 100644
--- a/app/soapbox/features/account_timeline/containers/header_container.js
+++ b/app/soapbox/features/account_timeline/containers/header_container.js
@@ -1,4 +1,3 @@
-import { List as ImmutableList } from 'immutable';
 import React from 'react';
 import { defineMessages, injectIntl, FormattedMessage } from 'react-intl';
 import { connect } from 'react-redux';
@@ -64,7 +63,6 @@ const makeMapStateToProps = () => {
 
   const mapStateToProps = (state, { accountId }) => ({
     account: getAccount(state, accountId),
-    identity_proofs: state.getIn(['identity_proofs', accountId], ImmutableList()),
   });
 
   return mapStateToProps;
diff --git a/app/soapbox/features/account_timeline/index.js b/app/soapbox/features/account_timeline/index.js
index 207344bcd..c394eecf2 100644
--- a/app/soapbox/features/account_timeline/index.js
+++ b/app/soapbox/features/account_timeline/index.js
@@ -13,7 +13,6 @@ import { makeGetStatusIds, findAccountByUsername } from 'soapbox/selectors';
 import { getFeatures } from 'soapbox/utils/features';
 
 import { fetchAccount, fetchAccountByUsername } from '../../actions/accounts';
-import { fetchAccountIdentityProofs } from '../../actions/identity_proofs';
 import { fetchPatronAccount } from '../../actions/patron';
 import { expandAccountFeaturedTimeline, expandAccountTimeline } from '../../actions/timelines';
 import StatusList from '../../components/status_list';
@@ -84,11 +83,10 @@ class AccountTimeline extends ImmutablePureComponent {
   };
 
   componentDidMount() {
-    const { params: { username }, accountId, accountApId, withReplies, me, patronEnabled } = this.props;
+    const { params: { username }, accountId, accountApId, withReplies, patronEnabled } = this.props;
 
     if (accountId && accountId !== -1) {
       this.props.dispatch(fetchAccount(accountId));
-      if (me) this.props.dispatch(fetchAccountIdentityProofs(accountId));
 
       if (!withReplies) {
         this.props.dispatch(expandAccountFeaturedTimeline(accountId));
@@ -105,11 +103,10 @@ class AccountTimeline extends ImmutablePureComponent {
   }
 
   componentDidUpdate(prevProps) {
-    const { params: { username }, me, accountId, withReplies, accountApId, patronEnabled } = this.props;
+    const { params: { username }, accountId, withReplies, accountApId, patronEnabled } = this.props;
 
     if (accountId && (accountId !== -1) && (accountId !== prevProps.accountId) || withReplies !== prevProps.withReplies) {
       this.props.dispatch(fetchAccount(accountId));
-      if (me) this.props.dispatch(fetchAccountIdentityProofs(accountId));
 
       if (!withReplies) {
         this.props.dispatch(expandAccountFeaturedTimeline(accountId));
diff --git a/app/soapbox/reducers/__tests__/identity_proofs-test.js b/app/soapbox/reducers/__tests__/identity_proofs-test.js
deleted file mode 100644
index fd004bfa8..000000000
--- a/app/soapbox/reducers/__tests__/identity_proofs-test.js
+++ /dev/null
@@ -1,32 +0,0 @@
-import { Map as ImmutableMap } from 'immutable';
-
-import * as actions from 'soapbox/actions/identity_proofs';
-
-import reducer from '../identity_proofs';
-
-describe('identity_proofs reducer', () => {
-  it('should return the initial state', () => {
-    expect(reducer(undefined, {})).toEqual(ImmutableMap());
-  });
-
-  it('should handle IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST', () => {
-    const state = ImmutableMap({ isLoading: false });
-    const action = {
-      type: actions.IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST,
-    };
-    expect(reducer(state, action).toJS()).toMatchObject({
-      isLoading: true,
-    });
-  });
-
-  it('should handle IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL', () => {
-    const state = ImmutableMap({ isLoading: true });
-    const action = {
-      type: actions.IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL,
-    };
-    expect(reducer(state, action).toJS()).toMatchObject({
-      isLoading: false,
-    });
-  });
-
-});
diff --git a/app/soapbox/reducers/identity_proofs.js b/app/soapbox/reducers/identity_proofs.js
deleted file mode 100644
index cb6721b07..000000000
--- a/app/soapbox/reducers/identity_proofs.js
+++ /dev/null
@@ -1,26 +0,0 @@
-import { Map as ImmutableMap, fromJS } from 'immutable';
-
-import {
-  IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST,
-  IDENTITY_PROOFS_ACCOUNT_FETCH_SUCCESS,
-  IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL,
-} from '../actions/identity_proofs';
-
-const initialState = ImmutableMap();
-
-export default function identityProofsReducer(state = initialState, action) {
-  switch (action.type) {
-    case IDENTITY_PROOFS_ACCOUNT_FETCH_REQUEST:
-      return state.set('isLoading', true);
-    case IDENTITY_PROOFS_ACCOUNT_FETCH_FAIL:
-      return state.set('isLoading', false);
-    case IDENTITY_PROOFS_ACCOUNT_FETCH_SUCCESS:
-      return state.update(identity_proofs => identity_proofs.withMutations(map => {
-        map.set('isLoading', false);
-        map.set('loaded', true);
-        map.set(action.accountId, fromJS(action.identity_proofs));
-      }));
-    default:
-      return state;
-  }
-}
diff --git a/app/soapbox/reducers/index.ts b/app/soapbox/reducers/index.ts
index 084b9d776..a05ef47c4 100644
--- a/app/soapbox/reducers/index.ts
+++ b/app/soapbox/reducers/index.ts
@@ -29,7 +29,6 @@ import group_lists from './group_lists';
 import group_relationships from './group_relationships';
 import groups from './groups';
 import history from './history';
-import identity_proofs from './identity_proofs';
 import instance from './instance';
 import listAdder from './list_adder';
 import listEditor from './list_editor';
@@ -86,7 +85,6 @@ const reducers = {
   search,
   notifications,
   custom_emojis,
-  identity_proofs,
   lists,
   listEditor,
   listAdder,
diff --git a/docs/store.md b/docs/store.md
index 7c0bff506..994059c22 100644
--- a/docs/store.md
+++ b/docs/store.md
@@ -691,8 +691,6 @@ If it's not documented, it's because I inherited it from Mastodon and I don't kn
   ]
     ```
 
-- `identity_proofs`
-
 - `lists`
 
     Sample:

From dc05a4a11ddb8c517b5de7d82ec7388036545d18 Mon Sep 17 00:00:00 2001
From: Alex Gleason <alex@alexgleason.me>
Date: Sat, 14 May 2022 12:20:39 -0500
Subject: [PATCH 09/11] SearchResults: use padding, not margins

---
 .../compose/components/search_results.js      | 35 +++++++++----------
 1 file changed, 16 insertions(+), 19 deletions(-)

diff --git a/app/soapbox/features/compose/components/search_results.js b/app/soapbox/features/compose/components/search_results.js
index b5fccc78b..05cbe2663 100644
--- a/app/soapbox/features/compose/components/search_results.js
+++ b/app/soapbox/features/compose/components/search_results.js
@@ -6,7 +6,6 @@ import ImmutablePureComponent from 'react-immutable-pure-component';
 import { FormattedMessage } from 'react-intl';
 import { defineMessages, injectIntl } from 'react-intl';
 
-import Pullable from 'soapbox/components/pullable';
 import ScrollableList from 'soapbox/components/scrollable_list';
 import PlaceholderAccount from 'soapbox/features/placeholder/components/placeholder_account';
 import PlaceholderHashtag from 'soapbox/features/placeholder/components/placeholder_hashtag';
@@ -151,24 +150,22 @@ class SearchResults extends ImmutablePureComponent {
         {this.renderFilterBar()}
 
         {noResultsMessage || (
-          <Pullable>
-            <ScrollableList
-              key={selectedFilter}
-              scrollKey={`${selectedFilter}:${value}`}
-              isLoading={submitted && !loaded}
-              showLoading={submitted && !loaded && results.isEmpty()}
-              hasMore={hasMore}
-              onLoadMore={this.handleLoadMore}
-              placeholderComponent={placeholderComponent}
-              placeholderCount={20}
-              className={classNames({
-                'divide-gray-200 dark:divide-slate-700 divide-solid divide-y': selectedFilter === 'statuses',
-                'space-y-4': selectedFilter === 'accounts',
-              })}
-            >
-              {searchResults}
-            </ScrollableList>
-          </Pullable>
+          <ScrollableList
+            key={selectedFilter}
+            scrollKey={`${selectedFilter}:${value}`}
+            isLoading={submitted && !loaded}
+            showLoading={submitted && !loaded && results.isEmpty()}
+            hasMore={hasMore}
+            onLoadMore={this.handleLoadMore}
+            placeholderComponent={placeholderComponent}
+            placeholderCount={20}
+            className={classNames({
+              'divide-gray-200 dark:divide-slate-700 divide-solid divide-y': selectedFilter === 'statuses',
+            })}
+            itemClassName={classNames({ 'pb-4': selectedFilter === 'accounts' })}
+          >
+            {searchResults}
+          </ScrollableList>
         )}
       </>
     );

From 3d76cd21e6af794edfe452460c985ec11d96b69e Mon Sep 17 00:00:00 2001
From: Justin <dice.warrant-02@icloud.com>
Date: Sat, 14 May 2022 14:33:14 -0400
Subject: [PATCH 10/11] Skip API requests if logged out

---
 app/soapbox/actions/custom_emojis.js |  3 +++
 app/soapbox/actions/suggestions.js   | 11 +++++++----
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/app/soapbox/actions/custom_emojis.js b/app/soapbox/actions/custom_emojis.js
index 9ec8156b1..49e903f37 100644
--- a/app/soapbox/actions/custom_emojis.js
+++ b/app/soapbox/actions/custom_emojis.js
@@ -6,6 +6,9 @@ export const CUSTOM_EMOJIS_FETCH_FAIL = 'CUSTOM_EMOJIS_FETCH_FAIL';
 
 export function fetchCustomEmojis() {
   return (dispatch, getState) => {
+    const me = getState().get('me');
+    if (!me) return;
+
     dispatch(fetchCustomEmojisRequest());
 
     api(getState).get('/api/v1/custom_emojis').then(response => {
diff --git a/app/soapbox/actions/suggestions.js b/app/soapbox/actions/suggestions.js
index 0221b3a05..ede37ff30 100644
--- a/app/soapbox/actions/suggestions.js
+++ b/app/soapbox/actions/suggestions.js
@@ -8,13 +8,13 @@ import { importFetchedAccounts } from './importer';
 
 export const SUGGESTIONS_FETCH_REQUEST = 'SUGGESTIONS_FETCH_REQUEST';
 export const SUGGESTIONS_FETCH_SUCCESS = 'SUGGESTIONS_FETCH_SUCCESS';
-export const SUGGESTIONS_FETCH_FAIL    = 'SUGGESTIONS_FETCH_FAIL';
+export const SUGGESTIONS_FETCH_FAIL = 'SUGGESTIONS_FETCH_FAIL';
 
 export const SUGGESTIONS_DISMISS = 'SUGGESTIONS_DISMISS';
 
 export const SUGGESTIONS_V2_FETCH_REQUEST = 'SUGGESTIONS_V2_FETCH_REQUEST';
 export const SUGGESTIONS_V2_FETCH_SUCCESS = 'SUGGESTIONS_V2_FETCH_SUCCESS';
-export const SUGGESTIONS_V2_FETCH_FAIL    = 'SUGGESTIONS_V2_FETCH_FAIL';
+export const SUGGESTIONS_V2_FETCH_FAIL = 'SUGGESTIONS_V2_FETCH_FAIL';
 
 export function fetchSuggestionsV1(params = {}) {
   return (dispatch, getState) => {
@@ -48,23 +48,26 @@ export function fetchSuggestionsV2(params = {}) {
 export function fetchSuggestions(params = { limit: 50 }) {
   return (dispatch, getState) => {
     const state = getState();
+    const me = state.get('me');
     const instance = state.get('instance');
     const features = getFeatures(instance);
 
+    if (!me) return;
+
     if (features.suggestionsV2) {
       dispatch(fetchSuggestionsV2(params))
         .then(suggestions => {
           const accountIds = suggestions.map(({ account }) => account.id);
           dispatch(fetchRelationships(accountIds));
         })
-        .catch(() => {});
+        .catch(() => { });
     } else if (features.suggestions) {
       dispatch(fetchSuggestionsV1(params))
         .then(accounts => {
           const accountIds = accounts.map(({ id }) => id);
           dispatch(fetchRelationships(accountIds));
         })
-        .catch(() => {});
+        .catch(() => { });
     } else {
       // Do nothing
     }

From 30017d62c9c9696458a5c37a408a2f0af7c83c70 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?marcin=20miko=C5=82ajczak?= <git@mkljczk.pl>
Date: Sun, 15 May 2022 09:06:40 +0200
Subject: [PATCH 11/11] Use arrow-left for ReplyMentionsModal
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/ui/modal/modal.tsx        | 16 +++++++++++++---
 .../ui/components/reply_mentions_modal.tsx       |  2 ++
 2 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/app/soapbox/components/ui/modal/modal.tsx b/app/soapbox/components/ui/modal/modal.tsx
index b78dfe823..5ac16ca14 100644
--- a/app/soapbox/components/ui/modal/modal.tsx
+++ b/app/soapbox/components/ui/modal/modal.tsx
@@ -15,6 +15,10 @@ interface IModal {
   cancelAction?: () => void,
   /** Cancel button text. */
   cancelText?: string,
+  /** URL to an SVG icon for the close button. */
+  closeIcon?: string,
+  /** Position of the close button. */
+  closePosition?: 'left' | 'right',
   /** Callback when the modal is confirmed. */
   confirmationAction?: () => void,
   /** Whether the confirmation button is disabled. */
@@ -40,6 +44,8 @@ const Modal: React.FC<IModal> = ({
   cancelAction,
   cancelText,
   children,
+  closeIcon = require('@tabler/icons/icons/x.svg'),
+  closePosition = 'right',
   confirmationAction,
   confirmationDisabled,
   confirmationText,
@@ -63,14 +69,18 @@ const Modal: React.FC<IModal> = ({
     <div data-testid='modal' className='block w-full max-w-xl p-6 mx-auto overflow-hidden text-left align-middle transition-all transform bg-white dark:bg-slate-800 text-black dark:text-white shadow-xl rounded-2xl pointer-events-auto'>
       <div className='sm:flex sm:items-start w-full justify-between'>
         <div className='w-full'>
-          <div className='w-full flex flex-row justify-between items-center'>
-            <h3 className='text-lg leading-6 font-medium text-gray-900 dark:text-white'>
+          <div
+            className={classNames('w-full flex items-center gap-2', {
+              'flex-row-reverse': closePosition === 'left',
+            })}
+          >
+            <h3 className='flex-grow text-lg leading-6 font-medium text-gray-900 dark:text-white'>
               {title}
             </h3>
 
             {onClose && (
               <IconButton
-                src={require('@tabler/icons/icons/x.svg')}
+                src={closeIcon}
                 title={intl.formatMessage(messages.close)}
                 onClick={onClose}
                 className='text-gray-500 hover:text-gray-700 dark:text-gray-300 dark:hover:text-gray-200'
diff --git a/app/soapbox/features/ui/components/reply_mentions_modal.tsx b/app/soapbox/features/ui/components/reply_mentions_modal.tsx
index 996aeee15..c1d3422fb 100644
--- a/app/soapbox/features/ui/components/reply_mentions_modal.tsx
+++ b/app/soapbox/features/ui/components/reply_mentions_modal.tsx
@@ -29,6 +29,8 @@ const ReplyMentionsModal: React.FC<IReplyMentionsModal> = ({ onClose }) => {
     <Modal
       title={<FormattedMessage id='navigation_bar.in_reply_to' defaultMessage='In reply to' />}
       onClose={onClickClose}
+      closeIcon={require('@tabler/icons/icons/arrow-left.svg')}
+      closePosition='left'
     >
       <div className='reply-mentions-modal__accounts'>
         {mentions.map(accountId => <Account key={accountId} accountId={accountId} added author={author === accountId} />)}