Hi,

I'm proposing the following patch which hides the vocabulary fieldset if all the vocabularies inside it are of type value and hidden. This avoids getting an empty vocabulary fieldset

CommentFileSizeAuthor
hide_fieldset_if_empty.patch1.22 KBguillaumev

Comments

David Lesieur’s picture

David Lesieur commented
Status: Needs review » Fixed

Committed a similar fix, but with some improvements:

  • Hide the fieldset server-side rather than client-side.
  • Avoid adding extra loops.
  • Mimic core behavior by removing the fieldset not only when no vocabularies are left, but also when there is just one vocabulary (in that case the vocabulary remains visible but does not appear within a fieldset).
  • Refactored logic a little bit to minimize code duplication.

Status: Fixed » Closed (fixed)

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