-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(sponsor): implement sponsor list on index page #40
Conversation
components/sponsors/SponsorModal.vue
Outdated
.lightBox__closeButton { | ||
@apply relative flex justify-end text-yellow-500; | ||
top: 10px; | ||
right: 10px; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add cursor: pointer;
to allow users to know this is pressable.
components/sponsors/SponsorModal.vue
Outdated
top: 10px; | ||
right: 10px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
top: 10px; | |
right: 10px; | |
top: -40px; | |
right: -20px; |
This looks closer to the design spec, give or take.
components/sponsors/SponsorModal.vue
Outdated
} | ||
|
||
.lightBox__title { | ||
@apply flex justify-start items-center; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the unnecessary justify-start
class as the justify-content
rule by default is already flex-start
.
pages/index.vue
Outdated
closeModal() { | ||
this.isOpened = false | ||
this.selectedSponsor = {} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this unused method
components/sponsors/SponsorCard.vue
Outdated
@@ -0,0 +1,43 @@ | |||
<template> | |||
<div v-show="!!logoUrl" :class="classObject"> | |||
<img :src="logoUrl" class="object-contain w-full" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<img :src="logoUrl" class="object-contain w-full" /> | |
<img :src="logoUrl" alt="" class="object-contain w-full" /> |
Provide the required attribute for <img>
components/sponsors/SponsorModal.vue
Outdated
<div class="lightBox__intro"> | ||
<p class="text-sm">{{ getAttributeByLocale('intro') }}</p> | ||
</div> | ||
<div class="lightBox__bottons"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div class="lightBox__bottons"> | |
<div class="lightBox__buttons"> |
Fix typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Types of Changes
Description
More Information
Preview: Sponsor List
web view
mobile view (iPhone 6/7/8)
Preview: Modal
web view
mobile view (iPhone 6/7/8)