-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add icons to 'Save as draft' and 'Finalize' #5936
Conversation
The colour shouldn't change - I was updating the styles for something else and didn't notice that it changed there! Could we center the icons and text, as we do with the "Back" and "Next" buttons and the "Save Form" dialog? |
464fd05
to
af3697b
Compare
af3697b
to
d1a4038
Compare
Yes, it's done. |
app:layout_constraintTop_toTopOf="parent" /> | ||
app:layout_constraintTop_toTopOf="parent" | ||
app:icon="@drawable/ic_send_24" | ||
app:iconGravity="textStart" /> |
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 think we do this for all Material buttons. Should we set our own ?materialButtonOutlinedIconStyle
and ?materialButtonIconStyle
that extend the default styles and add this do you think?
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 for all, first_launch_layout
is one example. There are some more but in other cases, we usually use wrap_content
as android:layout_width
, and then that difference is not visible. @alyblenkin what do you think should we always remove that space between the icon and the text? If so please file a separate issue.
Tested with Success! Verified on device with Android 13 Verified cases:
|
Tested with Success! Verified on device with Android 10 |
Closes #5935
Why is this the best possible solution? Were any other approaches considered?
I've just added icons to the two buttons mentioned in the issue. @alyblenkin in https://www.figma.com/file/KFi9hIQdrRgo0rLqIC7Sg8/ODK-user-flows?type=design&node-id=4092-15997&mode=design&t=LxacBD1eYjJoYsp3-0 the first button uses black color but it's just a mockup and the color shouldn't be changed right?
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
Just make sure the icons look good.
Do we need any specific form for testing your changes? If so, please attach one.
No.
Does this change require updates to documentation? If so, please file an issue here and include the link below.
No.
Before submitting this PR, please make sure you have:
./gradlew connectedAndroidTest
(or./gradlew testLab
) and confirmed all checks still pass