Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

am 033837dd: Merge "Make fading transitions work better" into klp-dev

Browse Source 
* commit '033837dd6bf32b33d9a67d340efd167f94bf80cf':
  Make fading transitions work better
This commit is contained in:
Chet Haase
2013-09-30 14:13:27 -07:00
committed by Android Git Automerger
parent 8ba6752935 033837dd6b
commit a16299b1cf
14 changed files with 155 additions and 112 deletions
Download Patch File Download Diff File Expand all files Collapse all files

54
core/java/android/transition/TextChange.java → core/java/android/transition/ChangeText.java
View File

@@ -37,7 +37,7 @@ import java.util.Map;
*
* @hide
*/
public class TextChange extends Transition {
public class ChangeText extends Transition {
private static final String LOG_TAG = "TextChange";
@@ -103,7 +103,7 @@ public class TextChange extends Transition {
* transition is run.
* @return this textChange object.
*/
public TextChange setChangeBehavior(int changeBehavior) {
public ChangeText setChangeBehavior(int changeBehavior) {
if (changeBehavior >= CHANGE_BEHAVIOR_KEEP && changeBehavior <= CHANGE_BEHAVIOR_OUT_IN) {
mChangeBehavior = changeBehavior;
}
@@ -179,9 +179,13 @@ public class TextChange extends Transition {
startSelectionStart = startSelectionEnd = endSelectionStart = endSelectionEnd = -1;
}
if (!startText.equals(endText)) {
view.setText(startText);
if (view instanceof EditText) {
setSelection(((EditText) view), startSelectionStart, startSelectionEnd);
final int startColor = (Integer) startVals.get(PROPNAME_TEXT_COLOR);
final int endColor = (Integer) endVals.get(PROPNAME_TEXT_COLOR);
if (mChangeBehavior != CHANGE_BEHAVIOR_IN) {
view.setText(startText);
if (view instanceof EditText) {
setSelection(((EditText) view), startSelectionStart, startSelectionEnd);
}
}
Animator anim;
if (mChangeBehavior == CHANGE_BEHAVIOR_KEEP) {
@@ -200,8 +204,6 @@ public class TextChange extends Transition {
});
} else {
// Fade out start text
final int startColor = (Integer) startVals.get(PROPNAME_TEXT_COLOR);
final int endColor = (Integer) endVals.get(PROPNAME_TEXT_COLOR);
ValueAnimator outAnim = null, inAnim = null;
if (mChangeBehavior == CHANGE_BEHAVIOR_OUT_IN ||
mChangeBehavior == CHANGE_BEHAVIOR_OUT) {
@@ -210,8 +212,8 @@ public class TextChange extends Transition {
@Override
public void onAnimationUpdate(ValueAnimator animation) {
int currAlpha = (Integer) animation.getAnimatedValue();
view.setTextColor(currAlpha << 24 | Color.red(startColor) << 16 |
Color.green(startColor) << 8 | Color.red(startColor));
view.setTextColor(currAlpha << 24 | startColor & 0xff0000 |
startColor & 0xff00 | startColor & 0xff);
}
});
outAnim.addListener(new AnimatorListenerAdapter() {
@@ -225,6 +227,8 @@ public class TextChange extends Transition {
endSelectionEnd);
}
}
// restore opaque alpha and correct end color
view.setTextColor(endColor);
}
});
}
@@ -239,6 +243,13 @@ public class TextChange extends Transition {
Color.green(endColor) << 8 | Color.red(endColor));
}
});
inAnim.addListener(new AnimatorListenerAdapter() {
@Override
public void onAnimationCancel(Animator animation) {
// restore opaque alpha and correct end color
view.setTextColor(endColor);
}
});
}
if (outAnim != null && inAnim != null) {
anim = new AnimatorSet();
@@ -251,21 +262,32 @@ public class TextChange extends Transition {
}
}
TransitionListener transitionListener = new TransitionListenerAdapter() {
boolean mCanceled = false;
int mPausedColor = 0;
@Override
public void onTransitionPause(Transition transition) {
view.setText(endText);
if (view instanceof EditText) {
setSelection(((EditText) view), endSelectionStart, endSelectionEnd);
if (mChangeBehavior != CHANGE_BEHAVIOR_IN) {
view.setText(endText);
if (view instanceof EditText) {
setSelection(((EditText) view), endSelectionStart, endSelectionEnd);
}
}
if (mChangeBehavior > CHANGE_BEHAVIOR_KEEP) {
mPausedColor = view.getCurrentTextColor();
view.setTextColor(endColor);
}
}
@Override
public void onTransitionResume(Transition transition) {
view.setText(startText);
if (view instanceof EditText) {
setSelection(((EditText) view), startSelectionStart, startSelectionEnd);
if (mChangeBehavior != CHANGE_BEHAVIOR_IN) {
view.setText(startText);
if (view instanceof EditText) {
setSelection(((EditText) view), startSelectionStart, startSelectionEnd);
}
}
if (mChangeBehavior > CHANGE_BEHAVIOR_KEEP) {
view.setTextColor(mPausedColor);
}
}
};

45
core/java/android/transition/Fade.java
View File

@@ -30,6 +30,24 @@ import android.view.ViewGroup;
* {@link View#setVisibility(int)} state of the view as well as whether it
* is parented in the current view hierarchy.
*
* <p>The ability of this transition to fade out a particular view, and the
* way that that fading operation takes place, is based on
* the situation of the view in the view hierarchy. For example, if a view was
* simply removed from its parent, then the view will be added into a {@link
* android.view.ViewGroupOverlay} while fading. If a visible view is
* changed to be {@link View#GONE} or {@link View#INVISIBLE}, then the
* visibility will be changed to {@link View#VISIBLE} for the duration of
* the animation. However, if a view is in a hierarchy which is also altering
* its visibility, the situation can be more complicated. In general, if a
* view that is no longer in the hierarchy in the end scene still has a
* parent (so its parent hierarchy was removed, but it was not removed from
* its parent), then it will be left alone to avoid side-effects from
* improperly removing it from its parent. The only exception to this is if
* the previous {@link Scene} was
* {@link Scene#getSceneForLayout(android.view.ViewGroup, int, android.content.Context)
* created from a layout resource file}, then it is considered safe to un-parent
* the starting scene view in order to fade it out.</p>
*
* <p>A Fade transition can be described in a resource file by using the
* tag <code>fade</code>, along with the standard
* attributes of {@link android.R.styleable#Fade} and
@@ -167,7 +185,7 @@ public class Fade extends Visibility {
if ((mFadingMode & OUT) != OUT) {
return null;
}
View view;
View view = null;
View startView = (startValues != null) ? startValues.view : null;
View endView = (endValues != null) ? endValues.view : null;
if (DBG) {
@@ -177,9 +195,28 @@ public class Fade extends Visibility {
View overlayView = null;
View viewToKeep = null;
if (endView == null || endView.getParent() == null) {
// view was removed: add the start view to the Overlay
view = startView;
overlayView = view;
if (endView != null) {
// endView was removed from its parent - add it to the overlay
view = overlayView = endView;
} else if (startView != null) {
// endView does not exist. Use startView only under certain
// conditions, because placing a view in an overlay necessitates
// it being removed from its current parent
if (startView.getParent() == null) {
// no parent - safe to use
view = overlayView = startView;
} else if (startView.getParent() instanceof View &&
startView.getParent().getParent() == null) {
View startParent = (View) startView.getParent();
int id = startParent.getId();
if (id != View.NO_ID && sceneRoot.findViewById(id) != null && mCanRemoveViews) {
// no parent, but its parent is unparented but the parent
// hierarchy has been replaced by a new hierarchy with the same id
// and it is safe to un-parent startView
view = overlayView = startView;
}
}
}
} else {
// visibility change
if (endVisibility == View.INVISIBLE) {

19
core/java/android/transition/Scene.java
View File

@@ -157,11 +157,11 @@ public final class Scene {
public void enter() {
// Apply layout change, if any
if (mLayoutId >= 0 || mLayout != null) {
if (mLayoutId > 0 || mLayout != null) {
// empty out parent container before adding to it
getSceneRoot().removeAllViews();
if (mLayoutId >= 0) {
if (mLayoutId > 0) {
LayoutInflater.from(mContext).inflate(mLayoutId, mSceneRoot);
} else {
mSceneRoot.addView(mLayout);
@@ -242,4 +242,19 @@ public final class Scene {
mExitAction = action;
}
/**
* Returns whether this Scene was created by a layout resource file, determined
* by the layoutId passed into
* {@link #getSceneForLayout(android.view.ViewGroup, int, android.content.Context)}.
* This is called by TransitionManager to determine whether it is safe for views from
* this scene to be removed from their parents when the scene is exited, which is
* used by {@link Fade} to fade these views out (the views must be removed from
* their parent in order to add them to the overlay for fading purposes). If a
* Scene is not based on a resource file, then the impact of removing views
* arbitrarily is unknown and should be avoided.
*/
boolean isCreatedFromLayoutResource() {
return (mLayoutId > 0);
}
}

12
core/java/android/transition/Transition.java
View File

@@ -118,6 +118,14 @@ public abstract class Transition implements Cloneable {
// Scene Root is set at createAnimator() time in the cloned Transition
ViewGroup mSceneRoot = null;
// Whether removing views from their parent is possible. This is only for views
// in the start scene, which are no longer in the view hierarchy. This property
// is determined by whether the previous Scene was created from a layout
// resource, and thus the views from the exited scene are going away anyway
// and can be removed as necessary to achieve a particular effect, such as
// removing them from parents to add them to overlays.
boolean mCanRemoveViews = false;
// Track all animators in use in case the transition gets canceled and needs to
// cancel running animators
private ArrayList<Animator> mCurrentAnimators = new ArrayList<Animator>();
@@ -1445,6 +1453,10 @@ public abstract class Transition implements Cloneable {
return this;
}
void setCanRemoveViews(boolean canRemoveViews) {
mCanRemoveViews = canRemoveViews;
}
@Override
public String toString() {
return toString("");

5
core/java/android/transition/TransitionManager.java
View File

@@ -178,6 +178,11 @@ public class TransitionManager {
Transition transitionClone = transition.clone();
transitionClone.setSceneRoot(sceneRoot);
Scene oldScene = Scene.getCurrentScene(sceneRoot);
if (oldScene != null && oldScene.isCreatedFromLayoutResource()) {
transitionClone.setCanRemoveViews(true);
}
sceneChangeSetup(sceneRoot, transitionClone);
scene.enter();

9
core/java/android/transition/TransitionSet.java
View File

@@ -339,6 +339,15 @@ public class TransitionSet extends Transition {
return (TransitionSet) this;
}
@Override
void setCanRemoveViews(boolean canRemoveViews) {
super.setCanRemoveViews(canRemoveViews);
int numTransitions = mTransitions.size();
for (int i = 0; i < numTransitions; ++i) {
mTransitions.get(i).setCanRemoveViews(canRemoveViews);
}
}
@Override
String toString(String indent) {
String result = super.toString(indent);

65
core/java/android/transition/Visibility.java
View File

@@ -30,22 +30,6 @@ import android.view.ViewGroup;
* changes occur. Subclasses should implement one or both of the methods
* {@link #onAppear(ViewGroup, TransitionValues, int, TransitionValues, int)},
* {@link #onDisappear(ViewGroup, TransitionValues, int, TransitionValues, int)},
*
* <p>Note that a view's visibility change is determined by both whether the view
* itself is changing and whether its parent hierarchy's visibility is changing.
* That is, a view that appears in the end scene will only trigger a call to
* {@link #onAppear(android.view.ViewGroup, TransitionValues, int, TransitionValues, int)
* appear()} if its parent hierarchy was stable between the start and end scenes.
* This is done to avoid causing a visibility transition on every node in a hierarchy
* when only the top-most node is the one that should be transitioned in/out.
* Stability is determined by either the parent hierarchy views being the same
* between scenes or, if scenes are inflated from layout resource files and thus
* have result in different view instances, if the views represented by
* the ids of those parents are stable. This means that visibility determination
* is more effective with inflated view hierarchies if ids are used.
* The exception to this is when the visibility subclass transition is
* targeted at specific views, in which case the visibility of parent views
* is ignored.</p>
*/
public abstract class Visibility extends Transition {
@@ -111,51 +95,6 @@ public abstract class Visibility extends Transition {
return visibility == View.VISIBLE && parent != null;
}
/**
* Tests whether the hierarchy, up to the scene root, changes visibility between
* start and end scenes. This is done to ensure that a view that changes visibility
* is only animated if that view's parent was stable between scenes; we should not
* fade an entire hierarchy, but rather just the top-most node in the hierarchy that
* changed visibility. Note that both the start and end parents are passed in
* because the instances may differ for the same view due to layout inflation
* between scenes.
*
* @param sceneRoot The root of the scene hierarchy
* @param startView The container view in the start scene
* @param endView The container view in the end scene
* @return true if the parent hierarchy experienced a visibility change, false
* otherwise
*/
private boolean isHierarchyVisibilityChanging(ViewGroup sceneRoot, ViewGroup startView,
ViewGroup endView) {
if (startView == sceneRoot || endView == sceneRoot) {
return false;
}
TransitionValues startValues = startView != null ?
getTransitionValues(startView, true) : getTransitionValues(endView, true);
TransitionValues endValues = endView != null ?
getTransitionValues(endView, false) : getTransitionValues(startView, false);
if (startValues == null || endValues == null) {
return true;
}
Integer visibility = (Integer) startValues.values.get(PROPNAME_VISIBILITY);
int startVisibility = (visibility != null) ? visibility : -1;
ViewGroup startParent = (ViewGroup) startValues.values.get(PROPNAME_PARENT);
visibility = (Integer) endValues.values.get(PROPNAME_VISIBILITY);
int endVisibility = (visibility != null) ? visibility : -1;
ViewGroup endParent = (ViewGroup) endValues.values.get(PROPNAME_PARENT);
if (startVisibility != endVisibility || startParent != endParent) {
return true;
}
if (startParent != null || endParent != null) {
return isHierarchyVisibilityChanging(sceneRoot, startParent, endParent);
}
return false;
}
private VisibilityInfo getVisibilityChangeInfo(TransitionValues startValues,
TransitionValues endValues) {
final VisibilityInfo visInfo = new VisibilityInfo();
@@ -225,9 +164,7 @@ public abstract class Visibility extends Transition {
int endId = endView != null ? endView.getId() : -1;
isTarget = isValidTarget(startView, startId) || isValidTarget(endView, endId);
}
if (isTarget || ((visInfo.startParent != null || visInfo.endParent != null) &&
!isHierarchyVisibilityChanging(sceneRoot,
visInfo.startParent, visInfo.endParent))) {
if (isTarget || ((visInfo.startParent != null || visInfo.endParent != null))) {
if (visInfo.fadeIn) {
return onAppear(sceneRoot, startValues, visInfo.startVisibility,
endValues, visInfo.endVisibility);

6
core/java/com/android/internal/transition/ActionBarTransition.java
View File

@@ -19,7 +19,7 @@ package com.android.internal.transition;
import android.transition.ChangeBounds;
import android.transition.Fade;
import android.transition.TextChange;
import android.transition.ChangeText;
import android.transition.Transition;
import android.transition.TransitionManager;
import android.transition.TransitionSet;
@@ -35,8 +35,8 @@ public class ActionBarTransition {
static {
if (TRANSITIONS_ENABLED) {
final TextChange tc = new TextChange();
tc.setChangeBehavior(TextChange.CHANGE_BEHAVIOR_OUT_IN);
final ChangeText tc = new ChangeText();
tc.setChangeBehavior(ChangeText.CHANGE_BEHAVIOR_OUT_IN);
final TransitionSet inner = new TransitionSet();
inner.addTransition(tc).addTransition(new ChangeBounds());
final TransitionSet tg = new TransitionSet();

6
core/java/com/android/internal/widget/ActionBarView.java
View File

@@ -29,12 +29,6 @@ import android.os.Parcel;
import android.os.Parcelable;
import android.text.Layout;
import android.text.TextUtils;
import android.transition.ChangeBounds;
import android.transition.Fade;
import android.transition.TextChange;
import android.transition.Transition;
import android.transition.TransitionManager;
import android.transition.TransitionSet;
import android.util.AttributeSet;
import android.view.CollapsibleActionView;
import android.view.Gravity;

5
tests/TransitionTests/res/layout/crossfade_multiple.xml
View File

@@ -33,6 +33,11 @@
android:onClick="changeTransitionType"
android:id="@+id/textfade2"
android:text="@string/textfade2"/>
<RadioButton android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:onClick="changeTransitionType"
android:id="@+id/textfade3"
android:text="@string/textfade3"/>
</RadioGroup>
<LinearLayout android:orientation="horizontal"
android:layout_width="match_parent"

5
tests/TransitionTests/res/values/strings.xml
View File

@@ -55,6 +55,7 @@
<string name="reveal">Reveal</string>
<string name="crossfade">Crossfade</string>
<string name="inout">In/Out</string>
<string name="textfade1">T1</string>
<string name="textfade2">T2</string>
<string name="textfade1">CT</string>
<string name="textfade2">CTO</string>
<string name="textfade3">CTI</string>
</resources>

4
tests/TransitionTests/src/com/android/transitiontests/ChangingText.java
View File

@@ -22,7 +22,7 @@ import android.view.ViewGroup;
import android.transition.Scene;
import android.transition.TransitionSet;
import android.transition.ChangeBounds;
import android.transition.TextChange;
import android.transition.ChangeText;
import android.transition.TransitionManager;
public class ChangingText extends Activity {
@@ -44,7 +44,7 @@ public class ChangingText extends Activity {
mScene2 = Scene.getSceneForLayout(mSceneRoot, R.layout.changing_text_2, this);
mChanger = new TransitionSet().setOrdering(TransitionSet.ORDERING_TOGETHER);
mChanger.addTransition(new ChangeBounds()).addTransition(new TextChange());
mChanger.addTransition(new ChangeBounds()).addTransition(new ChangeText());
mCurrentScene = mScene1;
}

4
tests/TransitionTests/src/com/android/transitiontests/ClippingText.java
View File

@@ -23,7 +23,7 @@ import android.view.ViewGroup;
import android.transition.Scene;
import android.widget.Button;
import android.transition.Fade;
import android.transition.TextChange;
import android.transition.ChangeText;
import android.transition.TransitionSet;
import android.transition.TransitionManager;
@@ -51,7 +51,7 @@ public class ClippingText extends Activity {
mChanger = new TransitionSet().setOrdering(TransitionSet.ORDERING_TOGETHER);
ChangeBounds changeBounds = new ChangeBounds();
changeBounds.setResizeClip(true);
mChanger.addTransition(changeBounds).addTransition(new TextChange());
mChanger.addTransition(changeBounds).addTransition(new ChangeText());
mCurrentScene = mScene1;
}

28
tests/TransitionTests/src/com/android/transitiontests/CrossfadeMultiple.java
View File

@@ -21,7 +21,7 @@ import android.transition.ChangeBounds;
import android.view.View;
import android.view.ViewGroup;
import android.transition.Crossfade;
import android.transition.TextChange;
import android.transition.ChangeText;
import android.transition.Transition;
import android.transition.TransitionSet;
import android.transition.TransitionManager;
@@ -41,7 +41,7 @@ public class CrossfadeMultiple extends Activity {
Button mButton;
Crossfade mCrossfade;
TransitionSet mCrossfadeGroup;
TransitionSet mTextChangeGroup1, mTextChangeGroup2;
TransitionSet mTextChangeGroup1, mTextChangeGroup2, mTextChangeGroup3;
TransitionSet mInOutGroup;
@Override
@@ -74,18 +74,21 @@ public class CrossfadeMultiple extends Activity {
mInOutGroup.addTransition(inOut).addTransition(changeBounds);
mTextChangeGroup1 = new TransitionSet();
TextChange textChangeInOut = new TextChange();
textChangeInOut.setChangeBehavior(TextChange.CHANGE_BEHAVIOR_OUT_IN);
mTextChangeGroup1.addTransition(textChangeInOut).addTransition(new ChangeBounds());
ChangeText changeTextInOut = new ChangeText();
changeTextInOut.setChangeBehavior(ChangeText.CHANGE_BEHAVIOR_OUT_IN);
mTextChangeGroup1.addTransition(changeTextInOut).addTransition(new ChangeBounds());
mTextChangeGroup2 = new TransitionSet();
mTextChangeGroup2.setOrdering(TransitionSet.ORDERING_SEQUENTIAL);
TextChange textChangeOut = new TextChange();
textChangeOut.setChangeBehavior(TextChange.CHANGE_BEHAVIOR_OUT);
TextChange textChangeIn = new TextChange();
textChangeIn.setChangeBehavior(TextChange.CHANGE_BEHAVIOR_IN);
mTextChangeGroup2.addTransition(textChangeOut).addTransition(new ChangeBounds()).
addTransition(textChangeIn);
ChangeText changeTextOut = new ChangeText();
changeTextOut.setChangeBehavior(ChangeText.CHANGE_BEHAVIOR_OUT);
mTextChangeGroup2.addTransition(changeTextOut).addTransition(new ChangeBounds());
mTextChangeGroup3 = new TransitionSet();
mTextChangeGroup3.setOrdering(TransitionSet.ORDERING_SEQUENTIAL);
ChangeText changeTextIn = new ChangeText();
changeTextIn.setChangeBehavior(ChangeText.CHANGE_BEHAVIOR_IN);
mTextChangeGroup3.addTransition(changeTextIn).addTransition(new ChangeBounds());
}
public void sendMessage(View view) {
@@ -135,6 +138,9 @@ public class CrossfadeMultiple extends Activity {
case R.id.textfade2:
mTransition = mTextChangeGroup2;
break;
case R.id.textfade3:
mTransition = mTextChangeGroup3;
break;
}
}
}