For the grid format "Fill up single line" does not work when using vertical alignment for a single row

I had the same issue and followed the same basic premise. To only do this when 'fill single line' is checked, I instead added a new loop (no need for handling $rows[1+] since it only occurs when there is only one row). this is at approximately row 737 of theme.inc for version 7.x-3.6. I too have yet to figure out patches:

if (!empty($handler->options['fill_single_line'])) {
	for ($i = 0; $i < $columns; $i++) {
		if (!isset($rows[0][$i])) {
			$rows[0][$i] = '';
		}
	}
}

Attempt at a patch...

Thanks danwonac. This replaces tabs with 2 spaces.

The above patch did not work for me.

The problem appears to be this line:
if (!empty($handler->options['fill_single_line']) && count($rows)) {

It should read.
if (!empty($handler->options['fill_single_line']) && $row_count==count($rows))

I've attached a new patch.

The last patch worked for me. Heres a re-rolled patch.

#7 seems to be perfect.

Why don't you put in the next views version?

#7 almost got me where I need to be but it doesn't appear to apply the row class to the rows it adds which I need for styling purposes. Any ideas? In looking through the code it appears that the row/column classes don't get applied till further down in the function and that they require a row_index to be set, just need to figure out if its possible to add these indexes to the filler rows that the patch in #7 generates.

Got it. Slight mod to #7 to allow column classes to be applied to the fill columns as well:

    if ($row) {
      // Fill up the last line only if it's configured, but this is default.
      if (!empty($handler->options['fill_single_line'])) {
        for ($i = 0; $i < ($columns - $col_count); $i++) {
          $row[] = '';
          $row_indexes[$row_count][$col_count+$i] = 'fill_'.$row_count.'_'.($col_count+$i);
        }
      }
      $rows[] = $row;
    }

Sadly I'm not sure how to create a patch so if anyone wants to review and roll a patch I'm sure it would be appreciated.

Hi
I needed this patch so here is my attempt taking into account last comment about column classes.
It add the same class as it normally would if there was content plus a new one ".col-fill"

New patch attached

Bumping to the next release.

Taking this off the plan for the next release, will consider it later.

The 2 year old patch in #14 to theme.inc applied cleanly to the latest views 7.x-3.x-dev

I think it was because of the lack of reviews.

Could someone please add a comment to the for() loop explaining the logic? Thanks.

Simplified the code a little.

Committed. Thanks all.