Bug in TMenu

[Enrico Maria Giordano]

In the following sample, the automatic generated menuitem has not the right color:

Code: Select all  Expand view

#include "Fivewin.ch"


FUNCTION MAIN()

    LOCAL oWnd, oMenu

    MENU oMenu 2007
        MENUITEM "Window"

        MENU
            MENUITEM "Test"
        ENDMENU
    ENDMENU

    DEFINE WINDOW oWnd MDI;
           TITLE "MDI Test";
           MENU oMenu;
           MENUINFO 1

    ACTIVATE WINDOW oWnd;
             ON INIT CREATECHILD( oWnd )

    RETURN NIL


STATIC FUNCTION CREATECHILD( oMdi )

    LOCAL oWnd, oMenu

    MENU oMenu 2007
        MENUITEM "Window"

        MENU
            MENUITEM "Test"
        ENDMENU
    ENDMENU

    DEFINE WINDOW oWnd MDICHILD OF oMdi;
           TITLE "MDI Child Test";
           MENU oMenu

    ACTIVATE WINDOW oWnd;
             VALID !GETKEYSTATE( VK_ESCAPE )

    RETURN NIL

EMG

[cnavarro]

Enrico
I will check and comment

Thanks

[Enrico Maria Giordano]

Thank you Cristobal!

EMG

[cnavarro]

Enrico, problem is in module Menudraw.c

Change this function and test

Code: Select all  Expand view

HB_FUNC( MITEMSFW )
 // hMenu, nIni, aHBmps, lSys
{
   .../...

   MENUITEMINFO mi;

   for( w = nIni; w < GetMenuItemCount( hMenu ); w++ )
   {
      memset( ( char * ) &mi, 0, sizeof( MENUITEMINFO ) );
      mi.cbSize = sizeof( MENUITEMINFO );
      if ( bSys )
         mi.fMask  = MIIM_TYPE;
      else
         mi.fMask  = MIIM_FTYPE;
      GetMenuItemInfo( hMenu, w, TRUE, &mi );

      if( ( mi.fType & MFT_OWNERDRAW ) != MFT_OWNERDRAW )
      {
         mi.cbSize = sizeof( MENUITEMINFO );
         mi.fMask  |= MIIM_DATA | MIIM_BITMAP ;
         mi.fType  |= MFT_OWNERDRAW ;
         mi.dwItemData = ( ULONG_PTR ) hMenu;
         SetMenuItemInfo( hMenu, w, TRUE, &mi );
      }
   }
}

 

Please comment result

[Enrico Maria Giordano]

Already tried your update lib. Still need to try that C function?

EMG

[cnavarro]

Enrico
I published the problem
This function is included in the library that I sent you

[Enrico Maria Giordano]

Ok. Thank you!

EMG

[Enrico Maria Giordano]

I also tried the above changed C function and it worked fine!

EMG

[cnavarro]

I also tried the above changed C function and it worked fine!

EMG

Enrico, thanks for your test

[cnavarro]

News features in next versions: LOGOMENU, IMGMENU, etc.

[Antonio Linares]

very nice!