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
[DDW-173] - Button should be disabled when there is nothing to submit #1998
[DDW-173] - Button should be disabled when there is nothing to submit #1998
Conversation
I'd say that button should be active when there is no blocking errors on the screen. |
@yakovkaravelov @tomothespian please review this PR 🙏 |
source/renderer/app/components/wallet/receive/WalletReceiveRandom.js
Outdated
Show resolved
Hide resolved
source/renderer/app/components/wallet/settings/ChangeSpendingPasswordDialog.js
Outdated
Show resolved
Hide resolved
Done |
Great job @DeeJayElly . The only outstanding issue I can still see is the paper wallet message being minorly truncated when Daedalus is not set to full screen |
@ManusMcCole this is a known issue which has nothing to do with this PR. cc @gnpf @a-rukin |
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.
Great work @DeeJayElly
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.
@DeeJayElly Great work, all fixes look great 🚀
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.
@DeeJayElly Looks good now. 🚀 I just left 3 minor comments related to code cleanup / optimization
source/renderer/app/components/wallet/settings/ChangeSpendingPasswordDialog.js
Outdated
Show resolved
Hide resolved
source/renderer/app/components/wallet/wallet-restore/ConfigurationDialog.js
Outdated
Show resolved
Hide resolved
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.
@DeeJayElly please remove password format validation on the "Current password" field - here we should only check for length > 0...
@DeeJayElly on the restore dialog the submit button on 2nd step should remain disabled until user enters all 12/15/27 words: |
CHANGELOG.md
Outdated
@@ -9,6 +9,7 @@ Changelog | |||
|
|||
### Fixes | |||
|
|||
- Disabled button on forms when there is nothing to submit ([PR 1998](https://github.com/input-output-hk/daedalus/pull/1998)) |
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.
@DeeJayElly please move this line under vNext
(add it if it is not already there!)
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.
Added
… - updating changelog
Fixed |
… - mnemonics dialog fixes
… - change spending password fixes
Fixed |
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.
The last fixes look good 👍
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
…n-there-is-nothing-to-submit # Conflicts: # CHANGELOG.md
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.
Works great @DeeJayElly 🎉
This PR disables button on forms if there is nothing to be submitted (if the input fields are empty)
Screenshots
Testing Checklist
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board