-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: you have to press the login button twice (#6633) (#6663) #8404
Conversation
<!-- Thanks for your contribution! Please ensure that any applicable requirements below are satisfied before submitting this pull request. This will help ensure a quick and efficient review cycle. --> **Description** When a user wants to login into the wallet they need to press the login button twice. The issue comes from the way that the Scrollview component is behaving when input is active. **Screenshots/Recordings** _If applicable, add screenshots and/or recordings to visualize the before and after of your change_ **Issue** Fix: #6633 **Checklist** * [x] There is a related GitHub issue * [ ] Tests are included if applicable * [ ] Any added code is fully documented --------- Co-authored-by: LeoTM <1881059+leotm@users.noreply.github.com> Co-authored-by: Cal Leung <cleun007@gmail.com> Co-authored-by: Frank von Hoven <141057783+frankvonhoven@users.noreply.github.com> Co-authored-by: sethkfman <10342624+sethkfman@users.noreply.github.com>
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/abdea665-403c-47ce-8cd2-7bebf6fbdfdd |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/365df091-fc80-4522-91ef-24ccc3888022 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #8404 +/- ##
==========================================
- Coverage 40.44% 40.43% -0.01%
==========================================
Files 1239 1239
Lines 29975 29976 +1
Branches 2875 2875
==========================================
Hits 12122 12122
- Misses 17156 17157 +1
Partials 697 697 ☔ View full report in Codecov by Sentry. |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
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
Description
When a user wants to login into the wallet they need to press the login button twice. The issue comes from the way that the Scrollview component is behaving when input is active.
Screenshots/Recordings
If applicable, add screenshots and/or recordings to visualize the before and after of your change
Issue
Fix: #6633
PR: #6663
Checklist
Description
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist