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

Commit e3795f2b authored by James Lemieux's avatar James Lemieux
Browse files

TimerFragment should only adjust the fab when it is selected 

TimerFragment has a vertical pager that presents a vertical
list of all defined timers. If changes occur to that pager
(as they do in TimerFragment.onResume()), it was possible
for the TimerFragment to incorrectly adjust the fab icon
when it was not the selected tab.

Bug: 22159053
Change-Id: Ie62fbf0a2a7bee4dd69660422ff8741de6cb20c8
parent b4db2297

src/com/android/deskclock/timer/TimerFragment.java

+8 −3
Original line number Diff line number Diff line
@@ -351,11 +351,16 @@ public class TimerFragment extends DeskClockFragment implements OnSharedPreferen 
    }



    private void setTimerViewFabIcon(TimerObj timer) {

        final Context context = getActivity();

        if (context == null || timer == null || mFab == null) {

        final DeskClock deskClock = (DeskClock) getActivity();

        if (deskClock == null || timer == null || mFab == null) {

            return;

        }

        final Resources r = context.getResources();



        if (deskClock.getSelectedTab() != DeskClock.TIMER_TAB_INDEX) {

            return;

        }



        final Resources r = deskClock.getResources();

        switch (timer.mState) {

            case TimerObj.STATE_RUNNING:

                mFab.setVisibility(View.VISIBLE);