-
Notifications
You must be signed in to change notification settings - Fork 9
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
[ABW-3522] Create Account flow updates #1217
Conversation
@@ -44,6 +44,8 @@ public struct AppTextField<FocusValue: Hashable, Accessory: View, InnerAccessory | |||
let accessory: Accessory | |||
let innerAccesory: InnerAccessory | |||
|
|||
@State private var isFocused: Bool = false |
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.
My first approach was to use @Environment(\.isFocused)
, but it didn't work. Therefore, I went with this custom solution that we manually updated the flag with the onEditingChanged
callback of the TextField
.
I think we could be smarter and use our Focus
and some callback when it changes, but given such property is optional, it would be tricky handling it when focus = nil
. Open to suggestions if someone comes up with a better idea for this.
Jira ticket: ABW-3522
Description
This PR makes the following updates:
NameAccount
&Completion
views.AppTextField
to only add grey border when focused (see comments in code).TooggleView
title to have more than one line (so that the Create with Ledger toggle doesn't look trimmed as first video shows).Video
before.mov
after.mov