-
Notifications
You must be signed in to change notification settings - Fork 98
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
Jordan/1603 sidebar links #1633
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1633 +/- ##
===========================================
- Coverage 97.63% 97.63% -0.01%
===========================================
Files 101 101
Lines 2031 2029 -2
Branches 90 90
===========================================
- Hits 1983 1981 -2
Misses 37 37
Partials 11 11
|
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.
Changes LGTM. Will test it shortly
@@ -16,7 +12,7 @@ | |||
| | |||
router-link(:to="{name: 'Validators'}") the validator list | |||
| | |||
| to spread some of your Atoms around. |
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.
lol why did we had spread
@@ -26,8 +31,7 @@ export default { | |||
if (this.proposal.proposal_status === `Passed`) | |||
return { | |||
button: null, | |||
message: `This proposal has passed`, | |||
color: `green` |
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.
I think we should leave it green here
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.
it's not a little green dot anymore - it's a green checkmark.
if felt the green status dot should be for signalling "voting".
truthfully, i'd like to move status from the little dot into its own column / tab soon.
@@ -45,7 +49,7 @@ export default { | |||
return { | |||
button: `vote`, | |||
message: `Voting for this proposal is open`, | |||
color: `blue` | |||
color: `green` |
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.
and use orange or similar color here
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.
no green is good for open voting! like a traffic light! GO GO GO!
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.
tested it. Please fix all the warnings on the console. E.g:
A warning has occurred: Property or method "totalRewards" is not defined on the instance but referenced during render...
|
totally agree. but trying to stay focussed on our milestones 😄 |
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.
tested ACK
Closes #1603
Description:
❤️ Thank you!
CHANGELOG.md
with issue # and GitHub usernameFiles changed
in the github PR explorer