-
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-3649] Improve the address details actions to support increased font size #1238
[ABW-3649] Improve the address details actions to support increased font size #1238
Conversation
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.
Not part of this, but given we are fixing the view for larger font types I believe we should also make sure that the address is fully displayed on bigger fonts.
This is, edit the fullAddress
on line 154
to be as this:
Text(colorisedText)
.fixedSize(horizontal: false, vertical: true)
This way it look like the image on the right, and not like the one on the left
HStack(spacing: .large3) { | ||
Button(L10n.AddressDetails.copy, image: .copy) { | ||
FlowLayout(multilineAlignment: .center, spacing: .large3) { | ||
actionButton(L10n.AddressDetails.copy, image: .copyMedium) { |
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.
If we are already resizing the image, why not use copy-big
(that is 24x24`) and avoid adding a new asset?
private var actions: some SwiftUI.View { | ||
HStack(spacing: .large3) { | ||
Button(L10n.AddressDetails.copy, image: .copy) { | ||
FlowLayout(multilineAlignment: .center, spacing: .large3) { |
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.
I believe we should decrease this spacing as well.
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.
Hmm, not sure how much. Reducing this space will deviate from the design.
Jira ticket: ABW-3649
Description
Notes
As shown in screenshot 3, since the screen is not scrollable, the QR code becomes smaller as the font size increases.
Screenshot