younginnovations / aidstream-old

[Deprecated] AidStream is a platform for small and medium size NGOs to publish their aid data in IATI standard.
http://aidstream.org
3 stars 13 forks source link

Form to add a sector confusing #112

Closed rolfkleef closed 8 years ago

rolfkleef commented 9 years ago

The form to add sector codes is confusing:

2015-09-25_01

  1. The narrative field is not needed for most vocabularies (the guidance document suggests to only use it for RO and RO2), but the mandatory Text* field suggests something needs to be entered. Maybe it can be made to show only when RO or RO2 is selected as vocabulary?
  2. "Remove this" and "Add more" are repeated at 3. "Remove this narrative" and "Add another narrative" would be clearer. Also: trying to remove a single narrative doesn't work.
  3. Better labels would be "Remove this sector" and "Add another sector".
rolfkleef commented 9 years ago

As suggested by @SJ-bond in https://github.com/younginnovations/aidstream-v201-migration/issues/22 for another element: perhaps it makes sense to indent the narrative block in many of such forms (css in style-wep.css):

2015-09-29_01

(And relabeling may add clarity)

anjesh commented 9 years ago

We have slightly indented the inner forms to separate from the container wherever they are present. You will need to do hard-refresh to see the change.

image

anjesh commented 8 years ago

Please reopen if there's anything with this issue.