Feature/passphrase validation UI feedback #122
Conversation
Signed-off-by: Deepam02 <[email protected]>
Signed-off-by: Deepam02 <[email protected]>
Signed-off-by: Deepam02 <[email protected]>
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
@SubstantialCattle5 |
Can the reqd be mentioned below the passpharese text box?
Also there isn't any indiciation when we use the flag
|
|
@Deepam02, are you still working on the issue? |
|
yes ,i got busy with some college activities, but I will finish it by tomorrow. |
|
@Deepam02 sure.. |
💡 Suggestions to Improve This PR
Consider addressing these points to make your contribution stronger. Thanks for contributing! |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| } | ||
|
|
||
| if hasLength { | ||
| statusParts[0] = fmt.Sprintf("%s12+(%d)", getSymbol(hasLength), len(passphrase)) |
Check warning
Code scanning / gosec
slice index out of range Warning
tried to do it but coudnt because Terminal-based prompts like promptui.Prompt don't naturally support displaying content below the input field while maintaining cursor position. The feedback clearing and repositioning logic is designed to update content above the cursor, so we placed requirements above the input instead.
the passphrase requirements are now displayed when the flag is used. |
|
@SubstantialCattle5 have you had a chance to review this. |
|
Hi @Deepam02 , |
2 participants
Closes #113
Enhanced the UI to display real-time validation feedback when users enter a passphrase
Screen.Recording.2025-10-07.153042.mp4