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

Commit d97cf8ad authored by Jeremy Sim's avatar Jeremy Sim
Browse files

Fix small bug with Overview action button placement 

Overview actions will now be centered properly when 3-button nav is active on tablets in landscape.

On most tablet and foldable screens, we shift the Overview action buttons left to accommodate when 3-button nav is active. The only exception is large-screen tablets in landscape orientation, where there is enough room to center them. This patch adds a conditional for that case.

Fixes: 239247432
Test: Manual
Change-Id: I96f9d5e428273b0611463a1f0f5898910eefe5ce
parent 27657927

quickstep/src/com/android/quickstep/views/OverviewActionsView.java

+5 −1
Original line number Diff line number Diff line
@@ -205,7 +205,11 @@ public class OverviewActionsView<T extends OverlayUICallbacks> extends FrameLayo 
        if (mDp == null) {

            return;

        }

        if (mDp.areNavButtonsInline) {

        boolean largeScreenLandscape = mDp.isTablet && !mDp.isTwoPanels && mDp.isLandscape;

        // If in 3-button mode, shift action buttons to accommodate 3-button layout.

        // (Special exception for landscape tablets, where there is enough room and we don't need to

        // shift the action buttons.)

        if (mDp.areNavButtonsInline && !largeScreenLandscape) {

            // Add extra horizontal spacing

            int additionalPadding = mDp.hotseatBarEndOffset;

            if (isLayoutRtl()) {