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

Enhace unlock contracts dialog #3091

Merged
merged 3 commits into from
Jul 7, 2024

Conversation

0oM4R
Copy link
Contributor

@0oM4R 0oM4R commented Jul 7, 2024

Description

enhance style of unlock contracts dialog,

Changes

  • include the cost of unlock contracts
    Screenshot from 2024-07-07 12-14-03

Screenshot from 2024-07-07 12-13-49

  • fix the header style of lock details
    Screenshot from 2024-07-07 12-13-56

  • fix calculating the available balance

Related Issues

Documentation PR

For UI changes, Please provide the Documetation PR on info_grid

Checklist

  • Tests included
  • Build pass
  • Documentation
  • Code format and docstrings
  • Screenshots/Video attached (needed for UI changes)

@samaradel
Copy link
Contributor

image
image

please make sure that the action buttons are consistent with the rest of dialoges

@0oM4R
Copy link
Contributor Author

0oM4R commented Jul 7, 2024

image image

please make sure that the action buttons are consistent with the rest of dialoges

@samaradel please clarify what do you mean by make it consistent, this first screenshot have a disabled action button

@samaradel
Copy link
Contributor

@0oM4R I mean, the space between the action btn and the dialog is different in the 2 dialogs I mentioned

@0oM4R
Copy link
Contributor Author

0oM4R commented Jul 7, 2024

will include the enhancements in development pr

@0oM4R 0oM4R merged commit a04fbd1 into development_2.5.1 Jul 7, 2024
@0oM4R 0oM4R deleted the development_2.5.1_enhace_unlockDialog branch July 7, 2024 12:38
@xmonader xmonader added this to the 2.6.0 milestone Sep 24, 2024
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