From a5e3c52e1d9e1da77177ae2e45c5f4d3e4ea7f8c Mon Sep 17 00:00:00 2001 From: mattwr18 Date: Thu, 3 Oct 2019 12:01:04 +0200 Subject: [PATCH 1/3] Show that a Post/Comment has been edited --- webapp/components/Comment/Comment.vue | 43 +++++++++++++++++++-------- webapp/graphql/CommentMutations.js | 2 ++ webapp/graphql/Fragments.js | 2 ++ webapp/locales/de.json | 10 ++++--- webapp/locales/en.json | 6 ++-- webapp/locales/pt.json | 22 +++++++++++--- webapp/pages/post/_id/_slug/index.vue | 33 +++++++++++++------- 7 files changed, 85 insertions(+), 33 deletions(-) diff --git a/webapp/components/Comment/Comment.vue b/webapp/components/Comment/Comment.vue index 31bbc811f..6fc076c7b 100644 --- a/webapp/components/Comment/Comment.vue +++ b/webapp/components/Comment/Comment.vue @@ -12,20 +12,33 @@
- + + + + + + + {{ this.$t('comment.edited') }} + + + + + + + + + + - - -
diff --git a/webapp/graphql/CommentMutations.js b/webapp/graphql/CommentMutations.js index dd8d1d3a0..133b61f18 100644 --- a/webapp/graphql/CommentMutations.js +++ b/webapp/graphql/CommentMutations.js @@ -10,6 +10,7 @@ export default i18n => { contentExcerpt content createdAt + updatedAt disabled deleted author { @@ -39,6 +40,7 @@ export default i18n => { contentExcerpt content createdAt + updatedAt disabled deleted author { diff --git a/webapp/graphql/Fragments.js b/webapp/graphql/Fragments.js index e50d4bbe1..e0c6e699e 100644 --- a/webapp/graphql/Fragments.js +++ b/webapp/graphql/Fragments.js @@ -40,6 +40,7 @@ export const postFragment = lang => gql` content contentExcerpt createdAt + updatedAt disabled deleted slug @@ -64,6 +65,7 @@ export const commentFragment = lang => gql` fragment comment on Comment { id createdAt + updatedAt disabled deleted content diff --git a/webapp/locales/de.json b/webapp/locales/de.json index c547e063f..b6ee3f320 100644 --- a/webapp/locales/de.json +++ b/webapp/locales/de.json @@ -2,7 +2,7 @@ "maintenance": { "title": "Human Connection befindet sich in der Wartung", "explanation": "Zurzeit führen wir einige geplante Wartungsarbeiten durch, bitte versuch es später erneut.", - "questions": "Bei Fragen oder Problemen erreichst du uns per E-Mail an" + "questions": "Bei Fragen oder Problemen erreichst du uns per E-Mail an" }, "index": { "no-results": "Keine Beiträge gefunden.", @@ -335,7 +335,8 @@ "submit": "Kommentiere", "submitted": "Kommentar Gesendet", "updated": "Änderungen gespeichert" - } + }, + "edited": "bearbeitet" }, "comment": { "content": { @@ -348,10 +349,11 @@ "show": { "more": "mehr anzeigen", "less": "weniger anzeigen" - } + }, + "edited": "bearbeitet" }, "quotes": { - "african": { + "african": { "quote": "Viele kleine Leute an vielen kleinen Orten, die viele kleine Dinge tun, werden das Antlitz dieser Welt verändern.", "author": "Afrikanisches Sprichwort" } diff --git a/webapp/locales/en.json b/webapp/locales/en.json index 48fd36c0d..b243e39a3 100644 --- a/webapp/locales/en.json +++ b/webapp/locales/en.json @@ -336,7 +336,8 @@ "submit": "Comment", "submitted": "Comment Submitted", "updated": "Changes Saved" - } + }, + "edited": "edited" }, "comment": { "content": { @@ -349,7 +350,8 @@ "show": { "more": "show more", "less": "show less" - } + }, + "edited": "edited" }, "quotes": { "african": { diff --git a/webapp/locales/pt.json b/webapp/locales/pt.json index d08d569c2..b43711998 100644 --- a/webapp/locales/pt.json +++ b/webapp/locales/pt.json @@ -118,7 +118,11 @@ }, "takeAction": { "name": "Tomar uma ação" - } + }, + "comment": { + "submit": "Commentar" + }, + "edited": "editado" }, "quotes": { "african": { @@ -202,8 +206,18 @@ "delete": "Apagar Contribuição" }, "comment": { - "edit": "Editar Comentário", - "delete": "Apagar Comentário" + "content": { + "unavailable-placeholder": "… este commenttário não está disponível" + }, + "menu": { + "edit": "Editar Comentário", + "delete": "Apagar Comentário" + }, + "show": { + "more": "mostrar mais", + "less": "mostrar menos" + }, + "edited": "editado" }, "followButton": { "follow": "Seguir", @@ -212,4 +226,4 @@ "shoutButton": { "shouted": "Aclamou" } -} \ No newline at end of file +} diff --git a/webapp/pages/post/_id/_slug/index.vue b/webapp/pages/post/_id/_slug/index.vue index 0fda144d0..0c45c0ca3 100644 --- a/webapp/pages/post/_id/_slug/index.vue +++ b/webapp/pages/post/_id/_slug/index.vue @@ -6,17 +6,30 @@ :class="{ 'post-card': true, 'disabled-content': post.disabled }" > - + + + + + + + {{ this.$t('post.edited') }} + + + + + + + + + + - - - {{ post.title }} From 3912b21ea2f24e2e25682060b7166d1511442e6e Mon Sep 17 00:00:00 2001 From: roschaefer Date: Fri, 4 Oct 2019 00:55:14 +0200 Subject: [PATCH 2/3] refactor: use named slot for additional text @mattwr18 I can see odd behaviour of the EDITED marker on mobile: The text would just overlap. The current behaviour has its flaws, e.g. if you have a long text it won't wrap but overflow the comment card. But putting the additional text into a named slot is certainly better than adding some nodes next to the `` component. --- webapp/components/Comment/Comment.vue | 48 ++++++++++----------------- webapp/components/User/User.vue | 4 +-- webapp/pages/post/_id/_slug/index.vue | 38 ++++++++------------- 3 files changed, 34 insertions(+), 56 deletions(-) diff --git a/webapp/components/Comment/Comment.vue b/webapp/components/Comment/Comment.vue index 6fc076c7b..534c45e6b 100644 --- a/webapp/components/Comment/Comment.vue +++ b/webapp/components/Comment/Comment.vue @@ -12,33 +12,25 @@
- - - - - - - {{ this.$t('comment.edited') }} + + + + + +
diff --git a/webapp/components/User/User.vue b/webapp/components/User/User.vue index eb7092bcc..3afef5c11 100644 --- a/webapp/components/User/User.vue +++ b/webapp/components/User/User.vue @@ -11,17 +11,17 @@
- + {{ userName | truncate(18) }} +
- {{ userSlug }} diff --git a/webapp/pages/post/_id/_slug/index.vue b/webapp/pages/post/_id/_slug/index.vue index 0c45c0ca3..1357f9653 100644 --- a/webapp/pages/post/_id/_slug/index.vue +++ b/webapp/pages/post/_id/_slug/index.vue @@ -6,30 +6,20 @@ :class="{ 'post-card': true, 'disabled-content': post.disabled }" > - - - - - - - {{ this.$t('post.edited') }} - - - - - - - - - - - + + + + + + {{ post.title }} From 99db1338c1522f389f5e5deec019cba839011179 Mon Sep 17 00:00:00 2001 From: roschaefer Date: Fri, 4 Oct 2019 01:05:39 +0200 Subject: [PATCH 3/3] docs: add storybook section for edit information --- webapp/components/User/User.story.js | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/webapp/components/User/User.story.js b/webapp/components/User/User.story.js index e7a66de3f..931f40da6 100644 --- a/webapp/components/User/User.story.js +++ b/webapp/components/User/User.story.js @@ -56,7 +56,7 @@ const user = { storiesOf('User', module) .addDecorator(withA11y) .addDecorator(helpers.layout) - .add('available user', () => ({ + .add('available', () => ({ components: { User }, store: helpers.store, data: () => ({ @@ -64,7 +64,21 @@ storiesOf('User', module) }), template: '', })) - .add('anonymous user', () => ({ + .add('has edited something', () => ({ + components: { User }, + store: helpers.store, + data: () => ({ + user, + }), + template: ` + + + + `, + })) + .add('anonymous', () => ({ components: { User }, store: helpers.store, data: () => ({