Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d0b6bfaa authored by Tony Wickham's avatar Tony Wickham
Browse files

Remove some unused hotseat/qsb scaling code

Now that QSB is always part of Hotseat, we don't need
getQsbScaleAndTranslation(), and now that HOTSEAT_ICONS are never in
Overview, we can remove the getHotseatScaleAndTranslation() override

Test: Everything works as before
Bug: 175137718
Change-Id: I902815d8e270269c1a2a9af54fa8480e5aef929d
parent 4076cae8
Loading
Loading
Loading
Loading
+0 −14
Original line number Diff line number Diff line
@@ -81,20 +81,6 @@ public class BackgroundAppState extends OverviewState {
        return false;
    }

    @Override
    public ScaleAndTranslation getHotseatScaleAndTranslation(Launcher launcher) {
        if ((getVisibleElements(launcher) & HOTSEAT_ICONS) != 0) {
            // Translate hotseat offscreen if we show it in overview.
            RecentsView recentsView = launcher.getOverviewPanel();
            ScaleAndTranslation scaleAndTranslation = super.getHotseatScaleAndTranslation(launcher);
            scaleAndTranslation.translationY += LayoutUtils.getShelfTrackingDistance(launcher,
                    launcher.getDeviceProfile(),
                    recentsView.getPagedOrientationHandler());
            return scaleAndTranslation;
        }
        return super.getHotseatScaleAndTranslation(launcher);
    }

    @Override
    protected float getDepthUnchecked(Context context) {
        return 1f;
+0 −29
Original line number Diff line number Diff line
@@ -77,40 +77,11 @@ public class OverviewState extends LauncherState {
        return new ScaleAndTranslation(scale, 0, -getDefaultSwipeHeight(launcher) * parallaxFactor);
    }

    @Override
    public ScaleAndTranslation getHotseatScaleAndTranslation(Launcher launcher) {
        if ((getVisibleElements(launcher) & HOTSEAT_ICONS) != 0) {
            DeviceProfile dp = launcher.getDeviceProfile();
            if (dp.allAppsIconSizePx >= dp.iconSizePx) {
                return new ScaleAndTranslation(1, 0, 0);
            } else {
                float scale = ((float) dp.allAppsIconSizePx) / dp.iconSizePx;
                // Distance between the screen center (which is the pivotY for hotseat) and the
                // bottom of the hotseat (which we want to preserve)
                float distanceFromBottom = dp.heightPx / 2 - dp.hotseatBarBottomPaddingPx;
                // On scaling, the bottom edge is moved closer to the pivotY. We move the
                // hotseat back down so that the bottom edge's position is preserved.
                float translationY = distanceFromBottom * (1 - scale);
                return new ScaleAndTranslation(scale, 0, translationY);
            }
        }
        return getWorkspaceScaleAndTranslation(launcher);
    }

    @Override
    public float[] getOverviewScaleAndOffset(Launcher launcher) {
        return new float[] {NO_SCALE, NO_OFFSET};
    }

    @Override
    public ScaleAndTranslation getQsbScaleAndTranslation(Launcher launcher) {
        if (this == OVERVIEW) {
            // Treat the QSB as part of the hotseat so they move together.
            return getHotseatScaleAndTranslation(launcher);
        }
        return super.getQsbScaleAndTranslation(launcher);
    }

    @Override
    public PageAlphaProvider getWorkspacePageAlphaProvider(Launcher launcher) {
        return new PageAlphaProvider(DEACCEL_2) {
+0 −4
Original line number Diff line number Diff line
@@ -176,10 +176,6 @@ public abstract class LauncherState implements BaseState<LauncherState> {
        return launcher.getNormalOverviewScaleAndOffset();
    }

    public ScaleAndTranslation getQsbScaleAndTranslation(Launcher launcher) {
        return new ScaleAndTranslation(NO_SCALE, NO_OFFSET, NO_OFFSET);
    }

    public float getOverviewFullscreenProgress() {
        return 0;
    }
+0 −1
Original line number Diff line number Diff line
@@ -96,7 +96,6 @@ public class WorkspaceStateTransitionAnimation {
        ScaleAndTranslation scaleAndTranslation = state.getWorkspaceScaleAndTranslation(mLauncher);
        ScaleAndTranslation hotseatScaleAndTranslation = state.getHotseatScaleAndTranslation(
                mLauncher);
        ScaleAndTranslation qsbScaleAndTranslation = state.getQsbScaleAndTranslation(mLauncher);
        mNewScale = scaleAndTranslation.scale;
        PageAlphaProvider pageAlphaProvider = state.getWorkspacePageAlphaProvider(mLauncher);
        final int childCount = mWorkspace.getChildCount();
+2 −3
Original line number Diff line number Diff line
@@ -110,6 +110,8 @@ public class DragLayer extends BaseDragLayer<Launcher> {
        mDragController = dragController;
        recreateControllers();

        mOverviewScrim.setup();

        mWorkspaceDragScrim = new WorkspaceDragScrim((this));
        mWorkspaceDragScrim.setWorkspace(workspace);

@@ -118,8 +120,6 @@ public class DragLayer extends BaseDragLayer<Launcher> {
        mRootView = (LauncherRootView) getParent();
        mSysUiScrim = new SysUiScrim(mRootView);
        mRootView.setSysUiScrim(mSysUiScrim);


    }

    @Override
@@ -555,7 +555,6 @@ public class DragLayer extends BaseDragLayer<Launcher> {
    public void setInsets(Rect insets) {
        super.setInsets(insets);
        mSysUiScrim.onInsetsChanged(insets, mAllowSysuiScrims);
        mOverviewScrim.onInsetsChanged(insets);
    }

    public WorkspaceDragScrim getWorkspaceDragScrim() {
Loading