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

[ui polish] Add states (Download, Decrypt) to Files display #571

Closed
ninavizz opened this issue Oct 16, 2019 · 2 comments
Closed

[ui polish] Add states (Download, Decrypt) to Files display #571

ninavizz opened this issue Oct 16, 2019 · 2 comments

Comments

@ninavizz
Copy link
Member

ninavizz commented Oct 16, 2019

Add following states to existing display of Files w/in the client's conversation pane

  • Updated Zeplin thingy
  • Downloading
    • Needs to show while files are downloading to the machine from SD server(s)
  • Decrypting
    • Should always show for a set timeframe; 2-3 seconds, I suspect?
    • Does not need to map to actual decryption time; point is just to regularly reinforce to users, that this happens.
  • Opening
    • Should show immediately after user-action to open file has been triggered; just before disp-vm begins its launch.
    • Should revert to static/downloaded state as (tbd) seconds after invoking, to allow disp-vm to launch so user is not hopelessly confused.
    • Cursor should also change to show i-bar with sand-clock thingy icon for the same duration.
    • Point is to tell user "yo, something is happening!" until visual affordance of the new disp-vm window opens atop sd-svs vm pane
@ninavizz ninavizz changed the title [ui polish] Add states, _Files_ display [ui polish] Add states, Files display Oct 16, 2019
@ninavizz ninavizz changed the title [ui polish] Add states, Files display [ui polish] Add states (Download, Decrypt) to Files display Oct 25, 2019
@eloquence
Copy link
Member

We do have a Download spinner now. I somewhat question the value of separating out the Decryption stage, but this may become more important for larger downloads in the context of #1104.

@zenmonkeykstop
Copy link
Contributor

Closing - we'll revisit the UX for files in the conversation view separately. The current implementation is a bit cluttered already.

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

No branches or pull requests

3 participants