From b7be7018e885bcffbe806f9f6f78d9876b70456e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Wolfgang=20Hu=C3=9F?= <wolle.huss@pjannto.com>
Date: Mon, 12 Sep 2022 08:26:10 +0200
Subject: [PATCH] Move important parts of 'FollowList' to sub component
 'ProfileList'

---
 .../features/ProfileList/FollowList.vue       | 147 ++--------------
 .../features/ProfileList/ProfileList.vue      | 158 ++++++++++++++++++
 2 files changed, 169 insertions(+), 136 deletions(-)
 create mode 100644 webapp/components/features/ProfileList/ProfileList.vue

diff --git a/webapp/components/features/ProfileList/FollowList.vue b/webapp/components/features/ProfileList/FollowList.vue
index 114a610f1..635eccd5b 100644
--- a/webapp/components/features/ProfileList/FollowList.vue
+++ b/webapp/components/features/ProfileList/FollowList.vue
@@ -1,64 +1,28 @@
 <template>
-  <base-card class="follow-list">
-    <template v-if="connections && connections.length">
-      <h5 class="title spacer-x-small">
-        {{ userName | truncate(15) }} {{ $t(`profile.network.${type}`) }}
-      </h5>
-      <ul :class="connectionsClass">
-        <li
-          v-for="connection in filteredConnections"
-          :key="connection.id"
-          class="connections__item"
-        >
-          <user-teaser :user="connection" />
-        </li>
-      </ul>
-      <base-button
-        v-if="hasMore"
-        :loading="loading"
-        class="spacer-x-small"
-        size="small"
-        @click="$emit('fetchAllConnections', type)"
-      >
-        {{
-          $t('profile.network.andMore', {
-            number: allConnectionsCount - connections.length,
-          })
-        }}
-      </base-button>
-      <ds-input
-        v-if="!hasMore"
-        :name="`${type}Filter`"
-        :placeholder="filter"
-        class="spacer-x-small"
-        icon="filter"
-        size="small"
-        @input.native="setFilter"
-      />
-    </template>
-    <p v-else class="nobody-message">{{ userName }} {{ $t(`profile.network.${type}Nobody`) }}</p>
-  </base-card>
+  <profile-list
+    :uniqueName="`${type}Filter`"
+    :title="$filters.truncate(userName, 15) + ' ' + $t(`profile.network.${type}`)"
+    :titleNobody="$filters.truncate(userName, 15) + ' ' + $t(`profile.network.${type}Nobody`)"
+    :allConnectionsCount="allConnectionsCount"
+    :connections="connections"
+    :loading="loading"
+    @fetchAllConnections="$emit('fetchAllConnections', type)"
+  />
 </template>
 
 <script>
-import { escape } from 'xregexp/xregexp-all.js'
-import UserTeaser from '~/components/UserTeaser/UserTeaser'
+import ProfileList from '~/components/features/ProfileList/ProfileList'
 
 export default {
   name: 'FollowerList',
   components: {
-    UserTeaser,
+    ProfileList,
   },
   props: {
     user: { type: Object, default: null },
     type: { type: String, default: 'following' },
     loading: { type: Boolean, default: false },
   },
-  data() {
-    return {
-      filter: null,
-    }
-  },
   computed: {
     userName() {
       const { name } = this.user || {}
@@ -70,95 +34,6 @@ export default {
     connections() {
       return this.user[this.type]
     },
-    hasMore() {
-      return this.allConnectionsCount > this.connections.length
-    },
-    connectionsClass() {
-      return `connections${this.hasMore ? '' : ' --overflow'}`
-    },
-    filteredConnections() {
-      if (!this.filter) {
-        return this.connections
-      }
-
-      // @example
-      //  this.filter = 'foo';
-      //  fuzzyExpression = /([^f]*f)([^o]*o)([^o]*o)/i
-      const fuzzyExpression = new RegExp(
-        `${this.filter.split('').reduce((expr, c) => `${expr}([^${escape(c)}]*${escape(c)})`, '')}`,
-        'i',
-      )
-
-      const fuzzyScores = this.connections
-        .map((user) => {
-          const match = user.name.match(fuzzyExpression)
-
-          if (!match) {
-            return false
-          }
-
-          let score = 1
-          for (let i = 1; i <= this.filter.length; i++) {
-            score *= match[i].length
-          }
-
-          return {
-            user,
-            score,
-          }
-        })
-        .filter(Boolean)
-        .sort((a, b) => a.score - b.score)
-
-      return fuzzyScores.map((score) => score.user)
-    },
-  },
-  methods: {
-    setFilter(evt) {
-      this.filter = evt.target.value
-    },
   },
 }
 </script>
-
-<style lang="scss">
-.follow-list {
-  display: flex;
-  flex-direction: column;
-  position: relative;
-  width: auto;
-
-  > .title {
-    color: $text-color-soft;
-    font-size: $font-size-base;
-  }
-
-  .connections {
-    height: $size-height-connections;
-    padding: $space-none;
-    list-style-type: none;
-
-    &.--overflow {
-      overflow-y: auto;
-    }
-
-    > .connections__item {
-      padding: $space-xx-small;
-
-      &.is-selected,
-      &:hover {
-        background-color: $background-color-primary-inverse;
-      }
-    }
-  }
-
-  .nobody-message {
-    text-align: center;
-    color: $text-color-soft;
-  }
-
-  > :nth-child(n):not(:last-child) {
-    margin-bottom: $space-small;
-  }
-}
-</style>
diff --git a/webapp/components/features/ProfileList/ProfileList.vue b/webapp/components/features/ProfileList/ProfileList.vue
new file mode 100644
index 000000000..c619da8fc
--- /dev/null
+++ b/webapp/components/features/ProfileList/ProfileList.vue
@@ -0,0 +1,158 @@
+<template>
+  <base-card class="follow-list">
+    <template v-if="connections && connections.length">
+      <h5 class="title spacer-x-small">
+        {{ title }}
+      </h5>
+      <ul :class="connectionsClass">
+        <li
+          v-for="connection in filteredConnections"
+          :key="connection.id"
+          class="connections__item"
+        >
+          <user-teaser :user="connection" />
+        </li>
+      </ul>
+      <base-button
+        v-if="hasMore"
+        :loading="loading"
+        class="spacer-x-small"
+        size="small"
+        @click="$emit('fetchAllConnections')"
+      >
+        {{
+          $t('profile.network.andMore', {
+            number: allConnectionsCount - connections.length,
+          })
+        }}
+      </base-button>
+      <ds-input
+        v-if="!hasMore"
+        :name="uniqueName"
+        :placeholder="filter"
+        class="spacer-x-small"
+        icon="filter"
+        size="small"
+        @input.native="setFilter"
+      />
+    </template>
+    <p v-else class="nobody-message">{{ titleNobody }}</p>
+  </base-card>
+</template>
+
+<script>
+import { escape } from 'xregexp/xregexp-all.js'
+import UserTeaser from '~/components/UserTeaser/UserTeaser'
+
+export default {
+  name: 'ProfileList',
+  components: {
+    UserTeaser,
+  },
+  props: {
+    uniqueName: { type: String, required: true },
+    title: { type: String, required: true },
+    titleNobody: { type: String, required: true },
+    allConnectionsCount: { type: Number, required: true },
+    connections: { type: Array, required: true },
+    type: { type: String, default: 'following' },
+    loading: { type: Boolean, default: false },
+  },
+  data() {
+    return {
+      filter: null,
+    }
+  },
+  computed: {
+    hasMore() {
+      return this.allConnectionsCount > this.connections.length
+    },
+    connectionsClass() {
+      return `connections${this.hasMore ? '' : ' --overflow'}`
+    },
+    filteredConnections() {
+      if (!this.filter) {
+        return this.connections
+      }
+
+      // @example
+      //  this.filter = 'foo';
+      //  fuzzyExpression = /([^f]*f)([^o]*o)([^o]*o)/i
+      const fuzzyExpression = new RegExp(
+        `${this.filter.split('').reduce((expr, c) => `${expr}([^${escape(c)}]*${escape(c)})`, '')}`,
+        'i',
+      )
+
+      const fuzzyScores = this.connections
+        .map((user) => {
+          const match = user.name.match(fuzzyExpression)
+
+          if (!match) {
+            return false
+          }
+
+          let score = 1
+          for (let i = 1; i <= this.filter.length; i++) {
+            score *= match[i].length
+          }
+
+          return {
+            user,
+            score,
+          }
+        })
+        .filter(Boolean)
+        .sort((a, b) => a.score - b.score)
+
+      return fuzzyScores.map((score) => score.user)
+    },
+  },
+  methods: {
+    setFilter(evt) {
+      this.filter = evt.target.value
+    },
+  },
+}
+</script>
+
+<style lang="scss">
+.follow-list {
+  display: flex;
+  flex-direction: column;
+  position: relative;
+  width: auto;
+
+  > .title {
+    color: $text-color-soft;
+    font-size: $font-size-base;
+  }
+
+  .connections {
+    height: $size-height-connections;
+    padding: $space-none;
+    list-style-type: none;
+
+    &.--overflow {
+      overflow-y: auto;
+    }
+
+    > .connections__item {
+      padding: $space-xx-small;
+
+      &.is-selected,
+      &:hover {
+        background-color: $background-color-primary-inverse;
+      }
+    }
+  }
+
+  .nobody-message {
+    text-align: center;
+    color: $text-color-soft;
+  }
+
+  > :nth-child(n):not(:last-child) {
+    margin-bottom: $space-small;
+  }
+}
+</style>