diff options
author | 2025-03-16 15:54:14 -0700 | |
---|---|---|
committer | 2025-03-16 15:54:14 -0700 | |
commit | d09d4c6ab71c3f61c10740f205dbb8c85d6d8581 (patch) | |
tree | f70bbb3719091dfbb48ef03a07a5ec0b225f2690 /src | |
parent | c767c9dbe1d554bc81fe7cf64bd73392a0d4839e (diff) | |
parent | dc3e73e728894bf178b3a10e2b2f36717ee8f557 (diff) |
Merge "Remove the full width "Use this folder" for all form factors" into main
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/documentsui/picker/PickFragment.java | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/src/com/android/documentsui/picker/PickFragment.java b/src/com/android/documentsui/picker/PickFragment.java index 66f05fa0f..0d20083a8 100644 --- a/src/com/android/documentsui/picker/PickFragment.java +++ b/src/com/android/documentsui/picker/PickFragment.java @@ -181,12 +181,21 @@ public class PickFragment extends Fragment { switch (mAction) { case State.ACTION_OPEN_TREE: mPick.setText(getString(R.string.open_tree_button)); - // On laptops we want the "Use this folder" button to appear with the "Cancel" - // button as a back gesture with a mouse is not easy. - if (!isUseMaterial3FlagEnabled() - || !getActivity().getPackageManager() - .hasSystemFeature(PackageManager.FEATURE_PC)) { + // When use_material3 flag is enabled, all form factors should have the pick button + // wrap the text content instead of taking up the full width. + if (!isUseMaterial3FlagEnabled()) { + mCancel.setVisibility(View.GONE); mPick.setWidth(Integer.MAX_VALUE); + mPickOverlay.setVisibility( + mPickTarget.isBlockedFromTree() && mRestrictScopeStorage + ? View.VISIBLE + : View.GONE); + } else if (!getActivity() + .getPackageManager() + .hasSystemFeature(PackageManager.FEATURE_PC)) { + // On non-desktop devices the back gesture is used to cancel the picker, so + // don't show the "Cancel" button on these devices and instead enable the pick + // overlay which enables showing a toast when the disabled button is pressed. mCancel.setVisibility(View.GONE); mPickOverlay.setVisibility( mPickTarget.isBlockedFromTree() && mRestrictScopeStorage |