Page MenuHomeGitPull.it

Allow setting the first screen to show on app startup
ClosedPublic

Authored by fabio.mazza on Jun 10 2022, 01:12.
Referenced Files
F2070843: D88.diff
Fri, Mar 29, 01:33
Unknown Object (File)
Tue, Mar 19, 23:57
Unknown Object (File)
Tue, Mar 19, 23:57
Unknown Object (File)
Tue, Mar 19, 23:57
Unknown Object (File)
Tue, Mar 19, 22:59
Unknown Object (File)
Tue, Mar 19, 22:34
Unknown Object (File)
Thu, Mar 14, 05:27
Unknown Object (File)
Feb 24 2024, 03:26

Details

Summary

Restore view state by hand after changing fragment (Android framework should be doing this automatically, but it doesn't)

Allow setting the first screen to open

fixes T1091 and T1086

Test Plan

Check it builds and that the changes are good (no crashes)

Diff Detail

Repository
R4 Libre BusTO
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 329
Build 471: BusTO Build Plan
Build 470: arc lint + arc unit

Event Timeline

Restricted Application added a reviewer: valerio.bozzolan. · View Herald Transcript
Restricted Application added subscribers: Libre BusTO , valerio.bozzolan. · View Herald Transcript

Due note:
a - i preferiti non si aggiornano quando non ne ho nessuno e ne aggiungo uno, forse perché adesso uso lo stesso fragment?
b - ci sono degli strani artefatti quando si torna indietro oppure si riapre l'app. Capita anche a voi?

Aggiungere
c - la schermata a cui si torna toccando indietro più e più volte rimane quella della ricerca degli arrivi

  • Fix weird fragment randomness by cleaning up methods
  • Fix bug not showing the new favorite when starting from 0

L'ultimo cambiamento dovrebbe avere risolto a), b) e c). Segnalate se ci sono altri problemi

Bro, it works perfectly. Thank you!

sgtm

This revision is now accepted and ready to land.Jun 11 2022, 08:46