mdonath / hit-app

De app voor de HIT
0 stars 0 forks source link

App moet niet sluiten na het drukken op de Vorige knop. #14

Closed mdonath closed 10 years ago

mdonath commented 10 years ago

Er moet iets van een callstack worden bijgehouden zodat je makkelijk terug kan bladeren. Mooi voorbeeld van hoe het mis gaat, is als je via een plaats naar een kamp gaat. Je wil dan terug naar de plaats. In plaats (no-pun-intended) daarvan sluit de hele app. Dat verwacht je niet. Een oplossing is dus om ervoor te zorgen dat als je via plaats naar een kamp gaat en dan op back drukt, dat dan de app niet meteen zonder waarschuwing afsluit.

mdonath commented 10 years ago

In eerste instantie cool op te lossen met de 'close on double back' methode.

private static long back_pressed;

@Override
public void onBackPressed()
{
        if (back_pressed + 2000 > System.currentTimeMillis()) super.onBackPressed();
        else Toast.makeText(getBaseContext(), "Press once again to exit!", Toast.LENGTH_SHORT).show();
        back_pressed = System.currentTimeMillis();
}
BasDV commented 10 years ago

Is het sluiten van de app door alle activiteiten met de terug knop weg te doen niet standaard functionaliteit in Android? Je oplossing met de toast is wel handig om de gebruiker te waarschuwen.

mdonath commented 10 years ago

Ja, dat is standaard. Maar: als je nu door de app heen navigeert, zal bij de eerste de beste druk op de knop de app meteen sluiten. Aangezien we nu (nog) geen callstack/history/whatever hebben, is dit een makkelijke oplossing om te voorkomen dat een app per ongeluk wordt gesloten. Het gebeurt me nu zelf al te vaak en het is behoorlijk irritant.

mdonath commented 10 years ago

De double-back-to-exit optie heb ik net ingebouwd. Works like a charm. Het implementeren van een callstack moet nog, maar kan misschien gecombineerd worden met het issue over swipen.

mdonath commented 10 years ago

Bij nader inzien nieuw issue aangemaakt en deze kan gesloten worden.