civicmakers / client

This is the code repository for the CivicMakers knowledge sharing experiment.
http://civicmakers.firebaseapp.com
GNU General Public License v3.0
4 stars 9 forks source link

tools list on add project form #179

Open ybentz opened 8 years ago

ybentz commented 8 years ago

Currently trying to add a tool to the tools list in the add project form is very confusing because it's hard for the user to understand that the tool was actually added.

  1. What needs to happen after a click on a tool:
    • The search input should clear to start a new search.
    • The list of available tools should close.
  2. The list of currently selected tools should be visible to the user instead of under the fold. - @olliebeannn can you help with this one?
  3. There should be a way to remove a tool after it was selected. Right now the only way to do it is reload and restart the whole form. Very frustrating! A solution could be to put a small X icon on hover which will be very intuitive for the user but it won't work on mobile. - @olliebeannn, @grodeska, @cpalmatier any ideas?
ybentz commented 8 years ago

I fixed (1) in this PR https://github.com/civicmakers/client/pull/187. I also made it so after a tool was selected it won't show on the selectable tool list.

grodeska commented 8 years ago

@cpalmatier Can you spend some time mocking up a better UI for the add tool feature for creating projects? We'd ideally use same UI for editing projects. As @ybentz mentioned, the big need is the ability to remove a tool once it's been selected, but the UI could be cleaned up all around.

grodeska commented 8 years ago

@ybentz and @cpalmatier we actually already have these designs, courtesy of Ollie: Here's the visual comp that should be used for part 2 of this story: https://app.zeplin.io/project.html#pid=5643f6ea7f758de72a21c434&sid=56cbf772dfa8a9ca17186d81