summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorBrian Attwell <brianattwell@google.com>2014-08-20 14:20:20 -0700
committerBrian Attwell <brianattwell@google.com>2014-08-20 14:45:47 -0700
commit7267ee24038dd87f1658cfdf621236a8244aaa9a (patch)
treeefe7348d3a0d1f790faf89e4394c613bb849b13c /src
parent763006167fd6906d18728adc82c81aa51d8d581a (diff)
downloadpackages_apps_Contacts-7267ee24038dd87f1658cfdf621236a8244aaa9a.tar.gz
packages_apps_Contacts-7267ee24038dd87f1658cfdf621236a8244aaa9a.tar.bz2
packages_apps_Contacts-7267ee24038dd87f1658cfdf621236a8244aaa9a.zip
Discuss nested scrolling options at top MS
To prevent people from copying MultiShrinkScroller's implementation of nested scrolling, add a comment at the top pointing readers to other approaches. Bug: 16951620 Change-Id: I68495df8141de6ff6af9abeca10b419caff7d367
Diffstat (limited to 'src')
-rw-r--r--src/com/android/contacts/widget/MultiShrinkScroller.java7
1 files changed, 7 insertions, 0 deletions
diff --git a/src/com/android/contacts/widget/MultiShrinkScroller.java b/src/com/android/contacts/widget/MultiShrinkScroller.java
index 82b39706c..26b032b89 100644
--- a/src/com/android/contacts/widget/MultiShrinkScroller.java
+++ b/src/com/android/contacts/widget/MultiShrinkScroller.java
@@ -54,6 +54,13 @@ import android.widget.TextView;
* MultiShrinkScroller's code is heavily influenced by ScrollView. Nonetheless, several ScrollView
* features are missing. For example: handling of KEYCODES, OverScroll bounce and saving
* scroll state in savedInstanceState bundles.
+ *
+ * Before copying this approach to nested scrolling, consider whether something simpler & less
+ * customized will work for you. For example, see the re-usable StickyHeaderListView used by
+ * WifiSetupActivity (very nice). Alternatively, check out Google+'s cover photo scrolling or
+ * Android L's built in nested scrolling support. I thought I needed a more custom ViewGroup in
+ * order to track velocity, modify EdgeEffect color & perform specific animations such as the ones
+ * inside snapToBottom(). As a result this ViewGroup has non-standard talkback and keyboard support.
*/
public class MultiShrinkScroller extends FrameLayout {