Page 1 of 1

TRadMenu bug with pixel metrix

Posted: Sun Nov 13, 2005 4:36 pm
by claudio.driussi
I found a bug in TRadMenu, the line 112:

AAdd( ::aItems, TRadio():New( nRow + ( n - 1 ) , nCol, acItems[ n ],;

seems to be changed to:

AAdd( ::aItems, TRadio():New( nRow + ( n - 1 )*nStep , nCol, acItems[ n ],;

Best regards.

Claudio

Posted: Mon Nov 14, 2005 7:02 am
by Antonio Linares
Claudio,

Yes, you are right. nStep was not being used.

have you tried it ? Does it works ok ?

Posted: Mon Nov 14, 2005 7:13 am
by Antonio Linares
Claudio,

In our tests it seems that we should better not use it and leave the code as it is right now. It looks better.

Posted: Mon Nov 14, 2005 7:39 am
by claudio.driussi
Antonio Linares wrote:Claudio,

In our tests it seems that we should better not use it and leave the code as it is right now. It looks better.
I tried modified code with pixel metrix and it works fine for me, with the original code radio buttons disappare. I did'nt tried with pixel switch off.

Best regards.
Claudio

Posted: Mon Nov 14, 2005 8:41 am
by Antonio Linares
Claudio,

Yes, the problem is with lPixel as .f.