Skip to content

Conversation

@Sotatek-DukeVu
Copy link
Collaborator

Description

Clear input when close the manual input modal on SSI screen

Checklist before requesting a review

Issue ticket number and link

  • This PR has a valid ticket number or issue: VT20-2361

Testing & Validation

  • This PR has been tested/validated in iOS, Android and browser.
  • Added new unit tests, if relevant.

Design Review

  • In case this PR contains changes to the UI, add some screenshots and/or videos to show the changes on relevant devices.
Screen.Recording.2025-12-15.at.12.05.02.mov

Copy link
Contributor

@sdisalvo-crd sdisalvo-crd left a comment

Choose a reason for hiding this comment

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

When you reopen the modal the error should be gone until you type something wrong and click out. Also the confirm button should be disabled.
https://www.figma.com/design/xDtjsTapugXYhlUvwdCq49/Veridian-App---LIVE?node-id=15758-40755&m=dev

@sdisalvo-crd sdisalvo-crd self-requested a review December 16, 2025 11:39
Copy link
Contributor

@sdisalvo-crd sdisalvo-crd left a comment

Choose a reason for hiding this comment

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

Please check that:

  • The confirm button is disabled and gray when the input is empty and it's enabled and blue when there is something in the input field and no errors
  • The input field border is light gray and it currently turns dark grey when selected. It should be light gray and dark blue.
Image

@sdisalvo-crd sdisalvo-crd merged commit e3fdf0f into develop Dec 17, 2025
3 checks passed
@jacklesotatek
Copy link
Collaborator

Tests passed on this branch. Thanks

@jacklesotatek jacklesotatek deleted the VT20-2361-ssi-agent-manual-input-not-clearing-out-on-close branch December 18, 2025 08:22
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