Record UI progression is awkward and adds extra Activities to the stack

[multidynamic]

Matthew Adler created an issue 2014-12-10
The way screens are pushed/popped as well as their transitions are awkward. I'd expect to be returned after indicating my recording. Or is the intention user wants to make multiple recordings?

Halsey Burgund REPO OWNER 2014-12-10
FYI, this is something that didn't happen in older versions of Android RW apps (mainly running on 2.3), so it may be related to 4/5? Or some code may have fundamentally changed recently...not sure. If you think it's possibly the latter, I can get you access to the apps that don't have this issue so you can poke around in the code if that would be useful.
And the intention is NOT that the user would want to make multiple recordings immediately. Users can make multiple recordings by actively pressing the "Record Again" button on the Thank You screen, but there shouldn't be any automatic expectation of multiple recordings.

[13rac1]

Part of the problem is:

roundware-android/rwapp/src/main/java/org/roundware/rwapp/RwSpeakActivity.java

Line 515 in 7e3930d

startActivity(new Intent(getApplicationContext(), ClassRegistry.get("RwListenActivity")));

mListenMoreButton.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        // TODO: quick hack - not the best way to do it
        startActivity(new Intent(getApplicationContext(), ClassRegistry.get("RwListenActivity")));
    }
});

[multidynamic]

@eosrei perhaps instead of pushing a ListenActivity, maybe we can just pop the activity stack. By calling finish() on the current activity. (Effectively going back.) You could make the icon conditional on whether it was pushed from the MainActivity or the ListenActivity.

[multidynamic]

I noticed the handling a back at the Android share dialog sends the user back to the choice webview, but with the selections unfunctional! Add something to catch that.

As a user I sort of expect the 3 2 1 countdown to begin after the user affirms the re-record alert dialog, but thats minor.

[hburgund]

Has anyone looked at the older code to determine what is different now? I have code for a different app that implements the original code for this app and it doesn't have this issue. I know a ton of changes have happened by several different people, so it seems possible something got wacko in the process. Happy to share this code if you think it might be useful.

[13rac1]

Here's the code from the initial commit (FYI it's the same):

roundware-android/SFMS/src/main/java/edu/si/sfms/SFMSSpeakActivity.java

Line 530 in f6c7fdf

startActivity(new Intent(getApplicationContext(), SFMSListenActivity.class));

[multidynamic]

Upon investigation I noticed several issues related to this one. I am going to make this the parent issue and track the other serious issues separately.

[multidynamic]

I also noticed cosmetic issues:

• change '2. listen' step to '2. playback' or similar...
• disabled labels should be gray
• remove cancel button on top or replace to as an android 'up' action
• change label from 'record again' to 'record another' or similar...
• change 'tell a friend' label on dialog to 'share'
• remove wonky transitions
• fix db meter the level meter needs to be fixed #13
• short screen Speak on short screen #45