Re: Progress on loose columns. (issue4841052)

[mtsolo]

I think I've hit loose column nirvana with this most recent patch set.
The only thing that I know I need to work on is the magic left_padding
number of 0.15 in spacing-loose-columns.cc .  I know this should be made
a property but I don't know where (I can implement it like it was
implemented in previous incarnations of this patch and tack it onto the
SpacingSpanner if people think that's a good idea).  Also, the name is
crappy: it is not padding per se in that it is not always added.
Rather, it is a distance tacked onto the horizontal length between two
between-columns after which loose columns start relaxing from their
tight spacing to their ideal spacing.  Perhaps this is in fact the
definition of "padding," but I'm not sure.  Anyway, I'd appreciate
suggestions of (a) what to call this; and (b) what grob to stick it to
as a property.

Cheers,
MS

http://codereview.appspot.com/4841052/