Roassal3 icon indicating copy to clipboard operation
Roassal3 copied to clipboard

Rename in RSTreeMapLayout >> prepareShapesForLayout: inRectangle:

Open NSUSpray opened this issue 3 years ago • 0 comments

I started gradually reviewing the TreeMap package. It seems to me that some points can be made clearer. Here is my first suggestion:

rename the method

RSTreeMapLayout >> prepareShapesForLayout: shapes inRectangle: rec

to

RSTreeMapLayout >> normalizeAreaOf: shapes byAreaOf: rec

Perhaps the time for refactoring hasn’t come yet and I’m running ahead.

NSUSpray avatar Sep 03 '22 23:09 NSUSpray