The vocabulary edit form suffers from over-fieldset-use.

  • The Information fieldset contains the essential Vocabulary name field. This patch removes the fieldset, the name should be upfront.
  • Help and Description are reordered. Description probably is not used anywhere in core and not important.
  • The Content types fieldset was redundant, it's gone.

And some code style is fixed. Tests updated.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

drumm’s picture

Status: Needs work » Needs review
FileSize
5.15 KB

Removed weight from forum test. Weight is removed because the main admin page has reordering and weight makes no sense out of context.

Dries’s picture

Neil, it would be great if you could do a before and after shot. :-)

drumm’s picture

Before shot attached. After is in the original issue here.

catch’s picture

This is a good clean up IMO, and turning the description into a text field instead of textarea makes it look a lot less foreboding. Updating tags.

dmitrig01’s picture

Status: Needs review » Reviewed & tested by the community

Looks great IMO. Tested, code is good.

yoroy’s picture

Obvious improvements, nice clean up. +1

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Great work! Committed to HEAD. :)

Status: Fixed » Closed (fixed)
Issue tags: -Screenshot, -Needs usability review, -Usability

Automatically closed -- issue fixed for 2 weeks with no activity.