From f8ff40148cba563562b67736bdf18ea6238f0145 Mon Sep 17 00:00:00 2001 From: aonomike Date: Fri, 3 May 2019 16:21:33 +0300 Subject: [PATCH 1/6] Add implementation to prefix relative image urls with '/api' --- webapp/components/Image/index.vue | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/webapp/components/Image/index.vue b/webapp/components/Image/index.vue index de05c5eda..fa95ab345 100644 --- a/webapp/components/Image/index.vue +++ b/webapp/components/Image/index.vue @@ -15,8 +15,8 @@ export default { }, computed: { imageSrc() { - // TODO implement - return 'this should be an image source attribute' + const src = this.imageProps.src + return src.startsWith('/') ? src.replace('/','/api/') : src } } } From f4b054d4c5096457f8f8880f00e83d381f8a10a6 Mon Sep 17 00:00:00 2001 From: aonomike Date: Fri, 3 May 2019 20:46:58 +0300 Subject: [PATCH 2/6] Fix linting error --- webapp/components/Image/index.vue | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webapp/components/Image/index.vue b/webapp/components/Image/index.vue index fa95ab345..e2f504578 100644 --- a/webapp/components/Image/index.vue +++ b/webapp/components/Image/index.vue @@ -16,7 +16,7 @@ export default { computed: { imageSrc() { const src = this.imageProps.src - return src.startsWith('/') ? src.replace('/','/api/') : src + return src.startsWith('/') ? src.replace('/', '/api/') : src } } } From 3ff034d0140f2d4ceb653e233dec8b4b5740a1e3 Mon Sep 17 00:00:00 2001 From: aonomike Date: Tue, 7 May 2019 15:21:42 +0300 Subject: [PATCH 3/6] Refactor tags to use Image component --- webapp/components/Badges.vue | 15 ++++++++++++--- webapp/components/Empty.vue | 13 ++++++++++--- webapp/pages/login.vue | 13 +++++++++---- webapp/pages/logout.vue | 15 ++++++++++++--- webapp/pages/settings/my-social-media.vue | 13 ++++++++++--- 5 files changed, 53 insertions(+), 16 deletions(-) diff --git a/webapp/components/Badges.vue b/webapp/components/Badges.vue index d0cb30e91..b30dba309 100644 --- a/webapp/components/Badges.vue +++ b/webapp/components/Badges.vue @@ -10,22 +10,31 @@ :key="badge.key" class="hc-badge-container" > - + /> diff --git a/webapp/components/Empty.vue b/webapp/components/Empty.vue index 8755a11bd..c1044137e 100644 --- a/webapp/components/Empty.vue +++ b/webapp/components/Empty.vue @@ -5,13 +5,13 @@ :margin="margin" > - Empty
+ />
diff --git a/webapp/pages/settings/my-social-media.vue b/webapp/pages/settings/my-social-media.vue index c031f54a4..c13d0eeb5 100644 --- a/webapp/pages/settings/my-social-media.vue +++ b/webapp/pages/settings/my-social-media.vue @@ -11,12 +11,12 @@ :key="link.url" > - Social Media link + /> {{ link.url }} @@ -45,8 +45,12 @@ diff --git a/webapp/pages/settings/my-social-media.vue b/webapp/pages/settings/my-social-media.vue index c13d0eeb5..f991305a6 100644 --- a/webapp/pages/settings/my-social-media.vue +++ b/webapp/pages/settings/my-social-media.vue @@ -12,7 +12,7 @@ > Date: Thu, 9 May 2019 02:27:03 +0300 Subject: [PATCH 5/6] Refactor code based on comments from PR review --- webapp/components/Badges.vue | 7 +------ webapp/components/Empty.vue | 7 ++----- webapp/pages/settings/my-social-media.vue | 5 +---- 3 files changed, 4 insertions(+), 15 deletions(-) diff --git a/webapp/components/Badges.vue b/webapp/components/Badges.vue index 9c7689aca..b28412abe 100644 --- a/webapp/components/Badges.vue +++ b/webapp/components/Badges.vue @@ -12,7 +12,7 @@ > @@ -30,11 +30,6 @@ export default { type: Array, default: () => [] } - }, - methods: { - imageProps(icon) { - return { src: icon } - } } } diff --git a/webapp/components/Empty.vue b/webapp/components/Empty.vue index 7f818e2f7..f8608c965 100644 --- a/webapp/components/Empty.vue +++ b/webapp/components/Empty.vue @@ -58,11 +58,8 @@ export default { } }, computed: { - iconPath() { - return `/img/empty/${this.icon}.svg` - }, - imageProps: { - src: iconPath + imageProps() { + return { src: `/img/empty/${this.icon}.svg` } } } } diff --git a/webapp/pages/settings/my-social-media.vue b/webapp/pages/settings/my-social-media.vue index f991305a6..a7b954182 100644 --- a/webapp/pages/settings/my-social-media.vue +++ b/webapp/pages/settings/my-social-media.vue @@ -12,7 +12,7 @@ > Date: Thu, 9 May 2019 21:04:07 +0300 Subject: [PATCH 6/6] Refactor to pass imageProps as an object without the computed section --- webapp/components/Empty.vue | 7 +------ webapp/pages/login.vue | 5 +---- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/webapp/components/Empty.vue b/webapp/components/Empty.vue index f8608c965..669676862 100644 --- a/webapp/components/Empty.vue +++ b/webapp/components/Empty.vue @@ -6,7 +6,7 @@ > diff --git a/webapp/pages/login.vue b/webapp/pages/login.vue index 537ef7d85..083c1cb71 100644 --- a/webapp/pages/login.vue +++ b/webapp/pages/login.vue @@ -33,7 +33,7 @@