FiveTech Support Forums

FiveWin / Harbour / xBase community
Board index Bugs report & fixes / Informe de errores y arreglos TMenuItem height [Fixed]
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Sun May 07, 2017 10:22 AM

If we cannot have intermediate values, 4 is better.

EMG

Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Sun May 07, 2017 05:50 PM

Please, if you changed Int with round, leave INT, and try
Thanks for your test
I'm outside these days

Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Sun May 07, 2017 09:28 PM

With INT() the best value is 5.

EMG

Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Mon May 08, 2017 10:03 PM

Enrico
I can not find the difference of 5 between the versions
I use GetSysMetrics as a reference
I'm still looking for

Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Sat May 20, 2017 06:05 PM

The problem has not been fixed in FWH 17.05. :-(

EMG

Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Sat May 20, 2017 06:47 PM

Enrico
Attach or send me an image with the previous height and the current height. I do not see the difference well
Thank you

Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Sat May 20, 2017 07:47 PM
The fix is

Code (fw): Select all Collapse
         ChgMenuMea( pMitStruct, Round( nWText * nFw, 0 ), ;
               if( oItem:lSeparator, Int( nHItem * GetnHeightItem() ),;
                Max( Int( nHItem * GetnHeightItem() * if( !lHFont, 1.5, 1 ) ) + 5, GetSysMetrics( 15 ) ) ) )


Please note + 5.

EMG
Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Sat May 20, 2017 08:26 PM

Enrico, yes, I know it
But I have not found why it has changed by 5 pixels the height

Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Mon Jun 05, 2017 07:29 AM

Bug still pending. :-)

EMG

Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Mon Jun 05, 2017 11:01 AM
Try

MENU oMenu HEIGHT 2.075 and comment
Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Mon Jun 05, 2017 11:19 AM
Height 2 seems to be fine:

Code (fw): Select all Collapse
MENU oMenu HEIGHT 2


EMG
Posts: 6755
Joined: Wed Feb 15, 2012 08:25 PM
Re: TMenuItem height
Posted: Mon Jun 05, 2017 11:24 AM

Not with 2.075 ?

Cristobal Navarro

Hay dos tipos de personas: las que te hacen perder el tiempo y las que te hacen perder la noci贸n del tiempo

El secreto de la felicidad no est谩 en hacer lo que te gusta, sino en que te guste lo que haces
Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Mon Jun 05, 2017 11:31 AM

Yes, you are right, 2 or 2.075 it's the same, the height doesn't change.

EMG

Posts: 9020
Joined: Thu Oct 06, 2005 08:17 PM
Re: TMenuItem height
Posted: Tue Jun 27, 2017 09:59 AM

In the new FWH release this bug is still there.

EMG