Revise progress UI in offline maps about collect HOT 11 OPEN

They do mention that progress indicators can be displayed in a card too, but they no longer have a reference (maybe I'm going mad). Regardless, I don't think it's necessary. The main thing is having a little descriptor so the user knows what's happening.

I think the weird part about the card for me is that we're not then going to show the layers in a card. Maybe we could just show a circular progress indicator in the space where the layers will be?

from collect.

I’ve been wondering why that step takes so long. It’s not doing any copying, it’s just getting the layer names. Shouldn’t that be close to instant?

It is copying it into a cache. Correct me if I'm wrong here @grzesiek2010, but my understanding was that we're doing that so we can inspect the file metadata (to show the tile names). Unless there's another reason, I am now wondering if we could safely do this via the Uri Android gives us (via the Kotlin extension Uri#toFile that I've just spotted) rather than performing the extra copy into our cache.

from collect.

I believe there's the same delay twice currently.

from collect.

Updated this one @seadowg. Let me know what you think.

from collect.

In both cases we need to disable all the controls right? I'm worried it might be confusing to have disabled stuff on screen while copying the files.

from collect.

In both cases we need to disable all the controls right? I'm worried it might be confusing to have disabled stuff on screen while copying the files.

I think you would only have to disable "Add layers" but they can still cancel or select layer access while it's importing, right? I agree that could be confusing if we had to disable both bottom actions. We can also leave it as is. This was a nice to have.

from collect.

I think you would only have to disable "Add layers" but they can still cancel or select layer access while it's importing, right?

Yeah, that sounds right! I definitely like the direction of 1 more than 2. I'm not sure about the "card" like background, but it could be that I've just never seen it used like that before.

from collect.

I swear they had an image of it in the docs just last week! They do mention that progress indicators can be displayed in a card too, but they no longer have a reference (maybe I'm going mad). Regardless, I don't think it's necessary. The main thing is having a little descriptor so the user knows what's happening.

from collect.

Maybe we could just show a circular progress indicator in the space where the layers will be?

That works for me. I think it would still be helpful to have the label "Importing layers" below the progress indicator just in case it takes a while.

from collect.

I’ve been wondering why that step takes so long. It’s not doing any copying, it’s just getting the layer names. Shouldn’t that be close to instant?

from collect.

@grzesiek2010 I'm thinking we should investigate removing the copy to cache before doing any more design here. It seems like people are seeing a big delay when copying to cache, but then not after "Add layers" for big files (please correct me if I'm wrong anyone). If that's the case, I can imagine that if we do get rid of the copy to cache, we might end up moving that delay to after "Add layers" making optimizing the first progress UI a bit pointless. What do you think?

from collect.