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

Commit cba834de authored by Fabrice Di Meglio's avatar Fabrice Di Meglio
Browse files

Fix bug #16578018 Wi-Fi network list items should be centered with icons

- force correct refresh of the Preference summary
- put some non used code into comment ...

Change-Id: I17fc6fe2cf0717bae613d78cfac2c88d3e160ef4
parent 1f4049ed
Loading
Loading
Loading
Loading
+18 −14
Original line number Diff line number Diff line
@@ -520,6 +520,10 @@ class AccessPoint extends Preference {
        final Context context = getContext();
        updateIcon(getLevel(), context);

        // Force new summary
        setSummary(null);

        // Update to new summary
        StringBuilder summary = new StringBuilder();

        if (mState != null) { // This is the active connection
@@ -551,20 +555,16 @@ class AccessPoint extends Preference {
                summary.append(context.getString(R.string.wifi_remembered));
            }

            if (security != SECURITY_NONE) {
                String securityStrFormat;
                if (summary.length() == 0) {
                    securityStrFormat = context.getString(R.string.wifi_secured_first_item);
                } else {
                    securityStrFormat = context.getString(R.string.wifi_secured_second_item);
                }
            }

        }
// TODO: Wi-Fi team needs to decide what to do with this code.
//            if (security != SECURITY_NONE) {
//                String securityStrFormat;
//                if (summary.length() == 0) {
//                    securityStrFormat = context.getString(R.string.wifi_secured_first_item);
//                } else {
//                    securityStrFormat = context.getString(R.string.wifi_secured_second_item);
//                }
//            }

        // This is a workaround, see bug report...
        if (summary.length() < 1) {
            summary.append("   ");
        }

        if (WifiSettings.mVerboseLogging > 0) {
@@ -591,7 +591,11 @@ class AccessPoint extends Preference {
            }
        }

        if (summary.length() > 0) {
            setSummary(summary.toString());
        } else {
            showSummary = false;
        }
    }

    /**