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

Download workflow: compression scheme (or lack thereof) for multiple files/folders #1629

Open
jessicaschilling opened this issue Sep 14, 2020 · 3 comments
Labels
area/screen/files Issues related to Files screen effort/days Estimated to take multiple days, but less than a week exp/intermediate Prior experience is likely helpful help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature P3 Low: Not priority right now status/ready Ready to be worked

Comments

@jessicaschilling
Copy link
Contributor

As originally discussed in ipfs/ipfs-desktop#1639: Current practice of downloading multiple files/folders results in the user receiving a .tar.gz file. Investigate and possibly implement the following to improve overall UX:

  • Default to using .zip as a compression scheme since not all OSes natively support .tar.gz uncompression
  • Add a prefs option for .zip vs .tar.gz for those who'd prefer the smaller files but possibly more work in uncompressing
  • Opt out of compression altogether if user is just downloading something off localhost (is there a foolproof way to detect this?); this could also be a prefs option
@jessicaschilling jessicaschilling added help wanted Seeking public contribution on this issue exp/intermediate Prior experience is likely helpful P2 Medium: Good to have, but can wait until someone steps up status/ready Ready to be worked kind/enhancement A net-new feature or improvement to an existing feature effort/days Estimated to take multiple days, but less than a week labels Sep 14, 2020
@lidel
Copy link
Member

lidel commented Sep 14, 2020

Adding support for ZIP archive format requires changes to go-ipfs/js-ipfs.

If we want to prioritize improvements that do not require landing changes to the backend:

  • Disable download compression by default
    Rationale: nearly all users are running local node so download is instant and compression only slows things down
  • Make it possible to enable compression via Settings screen:
    • Add Files section somewhere (maybe between Language and Analytics?) with a single checkbox option to "Enable compression of downloaded TAR archives" (unchecked by default)
  • Research and document open source software that is able to open .tar.gz and .tar.gz on each platform (macOS, Windows, Linux)
    • I suspect we only need a hint that is displayed for Windows users
    • For the sake of discoverability, it could be a link on Settings screen, perhaps a short explainer next to/below the compression checkbox?

@lidel lidel added the area/screen/files Issues related to Files screen label Sep 15, 2020
@jessicaschilling
Copy link
Contributor Author

Mockup below. I'd recommend just explaining that the user may need a decompression app rather than creating an ongoing-maintenance item for keeping a list of third-party apps up to date.

image.png

@jessicaschilling jessicaschilling added this to the v2.12 milestone Sep 21, 2020
@lidel lidel modified the milestones: v2.13, v2.14 Apr 13, 2021
@SgtPooki
Copy link
Member

SgtPooki commented Dec 2, 2022

I'm not sure if this is solved by #1894, but we should double check before starting work on this

@SgtPooki SgtPooki removed this from the v2.14 milestone Dec 2, 2022
@SgtPooki SgtPooki added P3 Low: Not priority right now and removed P2 Medium: Good to have, but can wait until someone steps up labels Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/screen/files Issues related to Files screen effort/days Estimated to take multiple days, but less than a week exp/intermediate Prior experience is likely helpful help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature P3 Low: Not priority right now status/ready Ready to be worked
Projects
No open projects
Status: Planned / Backlog
Development

No branches or pull requests

3 participants