Merge "Hide QS customizer correctly"

This commit is contained in:
Jason Monk
2016-10-19 15:05:41 +00:00
committed by Gerrit Code Review
2 changed files with 9 additions and 3 deletions

View File

@@ -215,7 +215,7 @@ public class QSPanel extends LinearLayout implements Tunable, Callback {
} }
public void onCollapse() { public void onCollapse() {
if (mCustomizePanel != null && mCustomizePanel.isCustomizing()) { if (mCustomizePanel != null && mCustomizePanel.isShown()) {
mCustomizePanel.hide(mCustomizePanel.getWidth() / 2, mCustomizePanel.getHeight() / 2); mCustomizePanel.hide(mCustomizePanel.getWidth() / 2, mCustomizePanel.getHeight() / 2);
} }
} }
@@ -392,7 +392,7 @@ public class QSPanel extends LinearLayout implements Tunable, Callback {
} }
public void closeDetail() { public void closeDetail() {
if (mCustomizePanel != null && mCustomizePanel.isCustomizing()) { if (mCustomizePanel != null && mCustomizePanel.isShown()) {
// Treat this as a detail panel for now, to make things easy. // Treat this as a detail panel for now, to make things easy.
mCustomizePanel.hide(mCustomizePanel.getWidth() / 2, mCustomizePanel.getHeight() / 2); mCustomizePanel.hide(mCustomizePanel.getWidth() / 2, mCustomizePanel.getHeight() / 2);
return; return;

View File

@@ -162,6 +162,10 @@ public class QSCustomizer extends LinearLayout implements OnMenuItemClickListene
} }
} }
public boolean isShown() {
return isShown;
}
private void setCustomizing(boolean customizing) { private void setCustomizing(boolean customizing) {
mCustomizing = customizing; mCustomizing = customizing;
mQsContainer.notifyCustomizeChanged(); mQsContainer.notifyCustomizeChanged();
@@ -216,7 +220,9 @@ public class QSCustomizer extends LinearLayout implements OnMenuItemClickListene
private final AnimatorListener mExpandAnimationListener = new AnimatorListenerAdapter() { private final AnimatorListener mExpandAnimationListener = new AnimatorListenerAdapter() {
@Override @Override
public void onAnimationEnd(Animator animation) { public void onAnimationEnd(Animator animation) {
if (isShown) {
setCustomizing(true); setCustomizing(true);
}
mNotifQsContainer.setCustomizerAnimating(false); mNotifQsContainer.setCustomizerAnimating(false);
} }