From 30b9af87e296790de8fa6fcf614c1deca6bf4766 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Fri, 31 Jan 2020 12:24:09 +0100
Subject: [PATCH] Use RichText component for activity
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 src/components/ActivityEntry.vue | 100 ++++++++++++++++++++++++++-----
 1 file changed, 84 insertions(+), 16 deletions(-)

diff --git a/src/components/ActivityEntry.vue b/src/components/ActivityEntry.vue
index 84e299056..f8b62ef94 100644
--- a/src/components/ActivityEntry.vue
+++ b/src/components/ActivityEntry.vue
@@ -22,35 +22,95 @@
 
 <template>
 	<div v-if="activity" class="activity">
-		<img :src="activity.icon" class="activity--icon">
-		<div class="activity--message" v-html="parseMessage(activity)" />
-		<div class="activity--timestamp">
-			{{ getTime(activity.datetime) }}
+		<div class="activity--header">
+			<img :src="activity.icon" class="activity--icon">
+			<RichText class="activity--subject" :text="message.subject" :arguments="message.parameters" />
+			<div class="activity--timestamp">
+				{{ getTime(activity.datetime) }}
+			</div>
 		</div>
+		<p v-if="activity.message" class="activity--message">
+			{{ activity.message }}
+		</p>
 	</div>
 </template>
 
 <script>
+import RichText from '@juliushaertl/vue-richtext'
+import { UserBubble } from '@nextcloud/vue'
+
+const InternalLink = {
+	name: 'InternalLink',
+	functional: true,
+	props: {
+		href: {
+			type: String,
+			default: '',
+		},
+		name: {
+			type: String,
+			default: '',
+		},
+	},
+	render(createElement, context) {
+		return createElement('a', { attrs: { href: context.props.href }, style: { 'font-weight': 600 } }, context.props.name)
+	},
+}
 export default {
 	name: 'ActivityEntry',
+	components: {
+		RichText,
+	},
 	props: {
 		activity: {
 			type: Object,
 			default: null,
 		},
 	},
-	methods: {
-		getTime(timestamp) {
-			return OC.Util.relativeModifiedDate(timestamp)
-		},
-		parseMessage(activity) {
-			const subject = activity.subject_rich[0]
-			const parameters = JSON.parse(JSON.stringify(activity.subject_rich[1]))
+	computed: {
+		message() {
+			const subject = this.activity.subject_rich[0]
+			const parameters = JSON.parse(JSON.stringify(this.activity.subject_rich[1]))
 			if (parameters.after && typeof parameters.after.id === 'string' && parameters.after.id.startsWith('dt:')) {
 				const dateTime = parameters.after.id.substr(3)
 				parameters.after.name = window.moment(dateTime).format('L LTS')
 			}
-			return OCA.Activity.RichObjectStringParser.parseMessage(subject, parameters)
+
+			Object.keys(parameters).map(function(key, index) {
+				const { type } = parameters[key]
+				switch (type) {
+				case 'highlight':
+					parameters[key] = {
+						component: InternalLink,
+						props: {
+							href: parameters[key].link,
+							name: parameters[key].name,
+						},
+					}
+					break
+				case 'user':
+					parameters[key] = {
+						component: UserBubble,
+						props: {
+							user: parameters[key].id,
+							displayName: parameters[key].name,
+						},
+					}
+					break
+				default:
+					parameters[key] = `{${key}}`
+				}
+
+			})
+
+			return {
+				subject, parameters,
+			}
+		},
+	},
+	methods: {
+		getTime(timestamp) {
+			return OC.Util.relativeModifiedDate(timestamp)
 		},
 	},
 }
@@ -58,8 +118,11 @@ export default {
 
 <style scoped lang="scss">
 	.activity {
-		display: flex;
-		padding: 10px;
+
+		.activity--header {
+			display: flex;
+			padding: 10px;
+		}
 
 		.activity--icon {
 			width: 16px;
@@ -67,14 +130,19 @@ export default {
 			flex-shrink: 0;
 			flex-grow: 0;
 		}
-		.activity--message {
+		.activity--subject {
 			margin-left: 10px;
 		}
+		.activity--message {
+			margin-left: 44px;
+			color: var(--color-text-light);
+			margin-bottom: 10px;
+		}
 		.activity--timestamp {
+			flex-grow: 1;
 			color: var(--color-text-maxcontrast);
 			text-align: right;
 			font-size: 0.8em;
-			width: 25%;
 			padding: 1px;
 		}
 	}