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

Simplify UX Around The State Of Gateway Redirect #325

Closed
lidel opened this issue Dec 2, 2017 · 3 comments
Closed

Simplify UX Around The State Of Gateway Redirect #325

lidel opened this issue Dec 2, 2017 · 3 comments
Assignees
Labels
exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors help wanted Seeking public contribution on this issue
Milestone

Comments

@lidel
Copy link
Member

lidel commented Dec 2, 2017

Based on anecdotal evidence 🙃 less savvy users are often confused about "gateway redirect".
It is mostly my fault. The term is used due to legacy reasons: initial name of this add-on was "ipfs-gateway-redirect".

To make things worse, current GUI always displays local gateway as "HTTP GATEWAY", no matter if it is actually used (enabled via redirect) or not:

Enabled: 2017-12-03-001242_1147x1421_scrot
Disabled: 2017-12-03-001230_1147x1421_scrot

Here's an Idea

"HTTP Gateway" should reflect the URL of currently active gateway:

  • If redirect is enabled, its value is URL of "Custom Gateway" (eg. http://127.0.0.1:8080)
  • If redirect is disabled, its value is URL of "Public Gateway" (eg. https://ipfs.io)

Additionally, we could rename toggle to make it self-explanatory and easier to understand:

  • "Enable Gateway Redirect" → "Switch to Custom Gateway"
  • "Disable Gateway Redirect" → "Switch to Public Gateway"

If we make these two changes, then we can decrease cognitive load by removing "Redirect" status from UI.

@lidel lidel added the UX label Dec 2, 2017
@lidel lidel added this to the 2018-Q1 milestone Dec 2, 2017
@daviddias
Copy link
Member

@lidel I like this proposal 👍

@lidel lidel added exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors help wanted Seeking public contribution on this issue P4 - Very Low labels Dec 6, 2017
@lidel lidel self-assigned this Dec 7, 2017
@lidel
Copy link
Member Author

lidel commented Dec 7, 2017

Done!

  • This change is in v2.1.1beta4 🌵
  • We may be missing translations for two labels. If anyone wants to help with translation, check https://crowdin.com/project/ipfs-companion
  • UX note: at some point in future, during general polish, we may consider cheating a bit, and display new HTTP Gateway immediately (instead of waiting for actual switch to propagate, like it happens in demo below).

peek 2017-12-08 00-06

@lidel lidel closed this as completed Dec 7, 2017
lidel added a commit that referenced this issue Dec 7, 2017
@olizilla
Copy link
Member

I'm all for this! #320 has got too big! I'd included a similar change there, but it's trapped in a larger changeset.

screenshot 2017-12-11 09 47 19

I love how similar these changes have turned out. Feels like we're heading in the same direction!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exp/novice Someone with a little familiarity can pick up good first issue Good issue for new contributors help wanted Seeking public contribution on this issue
Projects
None yet
Development

No branches or pull requests

3 participants