Re: add the "visible" radio_property to all objects.

[David Bateman]

Shai Ayal wrote:
> On Dec 17, 2007 9:09 PM, John W. Eaton <address@hidden> wrote:
>> On 15-Dec-2007, Shai Ayal wrote:
>>
>> | I attach a patch to add the visible property to all objects. The
>> | property is of type radio_property with the allowed values of
>> | "{on}|off"
>> |
>> | Shai
>> |
>> |
>> | 2007-12-15  Shai Ayal  <address@hidden>
>> |
>> |         * graphics.h.in (all object classes) Add the visible property.
>> |         * graphics.cc (all object classes) Ditto.
>>
>> If the visible property appears in all graphics objects, then I think
>> it should probably be added to the base_properties class and inherited
>> by all the others.
> 
> I thought about it, but since even if it is in base_properties there
> is still a lot of work getting it into each object::properties'
> get/set methods, I decided to do alittle more work and add it to each
> object.
> Maybe we should change this somehow --
> How about making each object::properties' get/set methods call
> base_property::get/set after taking care of all the object's
> properties, at the end, which should take care of all the properties
> in base_properties?
> 
> Shai
> 


Eventually, when we add hggroups and the addprop function, then
properties should be addable to the base class, and then the addprop
function will only be needed in the base class. Then hggroup will just
be an object with no properties till it is added. This being the case it
makes sense that "visible" is also part of the base class.

D.