From 262002f52280997b68622d38a4578d7c0ca5af2d Mon Sep 17 00:00:00 2001 From: Matthias Schmidt Date: Mon, 12 Mar 2018 07:15:38 +0100 Subject: [PATCH] Ignore container header for form field dependencies See #2509 --- .../Form/Builder/Field/Dependency/Container/Default.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/wcfsetup/install/files/js/WoltLabSuite/Core/Form/Builder/Field/Dependency/Container/Default.js b/wcfsetup/install/files/js/WoltLabSuite/Core/Form/Builder/Field/Dependency/Container/Default.js index 67b4129db6..de04823196 100644 --- a/wcfsetup/install/files/js/WoltLabSuite/Core/Form/Builder/Field/Dependency/Container/Default.js +++ b/wcfsetup/install/files/js/WoltLabSuite/Core/Form/Builder/Field/Dependency/Container/Default.js @@ -32,7 +32,13 @@ define(['./Abstract', 'Core', '../Manager'], function(Abstract, Core, Dependency var containerShouldBeVisible = false; var children = this._container.children; - for (var i = 0, length = children.length; i < length; i++) { + var start = 0; + // ignore container header for visibility considerations + if (this._container.children.item(0).tagName === 'H2' || this._container.children.item(0).tagName === 'HEADER') { + var start = 1; + } + + for (var i = start, length = children.length; i < length; i++) { if (!elIsHidden(children.item(i))) { containerShouldBeVisible = true; } -- 2.20.1