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.

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

drumm’s picture

Status: Needs work » Needs review
StatusFileSize
new5.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

StatusFileSize
new103.23 KB

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.