about function "WADlg_init(hwndWinamp)" in "wa_dlg.h"

[Lemonsoft]

i want to create a windows in General Purpose Plugin,
but i can't use winamp skin at my windows through function "WADlg_init(hwndWinamp)"
who can help me?
thanks!!!!!!!!

LRESULT CALLBACK WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
WADlg_handleDialogMsgs( hwnd, message, wParam, lParam);
switch (message)
{
case WM_DISPLAYCHANGE:
WADlg_init(plugin.hwndParent);
return 0;
case WM_CLOSE:
ShowWindow(hWnd,SW_HIDE);
return 0;
return 0;
}
return DefWindowProc(hwnd,message,wParam,lParam);
}
int init() {
WNDCLASS wc;
memset(&wc,0,sizeof(wc));
wc.lpfnWndProc = WndProc;
wc.hInstance = plugin.hDllInstance;
wc.lpszClassName= PlugInName;

RegisterClass(&wc);
hWnd = CreateWindowEx(
0,
PlugInName,PlugInName,
WS_SYSMENU | WS_CAPTION | WS_MINIMIZEBOX,
0,0,
320,240,
plugin.hwndParent,
NULL,
plugin.hDllInstance,
NULL);
ShowWindow(hWnd,SW_SHOWNORMAL);
return 0;
}

[DrO]

you seem to be having problems with this window thing - i'll try and get an example up for you for tomorrow (hopefully you can wait till then )

-daz

[DrO]

just a quick reply on this, for the code to compile correctly the source file you are using needs to end in *.cpp (is due to the way the header has been written)

as for an example i've got something that will have a winamp skin style in a window but it's not using a full winamp window frame at the moment will post once i've done it all (along with all my other bits to do )

-daz

[gb115]

Daz

did you ever get the winamp window frame working.. I think its been mentioned elsewhere, but I would really love a General plugin using the new api, with a simple "hello world" in a skinned window.

kind regards

G

[DrO]

not forgotten i just haven't had as much time as i wanted.
i got the internal parts of a winamp style window working, it's just the outer frame to do (and i found the correct part in the headers last night).

i'll try to get it done for tomorrow (may use it myself for some bit's i'm doing...)

-daz

[gb115]

any chance of it being MFC?

[DrO]

it's all pure 'c' at the moment. it's basically a few sendmessage calls at the end of the day. the only mfc bits would be the on_message handlers and things like that for the dialog proceedure which is fairly easy for someone else to convert over to

-daz

[Bizzeh]

never ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever ever use MFC, always use win32 api... mfc = slow and pooeh compaired to directly using the api

[Lemonsoft]

Originally posted by DrO
you seem to be having problems with this window thing - i'll try and get an example up for you for tomorrow (hopefully you can wait till then )

-daz

i am back, and very grateful after i see you article.
thanks!

[DrO]

here it is.

it's a 'c' example but read the notes in the source (should be clear) to get an idea of a few things that are needed to be done.

any problems then don't hesitate to ask, enjoy

-daz

[Lemonsoft]

thanks! it run best on winamp 5
but i have another question,

My Code:

if(wa2){
// here we attach a dialog into the window
// if need by this could been done with CreateWindow(), etc depending
// on what you want to do
RegisterClass(&wc);
HWND hWnd = CreateWindow(
PlugInName,
PlugInName,
WS_POPUP,
0, 0, 100, 100,
(HWND) NULL,
(HMENU) NULL,
plugin.hDllInstance,
(LPVOID) NULL);

ShowWindow(hWnd,SW_SHOWNORMAL);
proc = (PROC)SetWindowLong(wa2,GWL_WNDPROC,(LONG)WinampProc);
}

I create a window , and at Wndproc, i deal with WM_CREATE message

switch(msg){
case WM_CREATE:
WADlg_init(plugin.hwndParent);
SetWindowText(GetParent(hwnd),"Project");
SetDlgItemText(hwnd,IDC_EDIT, "Lemonsoft");
// shows the owner window and this dialog
//ShowWindow(hwnd,SW_SHOWNORMAL);
ShowWindow(GetParent(hwnd),SW_SHOWNORMAL);
.......

Background Color changed,
but winamp base skin don't display, why?

[DrO]

it run best on winamp 5

i agree on that, using the modern skins does look better for a winamp styled window. glad it's what you were looking for.

with your problem, you need to alter the window styles used for the window you are creating. i think you need WS_CHILD and WS_VISIBLE and to change to CreateWindowEx using WS_EX_CONTROLPARENT as the extended window style.

i'll check it tonight and post you a new version unless you beat me to it (the part above should be part on the way to things).

also as a point of interest for you, it may be an idea to allow the ability to create a normal owner window as i think you had incase winamp is not able to return a valid window to use (such as on versions below 2.9x).

-daz

[gb115]

Bizzy D.

I'm not just usign MFC for the sheer fun of it, (in fact I'm preety sure no fun is to be gained at all) but because I need the message pump for my plugin which involves lots of two way serial communication for a remote control.

Is there an easier way of doing this that I skipped over? I'm using the CSerial on codeporject at the mo.

Dr. O cheers for that, I look forward to butchering your code beyong recognition.

[DrO]

Dr. O cheers for that, I look forward to butchering your code beyong recognition.

that's what it's there for, if it helps at least one person then that's the main thing i think at the end of the day. also i needed the code myself

as for the MFC thing i agree with Bizzy D. about it but i'm sure you can do waht you need without using MFC since at the end of the day it is just a big wrapper for the basic win32 functions (codeproject and codeguru should be able to help)

-daz

[gb115]

Dr.O sorry this is goign to be a total n00b question.

when I add another dialog box (for say a configuration screen) when I actually creat the class and .cpp file in MS visual studio. it will no longer link.

I get lots of errors like
"gen_test4 error LNK2005: "int __cdecl WADlg_getColor(int)" (?WADlg_getColor@@YAHH@Z) already defined in gen_test4.obj"

can you help me?

[saivert]

The shit you are talking about is just some custom code
written by Nullsoft inside wa_dlg.h. It's purpose is to owner-draw
(paint) various buttons and edit boxes in your dialog so
they look like Winamp. When designing the dialog template
remember to set the Owner-draw style for the control to on (checked).
Then call WADlg_handleDialogMsgs in your WndProc like this:
{
int a=WADlg_handleDialogMsgs( hwnd, message, wParam, lParam);
if (a) return a;
}
You must return from the Window Procedure if WADlg_handleDialogMsgs
returns something else than NULL.

I sorted this out after looking at the Media Library example (ml_ex).
Have closer look at this you too (plural).

Then you have the option of embedding your dialog (which must have
the WS_CHILD style as its only style set) in a "General Wnd" window.
To do this set up an embedWindowStruct:

HWND (*embedWindow)=0;

embedWindow = SendMessage(hwndWinamp, WM_WA_IPC, 0, IPC_GETEMBEDIF);

embedWindowStruct ews;
ews.flags = 0;
SetRect(&ews.r, 0, 0, 320, 240);

embedWindow(&ews);
SetWindowText(ews.me, "My filthy window!");
ShowWindow(ews.me, SW_SHOW);
SetParent(yourdlghandle, ews.me);

Question:
What I need to know is: How do Winamp paint the Scroll-bars in the
Media Library. How do you owner-draw scroll-bars. Is Winamp using
some custom control scroll-bar thingy??

[Lemonsoft]

Originally posted by DrO

with your problem, you need to alter the window styles used for the window you are creating. i think you need WS_CHILD and WS_VISIBLE and to change to CreateWindowEx using WS_EX_CONTROLPARENT as the extended window style.versions below 2.9x).

-daz [/B]

Dr.O thank you
I try your way,but CreateWindowEx Error and GetLasterror returned 1406.

1406: Cannot create a top-level child window. ERROR_TLW_WITH_WSCHILD

My Code
wnd=CreateWindowEx(
WS_EX_CONTROLPARENT,app_name,"Test",
WS_CHILD | WS_VISIBLE ,
10,10,300,300,
NULL,NULL,plugin.hDllInstance,NULL);

[DrO]

@gb115: the problem is that the header can only be included once - forgot to mention that, oops (is because the parts in the header are statically defined and so you get the link error).

@saivert: i'm looking into that one already at the moment for my forth coming plugin - will post what i find out

-daz

[Joonas]

Lemonsoft: You ever solved that? I have the same problem...

[DrO]

Joonas: there is no parent assigned to the window in Lemonsoft's CreateWindowEx(..) and that's why it fails.

-daz

[Safai]

oh... and there is a restriction (bug?) that you cannot have more than one wadlg with the same window name... otherwise, the second one will not have it's frame drawn properly... learnt the hard way... debugged for days... when i'm trying to get multiple Album List windows up...

i have some code around too... if you guys have questions still...

[Joonas]

Yeah, I would love any code on this subject. The gen_wa2dlg project is great but somewhat limited.

I also wonder if someone knows how to skin a listview control?

/Jonas

[DrO]

ListView_SetTextColor(enq_list,WADlg_getColor(WADLG_ITEMFG));
ListView_SetBkColor(enq_list,WADlg_getColor(WADLG_ITEMBG));
ListView_SetTextBkColor(enq_list,WADlg_getColor(WADLG_ITEMBG));

that will set the colours for the listview control though it won't do the scrollbars and header control - i'm trying to update wa_dlg.h to provide support for most controls (like the tab control, etc - as the skinned prefs plugin in my signature does)

as for the example, yes it is limited but that was the best i knew at the time if i can get the time i will try to update the example with the extra fixes that are incorporated into jtfe (since the example was using the original window skinning code i was using for jtfe at the time)

-daz

[Safai]

custom draw code for the header control...

PHP Code:

NMCUSTOMDRAW* pCD = (NMCUSTOMDRAW*)pnmh;

if (CDDS_PREPAINT == pCD->dwDrawStage)
{
    return CDRF_NOTIFYITEMDRAW|CDRF_NOTIFYPOSTPAINT;
}
else if (CDDS_ITEMPREPAINT == pCD->dwDrawStage)
{
    HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_LISTHEADER_BGCOLOR));
    FillRect(pCD->hdc, &pCD->rc, hbr);
    DeleteObject(hbr);

    HPEN pen1=CreatePen(PS_SOLID,0,WADlg_getColor(WADLG_LISTHEADER_FRAME_TOPCOLOR));
    HPEN pen2=CreatePen(PS_SOLID,0,WADlg_getColor(WADLG_LISTHEADER_FRAME_MIDDLECOLOR));
    HPEN pen3=CreatePen(PS_SOLID,0,0);
    HGDIOBJ o=SelectObject(pCD->hdc, pen1);

    if ((pCD->uItemState & CDIS_SELECTED) == CDIS_SELECTED)
    {
        SelectObject(pCD->hdc, pen3);
        MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom,NULL);
        LineTo(pCD->hdc,pCD->rc.left,pCD->rc.top);
        LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.top);
        LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.bottom-1);
        LineTo(pCD->hdc,pCD->rc.left,pCD->rc.bottom-1);
    }
    else
    {
        // left, top
        MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom,NULL);
        LineTo(pCD->hdc,pCD->rc.left,pCD->rc.top);
        LineTo(pCD->hdc,pCD->rc.right,pCD->rc.top);

        // right, bottom
        SelectObject(pCD->hdc, pen2);
        MoveToEx(pCD->hdc,pCD->rc.left+1,pCD->rc.bottom-2,NULL);
        LineTo(pCD->hdc,pCD->rc.right-2,pCD->rc.bottom-2);
        LineTo(pCD->hdc,pCD->rc.right-2,pCD->rc.top);

        // right, bottom (black line)
        SelectObject(pCD->hdc, pen3);
        MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom-1,NULL);
        LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.bottom-1);
        LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.top-1);
    }

    SetTextColor(pCD->hdc, WADlg_getColor(WADLG_LISTHEADER_FONTCOLOR));
    SetBkMode(pCD->hdc, TRANSPARENT);

    char name***91;MAX_PATH***93; = "";
    HDITEM hdi;
    hdi.mask = HDI_TEXT|HDI_FORMAT;
    hdi.pszText = name;
    hdi.cchTextMax = MAX_PATH;
    Header_GetItem(pCD->hdr.hwndFrom, pCD->dwItemSpec, &hdi);

    RECT rcItem = pCD->rc;
    rcItem.left += 5;
    rcItem.right -= 5;
    rcItem.top ++;

    if ((pCD->uItemState & CDIS_SELECTED) == CDIS_SELECTED)
    {
        rcItem.left ++;
        rcItem.right ++;
    }

    int align = (hdi.fmt & HDF_JUSTIFYMASK) == LVCFMT_RIGHT ? DT_RIGHT : DT_LEFT;

    DrawText(pCD->hdc, hdi.pszText, -1, &rcItem, align | DT_VCENTER | DT_SINGLELINE);

    // clean up
    SelectObject(pCD->hdc, o);
    DeleteObject(pen1);
    DeleteObject(pen2);
    DeleteObject(pen3);

    return CDRF_SKIPDEFAULT;
}
else if (CDDS_POSTPAINT == pCD->dwDrawStage)
{
    RECT rc1;
    int nCount = Header_GetItemCount(pCD->hdr.hwndFrom);
    Header_GetItemRect(pCD->hdr.hwndFrom, nCount - 1, &rc1);
    RECT rc;
    GetClientRect(&rc);

    if (rc1.right < rc.right)
    {
        rc.left = rc1.right;
        rc.right += 10;
        HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_LISTHEADER_EMPTY_BGCOLOR));
        FillRect(pCD->hdc, &rc, hbr);
        DeleteObject(hbr);
    }

    return CDRF_DODEFAULT;
}
return FALSE; 


[Safai]

for custom scrollbar... check this article... this is what media library is using...

http://www.codeproject.com/dialog/coolscroll.asp

here is the custom draw for the "coolsb"...

PHP Code:

typedef struct 
{
    int x, y;
    int width, height;
} CustomDrawTable;

CustomDrawTable cdt_horz_normal***91;***93; = 
{
    { 0,  45, 14, 14 },    //left arrow  NORMAL
    { 14, 45, 14, 14 }, //right arrow NORMAL
    { WADLG_SCROLLBAR_FGCOLOR, WADLG_SCROLLBAR_BGCOLOR }, //page left   NORMAL
    { WADLG_SCROLLBAR_FGCOLOR, WADLG_SCROLLBAR_BGCOLOR }, //page right  NORMAL
    
    { -1, -1, -1, -1 },    //padding

    { 84,  31, 4,  4  }, //horz thumb (top,left)
    { 84,  35, 4,  6  }, //horz thumb (middle,left)
    { 84,  41, 4,  4  }, //horz thumb (bottom,left)

    { 88,  31, 20, 4  }, //horz thumb (top,middle)
    { 88,  35, 6,  6  }, //horz thumb (middle,middle,left)
    { 102, 35, 6,  6  }, //horz thumb (middle,middle,right)
    { 88,  41, 20, 4  }, //horz thumb (bottom,middle)

    { 108, 31, 4,  4  }, //horz thumb (top,right)
    { 108, 35, 4,  6  }, //horz thumb (middle,right)
    { 108, 41, 4,  4  }, //horz thumb (bottom,right)

    { 94, 35, 8,  6  }, //horz thumb (middle,middle,middle)
};

CustomDrawTable cdt_horz_active***91;***93; = 
{
    { 28, 45, 14, 14 }, //left arrow  ACTIVE
    { 42, 45, 14, 14 }, //right arrow ACTIVE
    { WADLG_SCROLLBAR_INV_FGCOLOR, WADLG_SCROLLBAR_INV_BGCOLOR }, //page left   ACTIVE
    { WADLG_SCROLLBAR_INV_FGCOLOR, WADLG_SCROLLBAR_INV_BGCOLOR }, //page right  ACTIVE

    { -1, -1, -1, -1 },    //padding

    { 84,  45, 4,  4  }, //horz thumb (top,left)
    { 84,  49, 4,  6  }, //horz thumb (middle,left)
    { 84,  55, 4,  4  }, //horz thumb (bottom,left)

    { 88,  45, 20, 4  }, //horz thumb (top,middle)
    { 88,  49, 6,  6  }, //horz thumb (middle,middle,left)
    { 103, 49, 6,  6  }, //horz thumb (middle,middle,right)
    { 88,  56, 20, 3  }, //horz thumb (bottom,middle)

    { 108, 45, 4,  4  }, //horz thumb (top,right)
    { 108, 49, 4,  6  }, //horz thumb (middle,right)
    { 108, 55, 4,  4  }, //horz thumb (bottom,right)

    { 94, 49, 9,  7  }, //horz thumb (middle,middle,middle)
};

CustomDrawTable cdt_vert_normal***91;***93; = 
{
    { 0,  31, 14, 14 }, //up arrow   NORMAL
    { 14, 31, 14, 14 }, //down arrow NORMAL
    { WADLG_SCROLLBAR_FGCOLOR, WADLG_SCROLLBAR_BGCOLOR }, //page up     NORMAL
    { WADLG_SCROLLBAR_FGCOLOR, WADLG_SCROLLBAR_BGCOLOR }, //page down  NORMAL

    { -1, -1, -1, -1 },    //padding

    { 56, 31, 4,  4  }, //vert thumb (top,left)
    { 60, 31, 6,  4  }, //vert thumb (top,middle)
    { 66, 31, 4,  4  }, //vert thumb (top,right)
    { 56, 35, 4, 20  }, //vert thumb (middle,left)
    { 60, 35, 6,  6  }, //vert thumb (middle,middle,top)
    { 60, 49, 6,  6  }, //vert thumb (middle,middle,bottom)
    { 66, 35, 4, 20  }, //vert thumb (middle,right)
    { 56, 55, 4,  4  }, //vert thumb (bottom,left)
    { 60, 55, 6,  4  }, //vert thumb (bottom,middle)
    { 66, 55, 4,  4  }, //vert thumb (bottom,right)
    { 60, 41, 6,  8  }, //vert thumb (middle,middle,middle)
};

CustomDrawTable cdt_vert_active***91;***93; = 
{
    { 28, 31,  14, 14 }, //up arrow   ACTIVE
    { 42, 31, 14, 14 }, //down arrow ACTIVE
    { WADLG_SCROLLBAR_INV_FGCOLOR, WADLG_SCROLLBAR_INV_BGCOLOR }, //page up     ACTIVE
    { WADLG_SCROLLBAR_INV_FGCOLOR, WADLG_SCROLLBAR_INV_BGCOLOR }, //page down  ACTIVE

    { -1, -1, -1, -1 },    //padding

    { 70, 31, 4,  4  }, //vert thumb (left)
    { 74, 31, 6,  4  }, //vert thumb (middle)
    { 80, 31, 4,  4  }, //vert thumb (right)
    { 70, 35, 4, 20  }, //vert thumb (middle,left)
    { 74, 35, 6,  6  }, //vert thumb (middle,middle,top)
    { 74, 50, 6,  6  }, //vert thumb (middle,middle,bottom)
    { 81, 35, 3, 20  }, //vert thumb (middle,right)
    { 70, 55, 4,  4  }, //vert thumb (bottom,left)
    { 74, 55, 6,  4  }, //vert thumb (bottom,middle)
    { 80, 55, 4,  4  }, //vert thumb (bottom,right)
    { 74, 41, 7,  9  }, //vert thumb (middle,middle,middle)
};

BOOL CUserInterface::OnCoolSBCustomDrawAlbumList(NMHDR* pnmh, LRESULT* pResult)
{
    NMCSBCUSTOMDRAW *pCSCD = (NMCSBCUSTOMDRAW *)pnmh;

    // Take the default processing unless we set this to something else below.
    *pResult = 0;

    if (pCSCD->dwDrawStage == CDDS_PREPAINT)
    {
        *pResult = CDRF_SKIPDEFAULT;
        return TRUE;
    }
    
    if(pCSCD->dwDrawStage == CDDS_POSTPAINT)
    {
        
    }

    RECT *rc;
    CustomDrawTable *cdt;

    //the sizing gripper in the bottom-right corner
    if (pCSCD->nBar == SB_BOTH)    
    {
        HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_SCROLLBAR_DEADAREA_COLOR));
        FillRect(pCSCD->hdc, &pCSCD->rect, hbr);
        DeleteObject(hbr);
            
        *pResult = CDRF_SKIPDEFAULT;
        return TRUE;
    }
    else if (pCSCD->nBar == SB_HORZ)
    {
        rc = &pCSCD->rect;

        if (pCSCD->uState == CDIS_SELECTED) 
            cdt = &cdt_horz_active***91;pCSCD->uItem***93;;
        else                   
            cdt = &cdt_horz_normal***91;pCSCD->uItem***93;;
        
        if ((pCSCD->uItem == HTSCROLL_PAGELEFT) ||
            (pCSCD->uItem == HTSCROLL_PAGERIGHT))

        {
            COLORREF fg = WADlg_getColor(cdt->x);
            COLORREF bg = WADlg_getColor(cdt->y);

            COLORREF color = RGB((GetRValue(fg)+GetRValue(bg))/2,
                                (GetGValue(fg)+GetGValue(bg))/2,
                                (GetBValue(fg)+GetBValue(bg))/2);

            HBRUSH hbr = CreateSolidBrush(color);
            FillRect(pCSCD->hdc, rc, hbr);
            DeleteObject(hbr);

            *pResult = CDRF_SKIPDEFAULT;
            return TRUE;
        }
        if (pCSCD->uItem == HTSCROLL_THUMB)
        {
            // get skin bitmap
            HDC hdcSkin = CreateCompatibleDC(pCSCD->hdc);
            HBITMAP oBitmap = (HBITMAP)SelectObject(hdcSkin, WADlg_getBitmap());
            SetStretchBltMode(pCSCD->hdc, COLORONCOLOR);

            // top left
            StretchBlt(pCSCD->hdc, rc->left, rc->top, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle left
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left, rc->top+4, 4, rc->bottom-rc->top-8, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // bottom left
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left, rc->bottom-4, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            
            // top middle
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->top, rc->right-rc->left-8, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle middle left
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->top+4, (rc->right-rc->left-8)/2+2, rc->bottom-rc->top-7, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle middle right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-(rc->right-rc->left-8)/2-3, rc->top+4, (rc->right-rc->left-8)/2, rc->bottom-rc->top-7, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // bottom middle
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->bottom-cdt->height, rc->right-rc->left-8, cdt->height, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);

            // top right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-4, rc->top, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-4, rc->top+4, 4, rc->bottom-rc->top-8, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // bottom right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-4, rc->bottom-4, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);

            // middle middle middle
            cdt++;
            if ((rc->right-rc->left-8) > cdt->width)
            {
                StretchBlt(pCSCD->hdc, rc->left+(rc->right-rc->left)/2-4, rc->top+4, cdt->width, cdt->height, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            }

            // free dc
            SelectObject(hdcSkin, oBitmap);
            DeleteDC(hdcSkin);

            *pResult = CDRF_SKIPDEFAULT;
            return TRUE;
        }
    }
    else if (pCSCD->nBar == SB_VERT)
    {
        rc = &pCSCD->rect;

        if (pCSCD->uState == CDIS_SELECTED)  
            cdt = &cdt_vert_active***91;pCSCD->uItem***93;;
        else                    
            cdt = &cdt_vert_normal***91;pCSCD->uItem***93;;

        if ((pCSCD->uItem == HTSCROLL_PAGEGUP) ||
            (pCSCD->uItem == HTSCROLL_PAGEGDOWN))

        {
            COLORREF fg = WADlg_getColor(cdt->x);
            COLORREF bg = WADlg_getColor(cdt->y);

            COLORREF color = RGB((GetRValue(fg)+GetRValue(bg))/2,
                                (GetGValue(fg)+GetGValue(bg))/2,
                                (GetBValue(fg)+GetBValue(bg))/2);

            HBRUSH hbr = CreateSolidBrush(color);
            FillRect(pCSCD->hdc, rc, hbr);
            DeleteObject(hbr);

            *pResult = CDRF_SKIPDEFAULT;
            return TRUE;
        }
        if (pCSCD->uItem == HTSCROLL_THUMB)
        {
            // get skin bitmap
            HDC hdcSkin = CreateCompatibleDC(pCSCD->hdc);
            HBITMAP oBitmap = (HBITMAP)SelectObject(hdcSkin, WADlg_getBitmap());
            SetStretchBltMode(pCSCD->hdc, COLORONCOLOR);

            // top left
            StretchBlt(pCSCD->hdc, rc->left, rc->top, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // top middle
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->top, rc->right-rc->left-8, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // top right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-4, rc->top, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);

            // middle left
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left, rc->top+4, 4, rc->bottom-rc->top-8, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle middle top
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->top+4, rc->right-rc->left-7, (rc->bottom-rc->top-8)/2+2, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle middle bottom
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->bottom-(rc->bottom-rc->top-8)/2-4, rc->right-rc->left-7, (rc->bottom-rc->top-8)/2, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-cdt->width, rc->top+4, cdt->width, rc->bottom-rc->top-8, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);

            // bottom left
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left, rc->bottom-4, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // bottom middle
            cdt++;
            StretchBlt(pCSCD->hdc, rc->left+4, rc->bottom-4, rc->right-rc->left-8, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // bottom right
            cdt++;
            StretchBlt(pCSCD->hdc, rc->right-4, rc->bottom-4, 4, 4, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            // middle middle middle
            cdt++;
            if ((rc->bottom-rc->top-8) > cdt->height)
            {
                StretchBlt(pCSCD->hdc, rc->left+4, rc->top+(rc->bottom-rc->top)/2-4, cdt->width, cdt->height, hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);
            }

            // free dc
            SelectObject(hdcSkin, oBitmap);
            DeleteDC(hdcSkin);

            *pResult = CDRF_SKIPDEFAULT;
            return TRUE;
        }
    }
    else
    {
        *pResult = CDRF_DODEFAULT;
        return TRUE;
    }

    // get skin bitmap
    HDC hdcSkin = CreateCompatibleDC(pCSCD->hdc);
    HBITMAP oBitmap = (HBITMAP)SelectObject(hdcSkin, WADlg_getBitmap());
    SetStretchBltMode(pCSCD->hdc, COLORONCOLOR);

    //normal bitmaps, use same code for HORZ and VERT
    StretchBlt(pCSCD->hdc, rc->left, rc->top, rc->right-rc->left, rc->bottom-rc->top,
        hdcSkin, cdt->x, cdt->y, cdt->width, cdt->height, SRCCOPY);

    // free dc
    SelectObject(hdcSkin, oBitmap);
    DeleteDC(hdcSkin);

    *pResult = CDRF_SKIPDEFAULT;
    return TRUE;
} 


[Safai]

the end result will look something like this...

[DrO]

some nice little bits of code in there

-daz

[Joonas]

Hmm, I add my listview dynamically with CreateWindow but when I press the close button on the skinned window it just closes (or hides?) the listview. What command are being sent when I press that button? And to which window?

/Jonas

[Joonas]

I posted Safai about one of these issues but I'll guess I take them both here as well.

First - The post-paint seems to erase the header controls sometimes. Often happens when the mouse is nearby...might be connected to the other issue...

Which is that I am using a virtual listview but when the header control code Safai used is implemented none of the items is shown. When removed it works fine...

Any help would be highly appreciated.

/Jonas

[Safai]

Joonas: i got your PM... but i guess i will answer here...

1) there was an issue when reordering the headers (drag&drop the column headers) that i had recently fixed but i guess the source that i posted was not up to date... sorry about that... other than that, i have never encountered that problem... replacement code for postpaint

PHP Code:

else if (CDDS_POSTPAINT == pCD->dwDrawStage)
{
    int nRight = 0;

    int nCount = Header_GetItemCount(pCD->hdr.hwndFrom);
    for (int i=0; i<nCount; ++i)
    {
        RECT rc1;
        Header_GetItemRect(pCD->hdr.hwndFrom, i, &rc1);
        nRight = max(nRight, rc1.right);
    }
    RECT rc;
    GetClientRect(&rc);

    if (nRight < rc.right)
    {
        rc.left = nRight;
        rc.right += 10;
        HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_LISTHEADER_EMPTY_BGCOLOR));
        FillRect(pCD->hdc, &rc, hbr);
        DeleteObject(hbr);
    }

    return CDRF_DODEFAULT;
} 


2) interesting, i'm using a virtual list too... that code should only affect the header, and not the list itself... might be you're not handling the WM_NOTIFY properly?? did you check for notification for ID '0' only? in your WM_NOTIFY handler, it has to check

pnmh->idFrom == 0
pnmh->code == NM_CUSTOMDRAW

[Safai]

Originally posted by Joonas
Hmm, I add my listview dynamically with CreateWindow but when I press the close button on the skinned window it just closes (or hides?) the listview. What command are being sent when I press that button? And to which window?

/Jonas

from my understanding, WM_CLOSE is sent to the window that is the child (most likely your listview) of wadlg...

and this is what i do in mine...

PHP Code:

BOOL CUserInterface::OnClose()
{
    GetParent().ShowWindow(SW_HIDE);
    m_bShow = FALSE;
    return TRUE;
} 


[DrO]

code:
case WM_USER+101:

case WM_DESTROY:



	// gets the top left corner for position restoring

	if(jumpex_skin && jumpex_pos){

	RECT rc;

		GetWindowRect((jumpex_skin?jump_skin:hwnd),&rc);

		jumpex_pos_x = rc.left;

		jumpex_pos_y = rc.top;

	}



	// if the window is already open then force a close of it

	// this will handle issues currently with the skin window not

	// working correctly (only if clicked on so switching on/off 

	// will work)

	if(msg == WM_USER+101 && jump_wnd){

		DestroyWindow(jump_wnd);

	}



	jump_skin = 0;

break;

i've just taken this out of jtfe from my subclassed window frame proc. by catching either of the messages it's possible to allow/disallow a close and call the relevant window destructors, etc

-daz

[Joonas]

Here is my windowproc...same problem as before though even with that new postpaint. if you have the time you could perhaps point out what I'm doing wrong...

LRESULT bleh = CallWindowProc(oProc, wnd, msg, wParam, lParam);

switch (msg)
{
case WM_NOTIFY:
{
switch (((LPNMHDR)lParam)->code)
{
case LVN_GETDISPINFO: // dynamisk fråga om listview item som ska ritas upp
{
LV_DISPINFO* pDispInfo = (LV_DISPINFO*)lParam;

if (pDispInfo->item.mask & LVIF_TEXT)
{
int index = pDispInfo->item.iItem;

FILESTRUCT* t = general->getFileStruct(index);

if (!general->isTagged(t))
{
ID3* tag = general->getWinampInterface()->getID3(strdup(t->path.c_str()));
t->title = strdup(tag->title);
t->artist = strdup(tag->artist);
t->album = strdup(tag->album);
t->tagged = true;

delete tag;
}

switch (pDispInfo->item.iSubItem)
{
case 0: strcpy(pDispInfo->item.pszText, t->title); break;
case 1: strcpy(pDispInfo->item.pszText, t->artist); break;
case 2: strcpy(pDispInfo->item.pszText, t->album); break;
case 3: strcpy(pDispInfo->item.pszText, t->path.c_str()); break;
}
} // end if

break;
} // end case

break;
} // end switch


case LISTVIEW: // om meddelandet gäller listview
{
NMCUSTOMDRAW* pCD = (NMCUSTOMDRAW*)lParam;

switch (pCD->dwDrawStage)
{
case CDDS_PREPAINT:
{
if (pCD->hdr.idFrom != 0)
return CDRF_NOTIFYITEMDRAW|CDRF_NOTIFYPOSTPAINT;
}

case CDDS_ITEMPREPAINT:
{
HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_LISTHEADER_BGCOLOR));
FillRect(pCD->hdc, &pCD->rc, hbr);
DeleteObject(hbr);

HPEN pen1=CreatePen(PS_SOLID,0,WADlg_getColor(WADLG_LISTHEADER_FRAME_TOPCOLOR));
HPEN pen2=CreatePen(PS_SOLID,0,WADlg_getColor(WADLG_LISTHEADER_FRAME_MIDDLECOLOR));
HPEN pen3=CreatePen(PS_SOLID,0,0);
HGDIOBJ o=SelectObject(pCD->hdc, pen1);

if ((pCD->uItemState & CDIS_SELECTED) == CDIS_SELECTED)
{
SelectObject(pCD->hdc, pen3);
MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom,NULL);
LineTo(pCD->hdc,pCD->rc.left,pCD->rc.top);
LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.top);
LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.bottom-1);
LineTo(pCD->hdc,pCD->rc.left,pCD->rc.bottom-1);
}
else
{
// left, top
MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom,NULL);
LineTo(pCD->hdc,pCD->rc.left,pCD->rc.top);
LineTo(pCD->hdc,pCD->rc.right,pCD->rc.top);

// right, bottom
SelectObject(pCD->hdc, pen2);
MoveToEx(pCD->hdc,pCD->rc.left+1,pCD->rc.bottom-2,NULL);
LineTo(pCD->hdc,pCD->rc.right-2,pCD->rc.bottom-2);
LineTo(pCD->hdc,pCD->rc.right-2,pCD->rc.top);

// right, bottom (black line)
SelectObject(pCD->hdc, pen3);
MoveToEx(pCD->hdc,pCD->rc.left,pCD->rc.bottom-1,NULL);
LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.bottom-1);
LineTo(pCD->hdc,pCD->rc.right-1,pCD->rc.top-1);
}

SetTextColor(pCD->hdc, WADlg_getColor(WADLG_LISTHEADER_FONTCOLOR));
SetBkMode(pCD->hdc, TRANSPARENT);

char name[MAX_PATH] = "";
HDITEM hdi;
hdi.mask = HDI_TEXT|HDI_FORMAT;
hdi.pszText = name;
hdi.cchTextMax = MAX_PATH;
Header_GetItem(pCD->hdr.hwndFrom, pCD->dwItemSpec, &hdi);

RECT rcItem = pCD->rc;
rcItem.left += 5;
rcItem.right -= 5;
rcItem.top ++;

if ((pCD->uItemState & CDIS_SELECTED) == CDIS_SELECTED)
{
rcItem.left ++;
rcItem.right ++;
}

int align = (hdi.fmt & HDF_JUSTIFYMASK) == LVCFMT_RIGHT ? DT_RIGHT : DT_LEFT;

DrawText(pCD->hdc, hdi.pszText, -1, &rcItem, align | DT_VCENTER | DT_SINGLELINE);

// clean up
SelectObject(pCD->hdc, o);
DeleteObject(pen1);
DeleteObject(pen2);
DeleteObject(pen3);

return CDRF_SKIPDEFAULT;
}

case CDDS_POSTPAINT:
{
int nRight = 0;

int nCount = Header_GetItemCount(pCD->hdr.hwndFrom);
for (int i=0; i<nCount; ++i)
{
RECT rc1;
Header_GetItemRect(pCD->hdr.hwndFrom, i, &rc1);
nRight = max(nRight, rc1.right);
}

RECT rc;
GetClientRect(listview->getHWND(), &rc); //listview is an object of mine that acts a wrapper for the listview control, returns the HWND for it...

if (nRight < rc.right)
{
rc.left = nRight;
rc.right += 10;
HBRUSH hbr = CreateSolidBrush(WADlg_getColor(WADLG_LISTHEADER_EMPTY_BGCOLOR));
FillRect(pCD->hdc, &rc, hbr);
DeleteObject(hbr);
}

return CDRF_DODEFAULT;
} // end case NM_CUSTOMDRAW
} // end switch
} // end case LISTVIEW
} // end case WM_NOTIFY
} // end switch

return bleh;

[Joonas]

Sorry, this line is of course invalid:

if (pCD->hdr.idFrom != 0)
return CDRF_NOTIFYITEMDRAW|CDRF_NOTIFYPOSTPAINT;

it should only be

return CDRF_NOTIFYITEMDRAW|CDRF_NOTIFYPOSTPAINT;

but the problem remains...

/Jonas

[Safai]

what's that LISTVIEW thing? shouldn't it be NM_CUSTOMDRAW, also, getting the CUSTOMDRAW message should be done inside the listview's window proc

assuming the code you posted is not listview's, if it is, i don't know how that LVN_GETDISPINFO notification will work in your case, cuz that should be handled in the parent of listview...

[Joonas]

I don't know if I have a Listview window proc per se...I am not using MFC...I run it all through the window procedure...guess I have to try something later on..

[Joonas]

Originally posted by Safai
Joonas: i got your PM... but i guess i will answer here...

2) interesting, i'm using a virtual list too... that code should only affect the header, and not the list itself... might be you're not handling the WM_NOTIFY properly?? did you check for notification for ID '0' only? in your WM_NOTIFY handler, it has to check

pnmh->idFrom == 0
pnmh->code == NM_CUSTOMDRAW

It was exactly that idFrom that was the problem Thanks!

/Jonas

[Joonas]

Jesus...the problems never end. Bear with me for a little bit more

When I add (dynamically) the listview to the window I receive back from

window = (HWND)SendMessage(winampWindow, WM_WA_IPC, (WPARAM)wa_wnd, IPC_GET_EMBEDIF);

everything seems well, except that when the window is moved (repainted I presume) it auto-fits the listview to cover the entire window which isn't exactly what I want...I have other controls I want to add there too...anyone knows why it does that?

/Jonas