Description: ------------ I was trying to create groups of form elements with the ITDynamic renderer, using very exact layout of the form elements. Although I set the group seperator to null or '', there was always a   between elements, resulting in empty lines in my layout. I consider this a bug. A fix is attached below. Reproduce code: --------------- --- ITDynamic.php.orig 2004-10-15 14:10:35.000000000 +0200 +++ ITDynamic.php 2004-10-15 14:10:40.000000000 +0200 @@ -191,7 +191,7 @@ $blockName = $this->_matchBlock($group); $this->_tpl->setCurrentBlock($blockName . '_loop'); $this->_groupElementIdx = 0; - $this->_groupSeparator = empty($group->_separator)? ' ': $group->_separator; + $this->_groupSeparator = isset($group->_separator) ? '' : $group->_separator; // show an '*' near the required element if ($required) { $this->_showRequired = true;

Fixed in CVS. This was a bug indeed since it behaved differently from Default renderer. Now if you set separator to '' (but not null), it will behave as expected. The pathc proposed is obviously broken, BTW. And the summary is wrong...