Skip to content
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

fix: staking buttons, changed chart colors, indicator display, stakin… #1427

Closed
wants to merge 23 commits into from

Conversation

CzarekDryl
Copy link
Contributor

https://tw.auroracreation.com/app/tasks/15685033
https://tw.auroracreation.com/app/tasks/15649439
https://tw.auroracreation.com/app/tasks/15655944

Things that I've fixed in this PR:

  • Fixed colors of staking chart progress bar
  • Hiding indicator of staking chart when for vote is higher than minimum value
  • Staking buttons can untoggle now

Closed #1397 PR and moved styling fixes here to prevent conflicts.

@ArmandoGraterol
Copy link
Contributor

@CzarekDryl Hello! Could you rebase this PR, please?

@CzarekDryl
Copy link
Contributor Author

@ArmandoGraterol Done :)

@rdig rdig force-pushed the fix/motion-logic-fixes branch 2 times, most recently from 77f05de to 44db006 Compare November 29, 2023 22:29
Copy link

@melyndav melyndav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @CzarekDryl great work on this one! Just one issue:

  1. The threshold indicator doesn't hide when the user inputs an amount that passes the threshold:
    image

@CzarekDryl
Copy link
Contributor Author

@melyndav Raul said something else in other PR
image

@arrenv arrenv mentioned this pull request Dec 1, 2023
Copy link

@melyndav melyndav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @CzarekDryl just adding the comment from our clack channel so I can keep track of all the open PR's.

One requested change:

Instead of hiding the line and the text when the 10% threshold is met, can we keep everything showing and just change the text to '10% threshold met, will be made public'

@rdig rdig force-pushed the fix/motion-logic-fixes branch 2 times, most recently from 29fe5b8 to 5529d5a Compare December 5, 2023 14:36
Base automatically changed from fix/motion-logic-fixes to master December 5, 2023 14:37
@CzarekDryl CzarekDryl closed this Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants