Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Close the app when pressing back button/doing back gesture on the main screen #395

Merged
merged 2 commits into from May 19, 2023

Conversation

kejml
Copy link
Contributor

@kejml kejml commented Apr 23, 2023

This is standard behaviour Android, I don't know about iOS

This would fix #189 and fix #347

…n screen.

This is standard behaviour Android, I don't know about iOS

This would fix lucien144#189 and fix lucien144#347
@lucien144
Copy link
Owner

Tak jednoduché to nebude :) Ten WillPopScope tam dělá to, že se nedostaneš swipem z HP screeny zpět na ten login screen.

@kejml
Copy link
Contributor Author

kejml commented Apr 24, 2023

Na androidu to funguje, z hlavní screeny pop ukončí aplikaci a při znovuotevření Fyxu jsem přihlášený. Na iOSu je to jinak? Nebo kde je problém?

@kejml
Copy link
Contributor Author

kejml commented Apr 29, 2023

Už jsem pochopil jak jsi to myslel a vyřešil jsem to

@lucien144
Copy link
Owner

Jo. Takhle nějak by to mělo být. Dík, zkusím mergnout vše přes pondělí.

@lucien144 lucien144 added this to the v0.9.2 milestone May 19, 2023
@lucien144 lucien144 merged commit 69bab20 into lucien144:develop May 19, 2023
1 check passed
lucien144 added a commit that referenced this pull request May 19, 2023
@lucien144
Copy link
Owner

@kejml Tak to ještě nebylo úplně ono, viz. 667a305 kdyby tě to zajímalo. Zapré tam chyběl ještě ten pushNamedAndRemoveUntil na login screeně. Zadruhé routing je udělaný přes onGenerateRoute (viz. níže), u kterého je v dokumentaci napsané, že pushNamedAndRemoveUntil se musí volat s callbackem. Předělal jsem to, že ten callback vrací false, čímž se celý ten Navigator vysype - což by asi mělo být správně vzhledem k tomu, že pokud se zaloguješ, historie by se měla vymazat.

onGenerateRoute: FyxApp.routes,

@kejml
Copy link
Contributor Author

kejml commented May 19, 2023

Aha, zajímavý. Jak jsi má to přišel? Jsi narazil na nějaký rozbitý chování, nebo jsi jen studoval dokumentaci? Já dělal jen rychlej zest a přišlo mi to ok, ale hlubice jsem to netestoval. Upřímně zrovna tahle věc mi vůbec nechyběla :)

@lucien144
Copy link
Owner

Hele chtěl jsem si jen ověřit funkčnost, plus mi došlo, že to bude ještě na login screenu. A ještě jsem chtěl smazat ten WillScope protože je zbytečný už.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

android vyskočení z aplikace "tlačítko" zpět
2 participants