Empty cck fields being rendered as a HTML comment. Affects fieldgroup tabs, panels etc.

OK, I'm looking into it, cheers.

OK, so this is really going to surprise you...

The issue caused by the comment in AT's content-field.tpl.php file, on the very last line of code you will see this:

<?php endif; ?> <!-- /content-field -->

Remove <!-- /content-field --> and the problem is fixed.

I'll remove the comment also and commit the change to the DEV version.

Committed.

This seems to have regressed.
It also causes panels panes to render when empty.

Patch against 6.x-2.x-dev

Committed to 2x dev, Thanks!

BTW - yes we had a regression, I made a silly mistake with CVS a while ago, my bad.

. ignore, something odd going on with comments here...

@Agileware - hey just thought I'd mention that if you really want to play with this theme I'd go for the 3x version that's in development, there's an RC1 but that's already out of date and needs a new release. In AT3 we ripped out about 1000 lines of code and massively simplified the advanced theme settings (the classes stuff) - after a year of usage we figured out what we used all the time and decided to just turn them on by default make the classes settings very simple.

Thanks I'll look into that.

Currently I have a site redesign about to go live on a sub-theme of adaptivetheme 2x so it's probably too late for that one.
Would it be a relatively smooth transition to upgrade from 2.x to 3.x or would it potentially cause a lot of headaches?

Lots of headaches... don't do it, live with AT2 for the life-cycle, I'm going to maintain it for all the D7 cycle because we have lots of clients sites that use it also.

I thought as much, thanks.