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

UX of "Download.." command under the cube menu #1499

Closed
lidel opened this issue May 12, 2020 · 2 comments · Fixed by #2061
Closed

UX of "Download.." command under the cube menu #1499

lidel opened this issue May 12, 2020 · 2 comments · Fixed by #2061
Labels
area/webui Issues specific to interaction with ipfs-webui effort/days Estimated to take multiple days, but less than a week exp/expert Having worked on the specific codebase is important kind/enhancement A net-new feature or improvement to an existing feature need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up topic/design-ux UX strategy, research, not solely visual design

Comments

@lidel
Copy link
Member

lidel commented May 12, 2020

This is mostly a bunch of notes so we don't forget to tackle this UX gap

Problems

We are missing progress reporting for "Download..." feature under the cube menu.
User is unable to see what is happening, nor to cancel the download if they choose to do so.

AFAIK there is no API for tracking background downloads in go-ipfs, so the download would have to happen in ipfs-desktop process, and progress needs to be passed to WebUI somehow.

Unify "Download" actions

To reduce the amount of code, we could unify behavior of "Download.." command under the cube menu Desktop with "Download" action from context menu present in Web UI Files.

Namely:

  • "Download.." command under the cube menu opens WebUI Files and shows download progress, reusing the new UI introduced for imports and downloads.
  • When a big directory tree is being downloaded user can close WebUI and download continues as long ipfs-desktop is running.
  • When user opens WebUI again, the ongoing download progress indicator is still there and user can cancel it if they want.

It will require ipfs-desktop to send "background download" status info to webui somehow. May be tricky if we decide to stop embedding webui in Electron and open it in external browser.

@lidel lidel added kind/enhancement A net-new feature or improvement to an existing feature topic/design-ux UX strategy, research, not solely visual design need/triage Needs initial labeling and prioritization need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) area/webui Issues specific to interaction with ipfs-webui labels May 12, 2020
@jessicaschilling
Copy link
Contributor

Added to epic ipfs/ipfs-gui#88 since it's part of the whole progress indicator mental model.

@jessicaschilling jessicaschilling added exp/expert Having worked on the specific codebase is important effort/days Estimated to take multiple days, but less than a week P1 High: Likely tackled by core team if no one steps up and removed need/triage Needs initial labeling and prioritization labels May 12, 2020
@NatoBoram
Copy link

AFAIK there is no API for tracking background downloads in go-ipfs

Not yet! ipfs/kubo#4028

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/webui Issues specific to interaction with ipfs-webui effort/days Estimated to take multiple days, but less than a week exp/expert Having worked on the specific codebase is important kind/enhancement A net-new feature or improvement to an existing feature need/analysis Needs further analysis before proceeding need/maintainers-input Needs input from the current maintainer(s) P1 High: Likely tackled by core team if no one steps up topic/design-ux UX strategy, research, not solely visual design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants