-
Notifications
You must be signed in to change notification settings - Fork 5
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
Ios 9244 button link accesibility #305
Conversation
Screenshot tests report βοΈ All passing |
d05240d
to
719f2e3
Compare
.padding(.vertical, verticalPadding) | ||
.padding(.horizontal, horizontalPadding) | ||
.background(backgroundColor) | ||
.foregroundColor(foregroundColor) | ||
.border(radiusStyle: .button, borderColor: borderColor) | ||
.padding(EdgeInsets(top: 0, leading: leadingInset, bottom: 0, trailing: trailingInset)) |
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.
those lines don't depend on the scale
, could we put in common with the other approach?
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.
That was the my first approach but they modify the button layout and breaks the tests :(
π This PR is included in version 26.2.0 π The release is available on GitHub release Your semantic-release bot π¦π |
ποΈ Jira ticket
IOS-9244
π₯ What's the goal?
We must enable that the right image of the
MisticaButton
follows the accesibility propertyπ§ How do we do it?
We have a new property wrapper in iOS 14 called
ScaledMetric
it is to define numbers that should scale automatically according to the userβs Dynamic Type settings.To use it due to support iOS 13 we have to create a new struct not available prior to iOS 14 called
ScaledMisticaButton
which receives a content, height and width in its init to keep the functionality of the old modifiers and extend them with the scaleπ§ͺ How can I verify this?
Accesibility.mov