Clean up term form UI

beeradb - March 7, 2009 - 20:24
Project:Drupal
Version:7.x-dev
Component:taxonomy.module
Category:task
Priority:normal
Assigned:Xano
Status:needs work
Issue tags:Cleanup, d7uxsprint, Needs text review, UBUserTesting2009, ui-text, Usability
Description

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.

#1

beeradb - March 7, 2009 - 20:25

updating tags.

#2

yoroy - March 7, 2009 - 21:38
Issue tags:+Needs text review

#3

chrisshattuck - March 16, 2009 - 03:47

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 KBIgnoredNoneNone
advanced-taxonomy-after.png21.33 KBIgnoredNoneNone
taxonomy.admin_.patch2.24 KBIdleFailed: Failed to apply patch.View details | Re-test

#4

karschsp - March 17, 2009 - 03:08
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

chrisshattuck - March 17, 2009 - 06:19

Thanks, karschsp!

#6

Bojhan - March 29, 2009 - 18:18

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

beeradb - April 6, 2009 - 18:41

@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

System Message - April 16, 2009 - 13:51
Status:needs review» needs work

The last submitted patch failed testing.

#9

emmajane - June 2, 2009 - 14:04

@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

berenddeboer - June 27, 2009 - 12:07

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

berenddeboer - June 27, 2009 - 12:08
Issue tags:+#d7uxsprint, +ui-text

Adding tags.

#12

berenddeboer - June 27, 2009 - 14:42

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

berenddeboer - June 27, 2009 - 14:54

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

seutje - June 27, 2009 - 15:45
Issue tags:-#d7uxsprint+d7uxsprint

changing d7uxsprint tag

#15

eigentor - June 27, 2009 - 16:17

Screenshots before and after the patch #13

before:

after:

AttachmentSizeStatusTest resultOperations
taxonomy-before.jpg45.67 KBIgnoredNoneNone
taxonomy-after.jpg51.17 KBIgnoredNoneNone

#16

Xano - June 28, 2009 - 10:02
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

Xano - June 28, 2009 - 09:48
Issue tags:+Cleanup

#18

System Message - June 28, 2009 - 10:49
Status:needs review» needs work

The last submitted patch failed testing.

#19

Bojhan - July 5, 2009 - 12:11

Looks good, please fix the patch.

#20

yoroy - November 4, 2009 - 23:29

Bump. Xano, still on this?

#21

Xano - November 5, 2009 - 15:54
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

catch - November 5, 2009 - 18:13

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

Bojhan - November 5, 2009 - 18:54
Status:needs review» reviewed & tested by the community

Yup, agreed - good terminology.

#24

yoroy - November 5, 2009 - 19:06

What catch said. RTBC

#25

System Message - November 7, 2009 - 22:20
Status:reviewed & tested by the community» needs work

The last submitted patch failed testing.

 
 

Drupal is a registered trademark of Dries Buytaert.