From 3f1c7b0f4974fd614c8ffea7929fc7fcf7faeadd Mon Sep 17 00:00:00 2001 From: Mill Chen Date: Fri, 25 Oct 2024 10:05:35 +0000 Subject: [PATCH] Refactor sound settings page for catalyst We have preferences existing in the sound settings page and new sound screen. In order to ensure VolumeSeekBarPreferences work smoothly, this changes support adding callback handling to VolumeSeekBarPreferences that are migrated in the new screens. Bug: 360015496 Test: visual test Flag: EXEMPT refactor the existing feature Change-Id: Iaee4581f1c2687db0b405cd9edbfef2088d9bb12 --- src/com/android/settings/notification/SoundSettings.java | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/com/android/settings/notification/SoundSettings.java b/src/com/android/settings/notification/SoundSettings.java index b069c7e1322..fffb7849bbc 100644 --- a/src/com/android/settings/notification/SoundSettings.java +++ b/src/com/android/settings/notification/SoundSettings.java @@ -113,6 +113,14 @@ public class SoundSettings extends DashboardFragment implements OnActivityResult if (phoneRingTonePreference != null && openPhoneRingtonePicker) { onPreferenceTreeClick(phoneRingTonePreference); } + if (isCatalystEnabled()) { + for (String key : getPreferenceKeysInHierarchy()) { + Preference preference = findPreference(key); + if (preference instanceof VolumeSeekBarPreference) { + ((VolumeSeekBarPreference) preference).setCallback(mVolumeCallback); + } + } + } } @Override