Codejock Developer Community : CXTPCommandBars on glitch on NT

CXTPCommandBars on glitch on NT : NT4 compares the size you specify...

Wed, 18 Aug 2004 03:27:33 +0000

Author: Sven
Subject: 1045
Posted: 18 August 2004 at 3:27am

NT4 compares the size you specify in MENUITEMINFO.cbSize with the size of the old MENUITEMINFO size. If the size is not equal the function returns with an error code and the values of MENUITEMINFO remain uninitialized. You can test this if you set a breakpoint in CXTPCommandBar::LoadMenu.

CXTPCommandBars on glitch on NT : Sven, Thanks for your reply. What...

Mon, 16 Aug 2004 10:58:22 +0000

Author: plamen
Subject: 1045
Posted: 16 August 2004 at 10:58am

Sven,
Thanks for your reply.
What I don't understand is how the new MENUITEMINFO size could affect the state of the menu under NT - fState field is part of the NT's MENUITEMINFO as well.
I can imagine that some of the new added flags/fields may not work on NT but, hey, that's why there weren't part of the original NT structure.

Btw does this mean that you've managed to reproduce it and fixed it that way?

Cheers
Plamen

CXTPCommandBars on glitch on NT : Since NT4/W95 the size of MENUITEMINFO...

Sat, 14 Aug 2004 13:42:31 +0000

Author: Sven
Subject: 1045
Posted: 14 August 2004 at 1:42pm

Since NT4/W95 the size of MENUITEMINFO has been changed. If you compile the toolkit with the latest Platform SDK and WINVER >= 0x0500 you have to adjust the MENUITEMINFO size in CXTPCommandBar::LoadMenu.


#if (WINVER >= 0x0500)
#define MENUITEMINFO_SIZE_VERSION_400A  CDSIZEOF_STRUCT(MENUITEMINFOA, cch)
#define MENUITEMINFO_SIZE_VERSION_400W  CDSIZEOF_STRUCT(MENUITEMINFOW, cch)
#define MENUITEMINFO_SIZE_VERSION_500A  sizeof(MENUITEMINFOA)
#define MENUITEMINFO_SIZE_VERSION_500W  sizeof(MENUITEMINFOW)
#ifdef UNICODE
#define MENUITEMINFO_SIZE_VERSION_400  MENUITEMINFO_SIZE_VERSION_400W
#define MENUITEMINFO_SIZE_VERSION_500  MENUITEMINFO_SIZE_VERSION_500W
#else
#define MENUITEMINFO_SIZE_VERSION_400  MENUITEMINFO_SIZE_VERSION_400A
#define MENUITEMINFO_SIZE_VERSION_500  MENUITEMINFO_SIZE_VERSION_500A
#endif
#endif

#define MENUITEMINFO_SIZE  MENUITEMINFO_SIZE_VERSION_400

BOOL CXTPCommandBar::LoadMenu(CMenu* pMenu, BOOL bRemoveControls)
{
        ...
//      MENUITEMINFO info = { sizeof(MENUITEMINFO), MIIM_TYPE|MIIM_STATE};
        MENUITEMINFO info = { MENUITEMINFO_SIZE, MIIM_TYPE|MIIM_STATE};
        ...
}

CXTPCommandBars on glitch on NT : I use the latest - v9.10 Anyway,...

Fri, 13 Aug 2004 06:53:05 +0000

Author: plamen
Subject: 1045
Posted: 13 August 2004 at 6:53am

I use the latest - v9.10
Anyway, if you still can't reproduce it I'll dive into your code to find some more info about it.

CXTPCommandBars on glitch on NT : Can't reproduce with nt4...

Wed, 11 Aug 2004 12:24:03 +0000

Author: Oleg
Subject: 1045
Posted: 11 August 2004 at 12:24pm

Can't reproduce with nt4 sp6.

May be in nt4 you have version earlier than 9.00?

TPM_NONOTIFY appeared in 9.00...

CXTPCommandBars on glitch on NT : Oleg, I use a clean NT4 Service...

Wed, 11 Aug 2004 02:26:17 +0000

Author: plamen
Subject: 1045
Posted: 11 August 2004 at 2:26am

Oleg,
I use a clean NT4 Service Pack 6 / IE 5.5.

Find the sample attached.

Cheers,
Plamen

2004-08-11_022548_testpopup.zip

CXTPCommandBars on glitch on NT : Strange, I can't reproduce...

Wed, 11 Aug 2004 02:17:32 +0000

Author: Oleg
Subject: 1045
Posted: 11 August 2004 at 2:17am

Strange, I can't reproduce it in my NT4 machine :(

Can you upload your example?

Can you manually debug CXTPCommandBar::LoadMenu and check if info.fState == MFS_DISABLED for ID_POPUP_DISABLED?

CXTPCommandBars on glitch on NT : Guys, I noticed a small problem...

Tue, 10 Aug 2004 09:05:01 +0000

Author: plamen
Subject: 1045
Posted: 10 August 2004 at 9:05am

Guys,
I noticed a small problem with CXTPCommandBars on Windows NT.
It looks like if you create a pop-up a menu with some items disabled, on NT displays them enabled instead. However, it looks like it works fine on XP.

To reproduce it build a standard MDI application and add the code below into the view.

Cheers,
Plamen

void CTest2View::OnRButtonDown(UINT nFlags, CPoint point)
{
     ClientToScreen( &point );
     CMenu menu;
     menu.LoadMenu(IDR_POPUPMENU);
     menu.GetSubMenu(0)->EnableM enuItem(ID_POPUP_DISABLED, MF_BYCOMMAND | MF_DISABLED);
     UINT iCmdID = CXTPCommandBars::TrackPopupMenu(menu.GetSubMenu(0), TPM_LEFTALIGN | TPM_RIGHTBUTTON | TPM_NONOTIFY | TPM_RETURNCMD, point.x, point.y, AfxGetMainWnd(), NULL);

     if (iCmdID == ID_POPUP_DISABLED)
           AfxMessageBox(L"This menu should be disabled...");

     CView::OnRButtonDown(nFlags, point);
}