Question

I am creating a system tray program with a shortcut/context menu, but I can’t seem to receive WM_COMMAND messages in the Windows Procedure. It simply doesn’t send when I click the menu item, and I’ve been checking for ages whether I have set up the menu correctly.

Here is my code:

#include <Windows.h>
#include <stdio.h>
#include "resource.h"
#define WM_TRAYICON (WM_USER + 0x0001) //a custom message for the notification icon

HWND hwnd; //window handle
HINSTANCE hinst; //module handle
WNDCLASSEX wnd; //window class
MSG msg; //event message or notification
NOTIFYICONDATA nid; //notification icon object
HMENU cmenu;
MENUITEMINFO menuitem1;
MENUITEMINFO menuitem2;
CURSORINFO cursor;

LRESULT CALLBACK MainWProc ( HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam )
{
    switch( uMsg )
    {
    case WM_COMMAND:
        printf("asfd\r\n");
        break;
    case WM_CREATE:
        printf("just created\r\n");
        break;
    case WM_TRAYICON:
        switch( LOWORD(lParam) )
        {
        case WM_CONTEXTMENU:
            GetCursorInfo( &cursor );
            //printf("xPos: %d\r\nyPos = %d\r\n\r\n", xPos, yPos );
            TrackPopupMenuEx( cmenu, TPM_RETURNCMD | TPM_LEFTBUTTON | TPM_NOANIMATION | TPM_HORIZONTAL | TPM_VERTICAL, cursor.ptScreenPos.x, cursor.ptScreenPos.y,  hwnd, NULL );
            //DestroyMenu(
            break;
        }
        break;
    case WM_INITMENU:
        printf("open menu\r\n");
        break;
    case WM_DESTROY:
        //clean things up
        Shell_NotifyIcon( NIM_DELETE, &nid );
        break;
    default:
        break;
    }

    return DefWindowProc( hwnd, uMsg, wParam, lParam );
}

void main()
{
    int result;

    hinst = GetModuleHandle( NULL );
    cursor.cbSize = sizeof( cursor );

    memset( &wnd, 0, sizeof( wnd ) );
    wnd.cbSize = sizeof( wnd );
    wnd.lpszClassName = "MainWClass";
    wnd.lpfnWndProc = MainWProc;
    wnd.hInstance = hinst;
    result = RegisterClassEx( &wnd );

    hwnd = CreateWindowEx
        (
        0, //extended styles
        wnd.lpszClassName, //class name
        "Main Window", //window name
        WS_OVERLAPPEDWINDOW | WS_HSCROLL | WS_VSCROLL | WS_MINIMIZEBOX, //style tags
        CW_USEDEFAULT, //horizontal position
        CW_USEDEFAULT, //vertical position
        CW_USEDEFAULT, //width
        CW_USEDEFAULT, //height
        (HWND) NULL, //parent window
        (HMENU) NULL, //class menu
        (HINSTANCE) wnd.hInstance, //some HINSTANCE pointer
        NULL //Create Window Data?
        );
    if( !hwnd )
    {
        printf("CreateWindowEx failed: %d\n", GetLastError() );
        Sleep( INFINITE );
    }

    nid.cbSize = sizeof( nid );
    nid.hWnd = hwnd;
    nid.uID = 1;
    nid.uVersion = NOTIFYICON_VERSION_4;
    nid.uCallbackMessage = WM_TRAYICON;
    nid.hIcon = LoadIcon( hinst, MAKEINTRESOURCE( IDI_ICON1 ) );
    strcpy( nid.szTip, "My Tooltip!" );
    nid.uFlags = NIF_MESSAGE | NIF_ICON | NIF_TIP | NIF_SHOWTIP;

    cmenu = CreatePopupMenu();
    menuitem1.cbSize = sizeof( menuitem1 );
    menuitem1.fMask = MIIM_TYPE;
    menuitem1.fType = MFT_STRING;
    menuitem1.hSubMenu = NULL;
    //menuitem1.cch = ;
    menuitem1.dwTypeData = "Open a world of wonder!";
    InsertMenuItem( cmenu, 0, true, &menuitem1 );

    if( ! Shell_NotifyIcon( NIM_ADD, &nid ) )
    {
        printf("Shell_NotifyIcon( NIM_ADD, &nid ) failed.\r\n");
        Sleep( INFINITE );
    }
    if( ! Shell_NotifyIcon( NIM_SETVERSION, &nid ) )
    {
        printf("Shell_NotifyIcon( NIM_SETVERSION, &nid ) failed.\r\n");
        Sleep( INFINITE );
    }

    UpdateWindow( hwnd );
    for( ; ; )
    {
        if( GetMessage(&msg, hwnd, 0, 0) )
        {
            TranslateMessage(&msg);
            DispatchMessage(&msg);
        }
    }
}

Answer 1

Um, you passed the TPM_RETURNCMD flag which means “don’t post a WM_COMMAND message. Just return the value you would have posted.”