Patches: Browse | Submit New | Admin

[#5187] make width/height accessible

Date:
2006-07-25 04:57
Priority:
3
Submitted By:
Adam Greene (greene)
Assigned To:
Nobody (None)
Category:
None
State:
Open
Summary:
make width/height accessible

Detailed description
I ran across a case where I was initalizing the graph in one method but needing size information (for layout) in a different
method.  Rather than passing extra info through the method, why not just add attr_read :width, :height?  But that raised
another point: Internally that information is stored in @rows and @columns, which is probably the wrong name as that
isn't how many rows and cols exist on the graph.  I didn't change those variable names (in case you don't like that
idea) but I did set some dup variables so the accessor methods make some sense.

hell...probably easier to see the code than try to describe this little change.

Let me know what you think!
thanks,
Adam

Add A Comment: Notepad

Please login


Followup

No Followups Have Been Posted

Attached Files:

Name Description Download
gruff_width_height_0.1.2.patch attr_read for the width and height of the graph Download

Changes:

Field Old Value Date By
File Added703: gruff_width_height_0.1.2.patch2006-07-25 04:57greene