Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix two confusing naming issues

Browse Source 
android.R.string.yes is not "Yes" but "OK"; prefer android.R.string.ok for clarity
android.R.string.no is not "No" but "Cancel"; prefer android.R.string.cancel for clarity
Found by Error Prone.
Bug: 27723540

Change-Id: Ia6f58a7e7543ad1cc4e4722de8d095d1de8df034
This commit is contained in:
Ian Rogers
2016-05-27 14:39:34 -07:00
parent 6f2f02e4ae
commit d73ff581f3
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/DocumentsUI/src/com/android/documentsui/dirlist/DirectoryFragment.java
View File

@@ -882,7 +882,7 @@ public class DirectoryFragment extends Fragment
new AlertDialog.Builder(getActivity())
.setView(message)
.setPositiveButton(
android.R.string.yes,
android.R.string.ok,
new DialogInterface.OnClickListener() {
public void onClick(DialogInterface dialog, int id) {
// Finish selection mode first which clears selection so we
@@ -903,7 +903,7 @@ public class DirectoryFragment extends Fragment
getActivity(), docs, srcParent, getDisplayState().stack);
}
})
.setNegativeButton(android.R.string.no, null)
.setNegativeButton(android.R.string.cancel, null)
.show();
}
}.execute(selected);