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

Show the additional witnesses you have voted for outside of the top 50 #151

Merged
merged 1 commit into from Aug 19, 2016

Conversation

Projects
None yet
2 participants
@aaroncox
Contributor

aaroncox commented Aug 19, 2016

After talking with a few people, there was some confusion as to whether
votes were actually being applied to people outside of the top 50. This
change adds those users below the input box, and also lets them upcast
their votes by clicking the button.

  • The additional witnesses are loaded off the current user's witness_votes profile field.
  • Any witnesses in the top 50 are filtered out.
  • Entering a username and voting will add the username to the list below.
  • Clicking the blue vote will uncast the vote, removing the user.

image

Show the additional witnesses you have voted for outside of the top 50
After talking with a few people, there was some confusion as to whether
votes were actually being applied to people outside of the top 50. This
change adds those users below the input box, and also lets them upcast
their votes by clicking the button.
@roadscape

This comment has been minimized.

Contributor

roadscape commented Aug 19, 2016

Looks good, works well.

@roadscape roadscape merged commit 5c7ce88 into steemit:develop Aug 19, 2016

@aaroncox aaroncox deleted the aaroncox:develop branch Aug 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment