-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
When a control in "Access control" settings is unchecked, hide that control #5589
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.
|
||
<View |
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 you can use layout_GoneMarginX
attributes instead of a spacer view here. There's an example in property.xml
.
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.
Right, done.
Fixed. |
<com.google.android.material.button.MaterialButton | ||
android:id="@+id/save_as_draft" | ||
style="@style/Widget.Material3.Button.OutlinedButton" | ||
<androidx.constraintlayout.widget.ConstraintLayout |
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 there must be a way to do this without nested ConstraintLayout
s (maybe using Barrier
), but it's really awkward to work out. Happy to go with this for now.
Tested With Success! Verified on device with Android 13 Verified cases:
|
Tested With Success! Verified on device with Android 10 |
Closes #5581
What has been done to verify that this works as intended?
I've tested the changes manually and updated automated tests.
Why is this the best possible solution? Were any other approaches considered?
As described in the issue, we decided that in order to make the behavior consistent across the app, the buttons should be hidden not disabled. When it comes to the implementation itself there is nothing important to discuss here.
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?
It's a safe pr, please just verify that buttons are hidden (if disabled in settings) as described in the issue.
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 checkAll
and confirmed all checks still pass OR confirm CircleCI build passes and run./gradlew connectedDebugAndroidTest
locally.