ZacSweers / CatchUp

An app for catching up on things.
Apache License 2.0
1.97k stars 201 forks source link

TabLayout restoration sometimes wrong #92

Closed ZacSweers closed 1 year ago

ZacSweers commented 6 years ago

Reported by @hzsweers

Description

The color here should be pink to match dribble, but seems to be hacker news instead

App

Version: 0.1.0-80-g98e1131-dev
Version code: 906

Device details

Make: Google
Model: Pixel 2 XL
Resolution: 2712x1440
Density: 560dpi (560)
Release: 8.1.0
API: 27

Screenshot

Logs

CatchUpApplication$onCreate D Updating daynight
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
ServiceController$loadData D Data load - dn - took: 216ms
ServiceController$loadData D Data load - github - took: 213ms
ServiceController$loadData D Data load - dribbble - took: 226ms
    CatchUpApplication D OnTrimMemory
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
                OkHttp V --> GET https://api.dribbble.com/v1/shots?page=0&per_page=50 h2
                OkHttp V <-- 200 https://api.dribbble.com/v1/shots?page=0&per_page=50 (816ms, unknown-length body)
ServiceController$loadData D Data load - dribbble - took: 1690ms
    CatchUpApplication D OnTrimMemory
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.service.ServiceController was found in io.sweers.catchup.ui.activity.MainActivity
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
                OkHttp V --> GET https://api.dribbble.com/v1/shots?page=0&per_page=50 h2
                OkHttp V --> POST https://api.github.com/graphql http/1.1 (890-byte body)
                OkHttp V --> GET https://www.designernews.co/api/v2/stories?page=0 http/1.1
                OkHttp V <-- 200 OK https://www.designernews.co/api/v2/stories?page=0 (277ms, unknown-length body)
                OkHttp V --> GET https://www.designernews.co/api/v2/users/3681,79377,7761,83292,33659,55164,80421,6470,6075,1959,70088,57885,60702,77484,58922,73648,3080,12412,20780,76348,3520,42864,35670,83076,40018 http/1.1
                OkHttp V <-- 200 https://api.dribbble.com/v1/shots?page=0&per_page=50 (524ms, unknown-length body)
ServiceController$loadData D Data load - dribbble - took: 1103ms
                OkHttp V <-- 200 OK https://www.designernews.co/api/v2/users/3681,79377,7761,83292,33659,55164,80421,6470,6075,1959,70088,57885,60702,77484,58922,73648,3080,12412,20780,76348,3520,42864,35670,83076,40018 (344ms, unknown-length body)
                OkHttp V <-- 200 OK https://api.github.com/graphql (810ms, unknown-length body)
ServiceController$loadData D Data load - dn - took: 1260ms
ServiceController$loadData D Data load - github - took: 1445ms
    CatchUpApplication D OnTrimMemory
    ConductorInjection D An injector for io.sweers.catchup.ui.controllers.PagerController was found in io.sweers.catchup.ui.activity.MainActivity
charlesdurham commented 6 years ago

I can reproduce every time with; don't keep activities -> background -> foreground -> rotate.

ZacSweers commented 6 years ago

Ah good catch! I'll debug this more this weekend