Project:Drupal core
Version:7.x-dev
Component:taxonomy.module
Category:bug report
Priority:normal
Assigned:Xano
Status:closed (won't fix)
Issue tags:Cleanup, d7uxsprint, Needs text review, UBUserTesting2009, ui-text, Usability

Issue Summary

We need to improve the help text under the advanced fieldset for term creation. People don't understand concepts like parents, related terms, and synonyms, and the help text doesn't do a great job of helping people understand.

Comments

#1

updating tags.

#2

Issue tags:+Needs text review

#3

Attached is a patch for this text. I was a little torn on weather to indicate that the inputs did not have any functionality by default, but after spending a while figuring this out myself, it seems like it might be an important piece of information.

AttachmentSizeStatusTest resultOperations
advanced-taxonomy-before.png6.46 KBIgnored: Check issue status.NoneNone
advanced-taxonomy-after.png21.33 KBIgnored: Check issue status.NoneNone
taxonomy.admin_.patch2.24 KBIdleFailed: Failed to apply patch.View details | Re-test

#4

Status:active» needs review

changing status to CNR. Also fixed a misspelling in the patch. Correction attached.

AttachmentSizeStatusTest resultOperations
taxonomy.admin_1.patch2.24 KBIdleFailed: Failed to apply patch.View details | Re-test

#5

Thanks, karschsp!

#6

wow, guys we are trying to decrease the amount of help text not increase it. I have to give this a thorough review, but few people really had trouble understanding the concept, as soon as they actually did it. So I think its save to assume, this issue although exiting might be solved by something else then a lot of help text (changing labels?)

#7

@bojhan although in general I agree with you, these are advanced features which are totally undocumented in the interface. Do you know what a synonym does? what effect it actually has on the vocabulary? I don't think these features are self explanatory.

I'm open to other ways to convey the meaning of these fields, but as is we're not doing that at all.

#8

Status:needs review» needs work

The last submitted patch failed testing.

#9

@bojhan What would you recommend for the help text on each of those fields?

Here are my suggestions for the text. The "relationship" information is based on the correct terms for semantic relationships in controlled vocabularies. See more at: http://www.slis.kent.edu/~mzeng/Z3919/4relationship.htm

1. Parents
Relationship: hierarchical. Items will exist below (or within) the parent(s) you select.

2. Related Terms
Relationship: associative. Items will exist alongside the ones you select.

3. Synonyms
Relationship: equivalent. Items entered will behave as if they were identical to one another. Enter one term per line.

I'm not convinced it makes sense to repeat the phrase, "X has/have no core functionality, but may be extended by other modules." I think there is a better place to put this information. What if Parents and Weight were grouped, and "Related Terms" and "Synonyms" were also grouped?

#10

The problem with improving things, is that Drupal does not appear to use the advanced options without third party modules. In the opinion of the UX-2009 Sprint team it would be better to move the advanced settings (without weight perhaps) to a third party (support) module.

AttachmentSizeStatusTest resultOperations
taxonomy.admin_.inc_.patch3.5 KBIdleFailed: Failed to apply patch.View details | Re-test

#11

Issue tags:+#d7uxsprint, +ui-text

Adding tags.

#12

Ignore last patch. We need the text a 3rd party module is necessary only once.

AttachmentSizeStatusTest resultOperations
taxonomy.admin_.inc_.patch2.25 KBIdleFailed: Failed to apply patch.View details | Re-test

#13

And this patch renames "Advanced options" to "Relationships" as that is basically what it is.

AttachmentSizeStatusTest resultOperations
taxonomy.admin_.inc_.patch2.46 KBIdleFailed: Failed to apply patch.View details | Re-test

#14

Issue tags:-#d7uxsprint+d7uxsprint

changing d7uxsprint tag

#15

Screenshots before and after the patch #13

before:

after:

AttachmentSizeStatusTest resultOperations
taxonomy-before.jpg45.67 KBIgnored: Check issue status.NoneNone
taxonomy-after.jpg51.17 KBIgnored: Check issue status.NoneNone

#16

Title:Advanced fieldset under create term needs help text» Clean up term form UI
Assigned to:Anonymous» Xano
Status:needs work» needs review

This patch removes the Identification fieldset and all redundant element descriptions. It also renames "Related terms" to "Similar terms", to prevent collision with the "Relations" fieldset and because parent terms are also related. #504078: Synonym collapsing in core adds a description to the Synonym textare explaining what it's used for.

This is an alternative alternate text.

AttachmentSizeStatusTest resultOperations
cleanup_term_form_ui_00.patch3.35 KBIdleFailed: 11626 passes, 6 fails, 0 exceptionsView details | Re-test

#17

Issue tags:+Cleanup

#18

Status:needs review» needs work

The last submitted patch failed testing.

#19

Looks good, please fix the patch.

#20

Bump. Xano, still on this?

#21

Status:needs work» needs review

Before & after.

AttachmentSizeStatusTest resultOperations
cleanup_term_form_ui_02.patch2.63 KBIdleFailed: Failed to apply patch.View details | Re-test

#22

Nice. I like 'relation's here - since that's what the fieldset has, and if I don't care what relations is, I don't need to open it, whereas you can never tell with 'advanced'.

#23

Status:needs review» reviewed & tested by the community

Yup, agreed - good terminology.

#24

What catch said. RTBC

#25

Status:reviewed & tested by the community» needs work

The last submitted patch failed testing.

#26

Status:needs work» needs review

Rerolled patch from #21

AttachmentSizeStatusTest resultOperations
cleanup_term_form_ui_03.patch1.89 KBIdlePassed on all environments.View details | Re-test

#27

Status:needs review» reviewed & tested by the community

back to rtbc

#28

We might even want to get rid of the "Optionally (...)", because the asterisks indicate required and optional fields anyway. No commit blocker, just nitpicking.

#29

We'll leave it in here. Without it, the sentence reads very 'required'.

#30

Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks.

#31

Status:fixed» needs work

The term form is pretty broken still:

tag.jpg

AttachmentSizeStatusTest resultOperations
tag.jpg308.24 KBIgnored: Check issue status.NoneNone

#32

Priority:normal» critical

#33

Related issue for the position of URL alias textfield: #678590: Path alias link is below submit button on taxonomy term add

#34

Category:task» bug report

This is a bug.

#35

All we need to fix is the location of the url alias? The rest seems to been fixed, the breadcrumb is a larger issue - not meant to be fixed here.

termsaddbugged.png

AttachmentSizeStatusTest resultOperations
termsaddbugged.png36.36 KBIgnored: Check issue status.NoneNone

#36

Priority:critical» normal

Also, this bug isn't really critical anymore

#37

This form has Field API 'fields'. The order is defined by the admins on admin/structure/taxonomy/%tid/fields.

Reordering doesn't play nice when there are fixed-weight items in the middle of user-reorderable items.
So the consequence for other, non-Field elements in the form is that :

a) Either they declare themselves as 'reorderable' as well, using hook_field_extra_fields(). They just provide a default weight.
That's the case of 'Name' and 'Description' : see taxonomy_field_extra_fields(), and the result on this screenshot.

Should be done only if we see value in letting users tweak their position. Having too much stuff available for reordering only adds confusion for users.
I'm not sure this is the case for 'URL alias' and the 'Relations' fieldset.

b) Or they stick with a fixed position at the bottom of the form, with a reasonably high weight (100+) so that they stay out of the range of weights of the reorderable items
That's what the buttons container does.

Currently, 'url alias' has an implicit fixed weight of 0 (path_form_taxonomy_form_term_alter()), and 'Relations' has a fixed weight of 10. Both are wrong.

AttachmentSizeStatusTest resultOperations
taxo_term_form_reorder.png17.46 KBIgnored: Check issue status.NoneNone

#38

I think its perfectly reasonable to let users re-order fields whenever they want. For the URL alias field, see #687994: Add hook_field_extra_fields() implementation to path module

#39

Status:needs work» closed (won't fix)

Won't fix it is then?

#40

Won't fix it is then?

Not yet. See my reply in #687994-7: Add hook_field_extra_fields() implementation to path module.

At any rate, "something" needs to be done for 'URL alias' and 'Relations'. Either a) or b) from #37 above.