Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 21141655 authored by Joel Galenson's avatar Joel Galenson
Browse files

Fix minor Talkback issues. 

Fixes: 132047086
Fixes: 132050707
Test: Select non-linkable preferences with Talkback.
Test: Click on clickable icon in header with Talkback.
Change-Id: I9362b10b6dfc90b8a05ec0456c0ebae3ed98ae41
parent 22e3da5a

src/com/android/packageinstaller/permission/ui/handheld/AppPermissionsFragment.java

+2 −0
Original line number Diff line number Diff line
@@ -298,11 +298,13 @@ public final class AppPermissionsFragment extends SettingsWithLargeHeader { 
        if (allowed.getPreferenceCount() == 0) {

            Preference empty = new Preference(context);

            empty.setTitle(getString(R.string.no_permissions_allowed));

            empty.setSelectable(false);

            allowed.addPreference(empty);

        }

        if (denied.getPreferenceCount() == 0) {

            Preference empty = new Preference(context);

            empty.setTitle(getString(R.string.no_permissions_denied));

            empty.setSelectable(false);

            denied.addPreference(empty);

        }

src/com/android/packageinstaller/permission/ui/handheld/PermissionAppsFragment.java

+3 −0
Original line number Diff line number Diff line
@@ -361,6 +361,7 @@ public final class PermissionAppsFragment extends SettingsWithLargeHeader implem 
        if (allowed.getPreferenceCount() == 0) {

            Preference empty = new Preference(context);

            empty.setTitle(getString(R.string.no_apps_allowed));

            empty.setSelectable(false);

            allowed.addPreference(empty);

        }

        if (allowedForeground.getPreferenceCount() == 0) {
@@ -371,6 +372,7 @@ public final class PermissionAppsFragment extends SettingsWithLargeHeader implem 
        if (denied.getPreferenceCount() == 0) {

            Preference empty = new Preference(context);

            empty.setTitle(getString(R.string.no_apps_denied));

            empty.setSelectable(false);

            denied.addPreference(empty);

        }
@@ -382,6 +384,7 @@ public final class PermissionAppsFragment extends SettingsWithLargeHeader implem 
            Preference footerText = new Preference(context);

            footerText.setSummary(context.getString(R.string.app_permission_footer_not_available));

            footerText.setIcon(R.drawable.ic_info_outline);

            footerText.setSelectable(false);

            footer.addPreference(footerText);

        }

src/com/android/packageinstaller/permission/ui/handheld/SettingsWithLargeHeader.java

+1 −0
Original line number Diff line number Diff line
@@ -102,6 +102,7 @@ public abstract class SettingsWithLargeHeader extends PermissionsFrameFragment 
            if (mInfoIntent != null) {

                appIcon.setOnClickListener(v -> getActivity().startActivityAsUser(mInfoIntent,

                        mUserHandle));

                appIcon.setContentDescription(mLabel);

            }



            TextView appName = header.requireViewById(R.id.entity_header_title);