diff options
author | Bruno Martins <bgcngm@gmail.com> | 2017-12-03 22:03:02 +0000 |
---|---|---|
committer | Bruno Martins <bgcngm@gmail.com> | 2017-12-06 12:31:48 +0000 |
commit | 9f648b7046d1f921707c749b02b911ae126fd0fa (patch) | |
tree | cd12b212fdcec9769819c4fdca30197c575addcf /sdk/src/java/lineageos/preference/SelfRemovingListPreference.java | |
parent | 8c4872f8013d8b34fabc0d1854b776d5a5bb229e (diff) | |
download | lineage-sdk-9f648b7046d1f921707c749b02b911ae126fd0fa.tar.gz lineage-sdk-9f648b7046d1f921707c749b02b911ae126fd0fa.tar.bz2 lineage-sdk-9f648b7046d1f921707c749b02b911ae126fd0fa.zip |
Revert "lineage-sdk self-removing prefs: move reap from onBindViewHolder to onAttached"lineage-15.0
This reverts commit 027b615325092f7676a34c137248dc9fcd57cc35.
While it served its original purpose, this was causing bad side-effects
when using our constraints (verified at least with requiresConfig and
requiresAction). Moving the reap to onAttached was rendering self-removing
prefs useless because if the view isn't loaded yet, the preference
(which also doesn't yet exists) can't really be hidden.
Change-Id: Ic61fe4b9979412d89c55ea1543c494e69bf6eab7
Diffstat (limited to 'sdk/src/java/lineageos/preference/SelfRemovingListPreference.java')
-rw-r--r-- | sdk/src/java/lineageos/preference/SelfRemovingListPreference.java | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/sdk/src/java/lineageos/preference/SelfRemovingListPreference.java b/sdk/src/java/lineageos/preference/SelfRemovingListPreference.java index 0b8c7985..d1d3ec5a 100644 --- a/sdk/src/java/lineageos/preference/SelfRemovingListPreference.java +++ b/sdk/src/java/lineageos/preference/SelfRemovingListPreference.java @@ -44,12 +44,6 @@ public class SelfRemovingListPreference extends ListPreference { } @Override - public void onAttached() { - super.onAttached(); - mConstraints.onAttached(); - } - - @Override public void onBindViewHolder(PreferenceViewHolder holder) { super.onBindViewHolder(holder); mConstraints.onBindViewHolder(holder); |