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

Make pinning hashes more intuitive #593

Closed
olizilla opened this issue Jan 12, 2018 · 7 comments
Closed

Make pinning hashes more intuitive #593

olizilla opened this issue Jan 12, 2018 · 7 comments

Comments

@olizilla
Copy link
Member

Right now, the pin hash feature requires the user to enter an optional title and a hash and click the check mark to confirm.

screen shot 2018-01-12 at 16 14 11

It's not obvious from the UI that the user needs to fill anything out in the green bar.

We can make this more intuitive by:

  • Make it look more like a form. Use a less saturated background color, and add some framing to the input fields with a per input background color and/or border.
  • Use a muted color for the placeholder text.
  • Indicate required and optional fields.
  • Change the placeholder text to show example content
    • title: My favourite pin
    • hash: QmHash...

As the title is optional, to make the action feel simpler, we could simply let the user enter a hash, pin it, and then make it easy for them to add a label while it's pinning.

If you try to pin a hash you already have pinned, we should informed the user, and scroll to the item in the list. Right now the new pin name (or empty name) overwrites any existing label you might have.

To be able to label an unlabelled pinned hash the user needs a way of previewing or opening the file to see what it is.

The add pin icon might be more intuitive in the top right corner, as a button with text for "Add". This applies to the "Add files" action on the files pane too.

@olizilla
Copy link
Member Author

Of note, this is a useful data point for https://github.com/ipfs-shipyard/ipfs-ui-style-guide as it should define some of these things

@hacdias
Copy link
Member

hacdias commented Jan 13, 2018

Hey @olizilla! You're right: the current interface is not the best and doesn't speak for itself. Would you mind to do some kind of sketch to show the way you think the form should be? 😄

@hacdias
Copy link
Member

hacdias commented Jan 20, 2018

@olizilla any updates on this? 😄

@lidel
Copy link
Member

lidel commented Jan 21, 2018

I suggest wait for canonical icons for pin actions: ipfs-inactive/ipfs-ui-style-guide#3 (comment)

@hacdias hacdias added the UX label Jan 29, 2018
@hacdias hacdias added UX and removed UX labels Mar 20, 2018
@nothingismagick
Copy link

The checkmark and the green color together make it seem like it was successful before the user even interacts. It always makes me pause when I see it.

@lidel
Copy link
Member

lidel commented Mar 30, 2018

Ditto. Let's create better UI for pinning!
Ideas and examples of apps with good UI are welcome at:

@hacdias
Copy link
Member

hacdias commented Nov 14, 2018

Closing as unrelated to the newer version.

@hacdias hacdias closed this as completed Nov 14, 2018
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

4 participants