From ea53553e913f3bb4303aa28c734b586d7572da7e Mon Sep 17 00:00:00 2001
From: Jonas Sulzer <jonas@violoncello.ch>
Date: Tue, 17 Sep 2019 18:36:11 +0200
Subject: [PATCH 1/6] =?UTF-8?q?=F0=9F=91=8C=20IMPROVE:=20move=20timeline?=
 =?UTF-8?q?=20to=20css=20grid=20layout=20and=20restructure=20code?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Jonas Sulzer <jonas@violoncello.ch>
---
 src/components/TimelineAvatar.vue |  47 +++++++++++++
 src/components/TimelineEntry.vue  |  78 ++++++++++++++++++----
 src/components/TimelinePost.vue   | 105 ++++++++++++------------------
 webpack.common.js                 |   6 +-
 4 files changed, 157 insertions(+), 79 deletions(-)
 create mode 100644 src/components/TimelineAvatar.vue

diff --git a/src/components/TimelineAvatar.vue b/src/components/TimelineAvatar.vue
new file mode 100644
index 00000000..ae395b64
--- /dev/null
+++ b/src/components/TimelineAvatar.vue
@@ -0,0 +1,47 @@
+<template>
+	<div v-if="item.actor_info" class="post-avatar">
+		<avatar v-if="item.local" :size="32" :user="userTest"
+			:display-name="item.actor_info.account" :disable-tooltip="true" />
+		<avatar v-else :size="32" :url="avatarUrl"
+			:disable-tooltip="true" />
+	</div>
+</template>
+
+<script>
+import Avatar from '@nextcloud/vue/dist/Components/Avatar'
+
+export default {
+	name: 'TimelineAvatar',
+	components: {
+		Avatar
+	},
+	props: {
+		item: { type: Object, default: () => {} }
+	},
+	computed: {
+		userTest() {
+			return this.item.actor_info.preferredUsername
+		},
+		avatarUrl() {
+			return OC.generateUrl('/apps/social/api/v1/global/actor/avatar?id=' + this.item.attributedTo)
+		}
+	}
+}
+</script>
+
+<style lang="scss" scoped>
+.post-avatar {
+    margin: 5px;
+    margin-right: 10px;
+    border-radius: 50%;
+    overflow: hidden;
+    width: 32px;
+    height: 32px;
+    min-width: 32px;
+    flex-shrink: 0;
+    grid-column: 1;
+    grid-row: 2;
+    align-self: start;
+}
+
+</style>
diff --git a/src/components/TimelineEntry.vue b/src/components/TimelineEntry.vue
index 61833343..db83637f 100644
--- a/src/components/TimelineEntry.vue
+++ b/src/components/TimelineEntry.vue
@@ -1,9 +1,12 @@
 <template>
 	<div class="timeline-entry">
-		<div v-if="item.type === 'SocialAppNotification'">
-			{{ actionSummary }}
-		</div>
-		<div v-if="item.type === 'Announce'" class="boost">
+		<template v-if="item.type === 'SocialAppNotification'">
+			<div class="notification-icon" :class="notificationIcon" />
+			<span class="notification-action">
+				{{ actionSummary }}
+			</span>
+		</template>
+		<template v-else-if="item.type === 'Announce'" class="boost">
 			<div class="container-icon-boost">
 				<span class="icon-boost" />
 			</div>
@@ -18,24 +21,28 @@
 				</span>
 			</a>
 			{{ boosted }}
-		</div>
-		<timeline-post
-			v-if="item.type === 'SocialAppNotification' && item.details.post"
-			:item="item.details.post" />
-		<timeline-post
-			v-else
-			:item="entryContent"
-			:parent-announce="isBoost" />
+		</template>
+		<user-entry v-if="item.type === 'SocialAppNotification' && item.details.actor" :key="item.details.actor.id" :item="item.details.actor" />
+		<template v-else>
+			<timeline-avatar :item="entryContent" />
+			<timeline-post
+				:item="entryContent"
+				:parent-announce="isBoost" />
+		</template>
 	</div>
 </template>
 
 <script>
 import TimelinePost from './TimelinePost.vue'
+import TimelineAvatar from './TimelineAvatar.vue'
+import UserEntry from './UserEntry.vue'
 
 export default {
 	name: 'TimelineEntry',
 	components: {
-		TimelinePost
+		TimelinePost,
+		TimelineAvatar,
+		UserEntry
 	},
 	props: {
 		item: { type: Object, default: () => {} }
@@ -48,6 +55,8 @@ export default {
 		entryContent() {
 			if (this.item.type === 'Announce') {
 				return this.item.cache[this.item.object].object
+			} else if (this.item.type === 'SocialAppNotification') {
+				return this.item.details.post
 			} else {
 				return this.item
 			}
@@ -102,12 +111,55 @@ export default {
 </script>
 <style scoped lang="scss">
 	.timeline-entry {
+		display: grid;
+    grid-template-columns: 44px 1fr;
+    grid-template-rows: 30px 1fr;
 		padding: 10px;
 		margin-bottom: 10px;
 		&:hover {
 			background-color: var(--color-background-hover);
 		}
 	}
+	.notification-header {
+		display: flex;
+		align-items: bottom;
+	}
+
+	.notification-action {
+		flex-grow: 1;
+		display: inline-block;
+		grid-row: 1;
+		grid-column: 2;
+	}
+
+	.notification-icon {
+		opacity: .5;
+		background-position: center;
+		background-size: contain;
+		overflow: hidden;
+		height: 20px;
+		min-width: 32px;
+		flex-shrink: 0;
+		display: inline-block;
+		vertical-align: middle;
+		grid-column: 1;
+		grid-row: 1;
+	}
+
+	.icon-boost {
+		display: inline-block;
+		vertical-align: middle;
+	}
+
+	.icon-favorite {
+		display: inline-block;
+		vertical-align: middle;
+	}
+
+	.icon-user {
+		display: inline-block;
+		vertical-align: middle;
+	}
 
 	.container-icon-boost {
 		display: inline-block;
diff --git a/src/components/TimelinePost.vue b/src/components/TimelinePost.vue
index 411dc5ff..1a124451 100644
--- a/src/components/TimelinePost.vue
+++ b/src/components/TimelinePost.vue
@@ -1,55 +1,47 @@
 <template>
-	<div class="entry-content">
-		<div v-if="item.actor_info" class="post-avatar">
-			<avatar v-if="item.local && item.type!=='SocialAppNotification'" :size="32" :user="item.actor_info.preferredUsername"
-				:display-name="item.actor_info.account" :disable-tooltip="true" />
-			<avatar v-else :size="32" :url="avatarUrl"
-				:disable-tooltip="true" />
-		</div>
-		<div class="post-content">
-			<div class="post-header">
-				<div class="post-author-wrapper">
-					<router-link v-if="item.actor_info"
-						:to="{ name: 'profile',
-							params: { account: (item.local && item.type!=='SocialAppNotification') ? item.actor_info.preferredUsername : item.actor_info.account }
-						}">
-						<span class="post-author">
-							{{ userDisplayName(item.actor_info) }}
-						</span>
-						<span class="post-author-id">
-							@{{ item.actor_info.account }}
-						</span>
-					</router-link>
-					<a v-else :href="item.attributedTo">
-						<span class="post-author-id">
-							{{ item.attributedTo }}
-						</span>
-					</a>
-				</div>
-				<a :data-timestamp="timestamp" class="post-timestamp live-relative-timestamp" @click="getSinglePostTimeline">
-					{{ relativeTimestamp }}
+	<div class="post-content">
+		<div class="post-header">
+			<div class="post-author-wrapper">
+				<router-link v-if="item.actor_info"
+					:to="{ name: 'profile',
+						params: { account: (item.local && item.type!=='SocialAppNotification') ? item.actor_info.preferredUsername : item.actor_info.account }
+					}">
+					<span class="post-author">
+						{{ userDisplayName(item.actor_info) }}
+					</span>
+					<span class="post-author-id">
+						@{{ item.actor_info.account }}
+					</span>
+				</router-link>
+				<a v-else :href="item.attributedTo">
+					<span class="post-author-id">
+						{{ item.attributedTo }}
+					</span>
 				</a>
 			</div>
-			<!-- eslint-disable-next-line vue/no-v-html -->
-			<div v-if="item.content" class="post-message">
-				<MessageContent :source="source" />
-			</div>
-			<!-- eslint-disable-next-line vue/no-v-html -->
-			<div v-else class="post-message" v-html="item.actor_info.summary" />
-			<div v-if="hasAttachments" class="post-attachments">
-				<post-attachment :attachments="item.attachment" />
-			</div>
-			<div v-if="this.$route.params.type!=='notifications' && !serverData.public" v-click-outside="hidePopoverMenu" class="post-actions">
-				<a v-tooltip.bottom="t('social', 'Reply')" class="icon-reply" @click.prevent="reply" />
-				<a v-if="item.actor_info.account !== cloudId" v-tooltip.bottom="t('social', 'Boost')"
-					:class="(isBoosted) ? 'icon-boosted' : 'icon-boost'"
-					@click.prevent="boost" />
-				<a v-tooltip.bottom="t('social', 'Like')" :class="(isLiked) ? 'icon-starred' : 'icon-favorite'" @click.prevent="like" />
-				<div v-if="popoverMenu.length > 0" v-tooltip.bottom="menuOpened ? '' : t('social', 'More actions')" class="post-actions-more">
-					<a class="icon-more" @click.prevent="togglePopoverMenu" />
-					<div :class="{open: menuOpened}" class="popovermenu menu-center">
-						<popover-menu :menu="popoverMenu" />
-					</div>
+			<a :data-timestamp="timestamp" class="post-timestamp live-relative-timestamp" @click="getSinglePostTimeline">
+				{{ relativeTimestamp }}
+			</a>
+		</div>
+		<!-- eslint-disable-next-line vue/no-v-html -->
+		<div v-if="item.content" class="post-message">
+			<MessageContent :source="source" />
+		</div>
+		<!-- eslint-disable-next-line vue/no-v-html -->
+		<div v-else class="post-message" v-html="item.actor_info.summary" />
+		<div v-if="hasAttachments" class="post-attachments">
+			<post-attachment :attachments="item.attachment" />
+		</div>
+		<div v-if="this.$route.params.type!=='notifications' && !serverData.public" v-click-outside="hidePopoverMenu" class="post-actions">
+			<a v-tooltip.bottom="t('social', 'Reply')" class="icon-reply" @click.prevent="reply" />
+			<a v-if="item.actor_info.account !== cloudId" v-tooltip.bottom="t('social', 'Boost')"
+				:class="(isBoosted) ? 'icon-boosted' : 'icon-boost'"
+				@click.prevent="boost" />
+			<a v-tooltip.bottom="t('social', 'Like')" :class="(isLiked) ? 'icon-starred' : 'icon-favorite'" @click.prevent="like" />
+			<div v-if="popoverMenu.length > 0" v-tooltip.bottom="menuOpened ? '' : t('social', 'More actions')" class="post-actions-more">
+				<a class="icon-more" @click.prevent="togglePopoverMenu" />
+				<div :class="{open: menuOpened}" class="popovermenu menu-center">
+					<popover-menu :menu="popoverMenu" />
 				</div>
 			</div>
 		</div>
@@ -57,7 +49,6 @@
 </template>
 
 <script>
-import Avatar from '@nextcloud/vue/dist/Components/Avatar'
 import * as linkify from 'linkifyjs'
 import pluginMention from 'linkifyjs/plugins/mention'
 import 'linkifyjs/string'
@@ -74,7 +65,6 @@ pluginMention(linkify)
 export default {
 	name: 'TimelinePost',
 	components: {
-		Avatar,
 		PostAttachment,
 		MessageContent
 	},
@@ -207,17 +197,6 @@ export default {
 		opacity: .7;
 	}
 
-	.post-avatar {
-		margin: 5px;
-		margin-right: 10px;
-		border-radius: 50%;
-		overflow: hidden;
-		width: 32px;
-		height: 32px;
-		min-width: 32px;
-		flex-shrink: 0;
-	}
-
 	.post-timestamp {
 		width: 120px;
 		text-align: right;
@@ -262,6 +241,8 @@ export default {
 
 	.post-content {
 		flex-grow: 1;
+    grid-column: 2;
+    grid-row: 2;
 	}
 
 	.post-header {
diff --git a/webpack.common.js b/webpack.common.js
index db2fadb7..3f7ba387 100644
--- a/webpack.common.js
+++ b/webpack.common.js
@@ -50,9 +50,7 @@ module.exports = {
 	},
 	plugins: [new VueLoaderPlugin()],
 	resolve: {
-		alias: {
-			vue$: 'vue/dist/vue.esm.js'
-		},
-		extensions: ['*', '.js', '.vue', '.json']
+		extensions: ['*', '.js', '.vue'],
+		symlinks: false
 	}
 };

From 4f127bd298c5392ca5bb22c9fa4a442d05b0da5a Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Thu, 8 Oct 2020 15:09:44 +0200
Subject: [PATCH 2/6] fix boosted layout

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 src/components/TimelineEntry.vue | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/src/components/TimelineEntry.vue b/src/components/TimelineEntry.vue
index db83637f..cb46bd29 100644
--- a/src/components/TimelineEntry.vue
+++ b/src/components/TimelineEntry.vue
@@ -6,21 +6,23 @@
 				{{ actionSummary }}
 			</span>
 		</template>
-		<template v-else-if="item.type === 'Announce'" class="boost">
-			<div class="container-icon-boost">
+		<template v-else-if="item.type === 'Announce'">
+			<div class="container-icon-boost boost">
 				<span class="icon-boost" />
 			</div>
-			<router-link v-if="item.actor_info" :to="{ name: 'profile', params: { account: item.local ? item.actor_info.preferredUsername : item.actor_info.account }}">
-				<span v-tooltip.bottom="item.actor_info.account" class="post-author">
-					{{ userDisplayName(item.actor_info) }}
-				</span>
-			</router-link>
-			<a v-else :href="item.attributedTo">
-				<span class="post-author-id">
-					{{ item.attributedTo }}
-				</span>
-			</a>
-			{{ boosted }}
+			<div class="boost">
+				<router-link v-if="item.actor_info" :to="{ name: 'profile', params: { account: item.local ? item.actor_info.preferredUsername : item.actor_info.account }}">
+					<span v-tooltip.bottom="item.actor_info.account" class="post-author">
+						{{ userDisplayName(item.actor_info) }}
+					</span>
+				</router-link>
+				<a v-else :href="item.attributedTo">
+					<span class="post-author-id">
+						{{ item.attributedTo }}
+					</span>
+				</a>
+				{{ boosted }}
+			</div>
 		</template>
 		<user-entry v-if="item.type === 'SocialAppNotification' && item.details.actor" :key="item.details.actor.id" :item="item.details.actor" />
 		<template v-else>

From 6e7cb67c934acc5092263196fe77b5a8c447e385 Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Thu, 8 Oct 2020 15:25:25 +0200
Subject: [PATCH 3/6] fix alignment for posts without boosts

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 src/components/TimelineAvatar.vue |  3 ---
 src/components/TimelineEntry.vue  | 10 ++++++++--
 src/components/TimelinePost.vue   |  6 ------
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/src/components/TimelineAvatar.vue b/src/components/TimelineAvatar.vue
index ae395b64..25104274 100644
--- a/src/components/TimelineAvatar.vue
+++ b/src/components/TimelineAvatar.vue
@@ -38,9 +38,6 @@ export default {
     width: 32px;
     height: 32px;
     min-width: 32px;
-    flex-shrink: 0;
-    grid-column: 1;
-    grid-row: 2;
     align-self: start;
 }
 
diff --git a/src/components/TimelineEntry.vue b/src/components/TimelineEntry.vue
index cb46bd29..3576b2ef 100644
--- a/src/components/TimelineEntry.vue
+++ b/src/components/TimelineEntry.vue
@@ -1,5 +1,5 @@
 <template>
-	<div class="timeline-entry">
+	<div :class="['timeline-entry', hasHeader ? 'with-header' : '']" @click="getSinglePostTimeline">
 		<template v-if="item.type === 'SocialAppNotification'">
 			<div class="notification-icon" :class="notificationIcon" />
 			<span class="notification-action">
@@ -69,6 +69,9 @@ export default {
 			}
 			return {}
 		},
+		hasHeader() {
+			return this.item.type === 'Announce' || this.item.type === 'SocialAppNotification'
+		},
 		boosted() {
 			return t('social', 'boosted')
 		},
@@ -112,10 +115,13 @@ export default {
 }
 </script>
 <style scoped lang="scss">
+  .timeline-entry.with-header {
+    grid-template-rows: 30px 1fr;
+  }
 	.timeline-entry {
 		display: grid;
     grid-template-columns: 44px 1fr;
-    grid-template-rows: 30px 1fr;
+    grid-template-rows: 1fr;
 		padding: 10px;
 		margin-bottom: 10px;
 		&:hover {
diff --git a/src/components/TimelinePost.vue b/src/components/TimelinePost.vue
index 1a124451..2cfe98c4 100644
--- a/src/components/TimelinePost.vue
+++ b/src/components/TimelinePost.vue
@@ -239,12 +239,6 @@ export default {
 		display: flex;
 	}
 
-	.post-content {
-		flex-grow: 1;
-    grid-column: 2;
-    grid-row: 2;
-	}
-
 	.post-header {
 		display: flex;
 		flex-direction: row;

From ef8c6f81fc74bd07670a6b6d31bea8cbef53a86e Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Wed, 14 Oct 2020 17:40:20 +0200
Subject: [PATCH 4/6] fix js

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 src/components/MessageContent.js | 3 +++
 src/components/TimelineEntry.vue | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/components/MessageContent.js b/src/components/MessageContent.js
index ff4e23cf..d8fe78a8 100644
--- a/src/components/MessageContent.js
+++ b/src/components/MessageContent.js
@@ -24,6 +24,9 @@ export default Vue.component('MessageContent', {
  * All attributes other than `href` for links are stripped from the source
  */
 export function formatMessage(createElement, source) {
+	if (!source.tag) {
+		source.tag = []
+	}
 	let mentions = source.tag.filter(tag => tag.type === 'Mention')
 	let hashtags = source.tag.filter(tag => tag.type === 'Hashtag')
 
diff --git a/src/components/TimelineEntry.vue b/src/components/TimelineEntry.vue
index 3576b2ef..97f714c6 100644
--- a/src/components/TimelineEntry.vue
+++ b/src/components/TimelineEntry.vue
@@ -1,5 +1,5 @@
 <template>
-	<div :class="['timeline-entry', hasHeader ? 'with-header' : '']" @click="getSinglePostTimeline">
+	<div :class="['timeline-entry', hasHeader ? 'with-header' : '']">
 		<template v-if="item.type === 'SocialAppNotification'">
 			<div class="notification-icon" :class="notificationIcon" />
 			<span class="notification-action">

From ba1adcef521f5e70af82e931bdd1b4cbac09e928 Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Wed, 14 Oct 2020 17:41:51 +0200
Subject: [PATCH 5/6] add route for profile page with trailing slash

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 appinfo/routes.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/appinfo/routes.php b/appinfo/routes.php
index 077c37ee..f97e1ebd 100644
--- a/appinfo/routes.php
+++ b/appinfo/routes.php
@@ -51,6 +51,7 @@ return [
 
 		['name' => 'ActivityPub#actor', 'url' => '/users/{username}', 'verb' => 'GET'],
 		['name' => 'ActivityPub#actorAlias', 'url' => '/@{username}', 'verb' => 'GET'],
+		['name' => 'ActivityPub#actorAlias', 'url' => '/@{username}/', 'verb' => 'GET'],
 		['name' => 'ActivityPub#inbox', 'url' => '/@{username}/inbox', 'verb' => 'POST'],
 		['name' => 'ActivityPub#getInbox', 'url' => '/@{username}/inbox', 'verb' => 'GET'],
 		['name' => 'ActivityPub#sharedInbox', 'url' => '/inbox', 'verb' => 'POST'],

From ccb6196a71a2fc2bc4db877a53331db1ec919885 Mon Sep 17 00:00:00 2001
From: Maxence Lange <maxence@artificial-owl.com>
Date: Mon, 19 Oct 2020 09:56:19 -0100
Subject: [PATCH 6/6] duplicate

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
---
 appinfo/routes.php | 1 -
 1 file changed, 1 deletion(-)

diff --git a/appinfo/routes.php b/appinfo/routes.php
index f97e1ebd..f88528b8 100644
--- a/appinfo/routes.php
+++ b/appinfo/routes.php
@@ -50,7 +50,6 @@ return [
 		['name' => 'Navigation#resizedGetPublic', 'url' => '/document/public/resized', 'verb' => 'GET'],
 
 		['name' => 'ActivityPub#actor', 'url' => '/users/{username}', 'verb' => 'GET'],
-		['name' => 'ActivityPub#actorAlias', 'url' => '/@{username}', 'verb' => 'GET'],
 		['name' => 'ActivityPub#actorAlias', 'url' => '/@{username}/', 'verb' => 'GET'],
 		['name' => 'ActivityPub#inbox', 'url' => '/@{username}/inbox', 'verb' => 'POST'],
 		['name' => 'ActivityPub#getInbox', 'url' => '/@{username}/inbox', 'verb' => 'GET'],