Closed pnutus closed 10 years ago
Looks very nice, thanks! I just have a couple minor comments, interspersed above.
Is it ready to be merged?
There's a few things to improve:
sameBoundingRect
is weird if the sizes and origins of the objects differ too much. I should be looking at their actual bounding rects and not just use size and position.gridCat'
) do not use the diagrams
convention (gridCat' (with & …)
), which they probably should.I don't have time to work on it right know since I'm teaching a university intro course for two weeks, but I'll tidy it up when that's over.
Yes, I think it's ready to be merged --- we can always improve those things later. In particular if you want to rename things or change their types (e.g. gridCat'
) we can be sure to do that before making a release.
This module contains functions for putting a list of diagrams in a grid layout with minimal effort from the user. I find it very useful for comparing diagrams in a more space-efficient way than using
hcat
orvcat
.