This is unfortunately one of the shortcomings of CSS. Believe it or not, the most common solution is to give the items a fixed height and be mindful of the amount of content that goes into the box. You can use the LimitWordCount template function to help you with that. I happen to think it looks a lot nicer, too.

But.. If you want to do handle every third item, here's a good hack.

function PortfolioItems()
{
$items = $this->ChildrenOf('portfolio');
$i = 0;
foreach($items as $item)
{
$i++
$item->Third = ($i % 3 == 0);
}
return $items;
}

That should work.. haven't tested it or anything.

Yeah, I tried $this->Pos and it didn't work. That was what I was looking for. Pos() must be a function that returns the property iteratorPos.

There's really no way to do that without decorating the DataObjectSet. Try it my way. It's uglier, but it's easier than decorating. The bigfoot guy might have another idea, though.

edit: I just noticed Hamish's note about iteratorPos being a property of the object. Therefore, all you need to do is put that Mod3() function on your object.

Put the newShelf() method in the Page class rather than Page_Controller.

* Methods put on the Page class are available anywhere - ie, within <% control %> blocks and things.
* Methods on the Page_Controller class are available only at the top level of a template.