PeerSay / Atlas

The PeerSay MVP project
0 stars 0 forks source link

RFE: Add a user description for a requirement #95

Closed talbendavid closed 9 years ago

talbendavid commented 9 years ago

Allow users to add their own description to a criteria, so add the specific meaning for their evaluation.

assafharel commented 9 years ago

Tal, please suggest a UI for this, as there's no place for it right now. Assigning the issue back to you, as Pavel should not be the one deciding that.

assafharel commented 9 years ago

The suggestion: make the description field editable. Users will be able to edit and change / add to the original description.

Pavek commented 9 years ago

Just to clarify: do you mean editable in Requirements page table? Asking becasue I think user asked to add it to Decision table, and this solution doesn't affect Decision table as description is not displayed there.

assafharel commented 9 years ago

Yes, I mean in requirements page table. Users read the requirement descriptions and wanted to change / add / clarify them to their best understanding.

Assaf

Pavek commented 9 years ago

Also remebered: editable description can be only for selected items. Unselected items reside in read-only global collection. Selected are the copies of global ones to Project doc and are indeed editable.

Note: it leads to some shaky consequences: e.g. unselecting item effectively removes customs description forever and then it is a dilemma: add special code to roll-back global description immediately or leave it and let user see it on next API call / page re-visit.

assafharel commented 9 years ago

After a user changes a description, she doesn't want to see the old one any more (regardless of the selection status). You may add a delete or rollback, etc. option that brings back the "old" description from the server.

Pavek commented 9 years ago

Here's alternative idea to edit description, which I think is more consistent with other UI and don't produce unwanted-side-effects:

Thoughts?

assafharel commented 9 years ago

I like it. Looks like the quickest and most easy fix. Go ahead. We will test it with users, and re-open if it's not clear to the users.

assafharel commented 9 years ago

Re-openning. When clicking edit on a criteria at the bottom of the page, you don't see that it changed the new fields at the top. So the suggestion is to add an edit line below the line that is currently being edited. That line will be used only to edit that line above it, and you close it with a close button or x or something like that.

Assaf

assafharel commented 9 years ago

With the new accordion control, there's a slight tweak to the new requirement as well: It should be allow to add a new requirement for a topic, by clicking +New on the topic bar. This should open the +"New" control directly below the topic (with topic field already filled or not exist (as it's known)).

Pavek commented 9 years ago

This is somewhat in conflict with what I changed recently, namely:

Note: setting a form's topic value to currently open Topic (if any) is fine with me.

Pavel

On Mon, Jul 13, 2015 at 11:11 AM, Assaf Harel notifications@github.com wrote:

With the new accordion control, there's a slight tweak to the new requirement as well: It should be allow to add a new requirement for a topic, by clicking +New on the topic bar. This should open the +"New" control directly below the topic (with topic field already filled or not exist (as it's known)).

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-120844542.

assafharel commented 9 years ago

I'm not sure I understand the conflict with what I've wrote, but assuming you're going to open the "+New form" immediately under the line (with that line information already written) that is being edited, or the topic that is being a criteria is being added to, I agree.

Assaf

assafharel commented 9 years ago

Re-opening. 1 problem, and 1 missing implementation.

Problem: It shouldn't be allowed to edit the Name and the topic. They can simply not exist in the control, or be read-only.

Missing: The option to add new to a topic. This is highly required, with the new accordion UI, as the user is adding requirements topic by topic, so the notion of a missing requirement is usually in the context of a topic.

Assaf

Pavek commented 9 years ago
  1. 'edit the Name and the topic.' OK, but my opinion this is bad design.

We will test it with users, and see their behavior. You're assuming that read-only will be annoying, and we think that anything else confuses the users, so we'll just wait and see their reaction.

  1. Missing: The option to add new to a topic. (I guess this is about adding new Requirement to currently open Topic) This is implemented according to what we discussed - currently open Topic (if any) sets the default Topic for newly added Requirements, that is:
    • if currently open Topics is X, and a) user clicks New button => addNew form is opened with topic X selected by default (can be changed of course)

What I'm missing is a "Add New" control in the topic ribbon that opens the form under the topic. The user is unaware of the "Add New" button at the top of the page, when working on a specific topic.

b) user uses search and hits Enter when no items found => addNew form is opened with topic X and Name as entered in search

Pavel

On Wed, Jul 15, 2015 at 10:15 AM, Assaf Harel notifications@github.com wrote:

Reopened #95 https://github.com/PeerSay/Atlas/issues/95.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#event-356154799.

assafharel commented 9 years ago
  1. 'edit the Name and the topic.' OK, but my opinion this is bad design.

We will test it with users, and see their behavior. You're assuming that read-only will be annoying, and we think that anything else confuses the users, so we'll just wait and see their reaction.

  1. Missing: The option to add new to a topic. (I guess this is about adding new Requirement to currently open Topic) This is implemented according to what we discussed - currently open Topic (if any) sets the default Topic for newly added Requirements, that is:
    • if currently open Topics is X, and a) user clicks New button => addNew form is opened with topic X selected by default (can be changed of course)

What I'm missing is a "Add New" control in the topic ribbon that opens the form under the topic. The user is unaware of the "Add New" button at the top of the page, when working on a specific topic.

b) user uses search and hits Enter when no items found => addNew form is opened with topic X and Name as entered in search

Pavel

On Wed, Jul 15, 2015 at 10:15 AM, Assaf Harel notifications@github.com wrote:

Reopened #95 https://github.com/PeerSay/Atlas/issues/95.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#event-356154799.

assafharel commented 9 years ago

What I'm missing is an "Add New" control in the topic ribbon that opens the form under the topic. The user is unaware of the "Add New" button at the top of the page, when working on a specific topic.

BTW, a 2nd click on the edit button should close the form (just like a 2nd click on the "+New” button close the form on top.

Thanks, Assaf

Assaf Harel, CTO and Co-Founder, +972.52.7206732

On Jul 15, 2015, at 3:30 PM, Pavel Zhytko notifications@github.com wrote:

  1. 'edit the Name and the topic.' OK, but my opinion this is bad design.
  2. Missing: The option to add new to a topic. (I guess this is about adding new Requirement to currently open Topic) This is implemented according to what we discussed - currently open Topic (if any) sets the default Topic for newly added Requirements, that is:
  3. if currently open Topics is X, and a) user clicks New button => addNew form is opened with topic X selected by default (can be changed of course) b) user uses search and hits Enter when no items found => addNew form is opened with topic X and Name as entered in search
    • if not Topics are open (all accordion items are closed), then the same as above except that default topic is empty

Pavel

On Wed, Jul 15, 2015 at 10:15 AM, Assaf Harel notifications@github.com wrote:

Reopened #95 https://github.com/PeerSay/Atlas/issues/95.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#event-356154799.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-121601972.

Pavek commented 9 years ago

Just to clarify the requirements + questions:

BTW, a 2nd click on the edit button should close the form (just like a 2nd click on the "+New” button close the form on top. OK

Pavel

On Wed, Jul 15, 2015 at 4:32 PM, Assaf Harel notifications@github.com wrote:

What I'm missing is an "Add New" control in the topic ribbon that opens the form under the topic. The user is unaware of the "Add New" button at the top of the page, when working on a specific topic.

BTW, a 2nd click on the edit button should close the form (just like a 2nd click on the "+New” button close the form on top.

Thanks, Assaf

Assaf Harel, CTO and Co-Founder, +972.52.7206732

On Jul 15, 2015, at 3:30 PM, Pavel Zhytko notifications@github.com wrote:

  1. 'edit the Name and the topic.' OK, but my opinion this is bad design.
  2. Missing: The option to add new to a topic. (I guess this is about adding new Requirement to currently open Topic) This is implemented according to what we discussed - currently open Topic (if any) sets the default Topic for newly added Requirements, that is:
  3. if currently open Topics is X, and a) user clicks New button => addNew form is opened with topic X selected by default (can be changed of course) b) user uses search and hits Enter when no items found => addNew form is opened with topic X and Name as entered in search
    • if not Topics are open (all accordion items are closed), then the same as above except that default topic is empty

Pavel

On Wed, Jul 15, 2015 at 10:15 AM, Assaf Harel notifications@github.com wrote:

Reopened #95 https://github.com/PeerSay/Atlas/issues/95.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#event-356154799.

— Reply to this email directly or view it on GitHub < https://github.com/PeerSay/Atlas/issues/95#issuecomment-121601972>.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-121617982.

assafharel commented 9 years ago

Don't remove it from the top and don't remove it from products. Just add a new one per topic. The top New still responds to search.

Pavek commented 9 years ago

OK - got it.

Having no location hints, should I assume this? http://gyazo.com/2976a14ebc86663615275d6990617e74

Pavel

On Wed, Jul 15, 2015 at 5:55 PM, Assaf Harel notifications@github.com wrote:

Don't remove it from the top and don't remove it from products. Just add a new one per topic. The top New still responds to search.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-121642928.

assafharel commented 9 years ago

Better put it to the right.

Assaf בתאריך Wed, Jul 15, 2015 ב-18:13 מאת Pavel Zhytko <notifications@github.com

:

OK - got it.

Having no location hints, should I assume this? http://gyazo.com/2976a14ebc86663615275d6990617e74

Pavel

On Wed, Jul 15, 2015 at 5:55 PM, Assaf Harel notifications@github.com wrote:

Don't remove it from the top and don't remove it from products. Just add a new one per topic. The top New still responds to search.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-121642928.

— Reply to this email directly or view it on GitHub https://github.com/PeerSay/Atlas/issues/95#issuecomment-121647330.

assafharel commented 9 years ago

One more thing: have the +New visible only when the topic is open.