• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 875
  • Last Modified:

CListBox ownerdraw

I'm writing a VC++ Dialog type application. Since I want to have some listbox with black background and white text, I created my own class  CModListBox which is based on CListBox.

so.. the followng listed what I did:
1. Create a class:  Class CModListBox : public CListBox
2. Add virtual function Drawitem and I use exactly the same code list in MSDN:
void CModListBox::DrawItem(LPDRAWITEMSTRUCT lpDrawItemStruct)
      // TODO: Add your code to draw the specified item
   //ASSERT(lpDrawItemStruct->CtlType == ODT_LISTBOX);
   LPCTSTR lpszText = (LPCTSTR) lpDrawItemStruct->itemData;
   //ASSERT(lpszText != NULL);  
   CDC dc;
   // Save these value to restore them when done drawing.
   COLORREF crOldTextColor = dc.GetTextColor();
   COLORREF crOldBkColor = dc.GetBkColor();
   // If this item is selected, set the background color
   // and the text color to appropriate values. Also, erase
   // rect by filling it with the background color.
   if ((lpDrawItemStruct->itemAction | ODA_SELECT) &&
      (lpDrawItemStruct->itemState & ODS_SELECTED))  
   // If this item has the focus, draw a red frame around the   // item's rect.
   if ((lpDrawItemStruct->itemAction | ODA_FOCUS) &&
      (lpDrawItemStruct->itemState & ODS_FOCUS))  
      CBrush br(RGB(255, 0, 0));
      dc.FrameRect(&lpDrawItemStruct->rcItem, &br);  
   }   // Draw the text.
   dc.DrawText(      lpszText,      strlen(lpszText),
      &lpDrawItemStruct->rcItem,      DT_CENTER|DT_SINGLELINE|DT_VCENTER);
   // Reset the background color and the text color back to their
   // original values.   dc.SetTextColor(crOldTextColor);

3. Add a variable CModListBox m_ctlScheme in my main dialog box

4. Add this line in the OnCreate() (of main dialog box)


Then whenever I try to click on the listbox, or try to use "addstring" to do something to the listbox, it gives an error msg to me.
Is there anything else I need to do ??  This is the first time I create a class to use the ownerdraw function so... :)
  • 5
  • 5
1 Solution
don't create it like that.

Put a normal listbox in your dialog (make sure it is owner-drawn etc).

Then use DDX_Control to subclass it.  Alternatively, use SubclassWindow if you don't like MFC's easy dataexchange and ClassWizard

Then that listbox in the dialog will be associated with you m_ctlScheme and m_ctlScheme will get the message etc.

BTW: instead of attaching and detaching from a DC, use this...

CDC* pDC = CDC::FromHandle(lpDIS->hDC);

then use pDC-> and no need to detatch, delete etc the pDC.

You _could_ use OnCtlColor to change the colour (background and text) instead.  Then you don't need to make it owner drawn.

rw8Author Commented:
can you give me an example of how to use OnCtlColor to change the colour? Do I need to derive a new class?
Free Tool: SSL Checker

Scans your site and returns information about your SSL implementation and certificate. Helpful for debugging and validating your SSL configuration.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Hi rw8
   RONSLOW is right. Just for changing the text and background color you need not make is owner drawn. OnCtlColor works but if I am right, text color and only background color of the text changes and not the background color of the control.
   Take a look at the Technical article 62(TN062) in MSDN. You will find a step by step procedure to change the text and bkground color of an edit box. Just apply it for the list box.

All the best.

rw8Author Commented:
Yes. It works! But I have a problem... the listbox remains the default colour (black text over white background) unless I put "invalidate" in the CtlColor function..  however, if I do this.. the listbox keeps on redrawing itself ..
should I put Invalidate() in somewhere else?
show us your code .. that behaviour shouldn't be happening
rw8Author Commented:
this is the class I created (base on CListBox)

      m_text = RGB(255, 0, 0 );
    m_background = RGB( 0, 255, 0 );
    m_brBkgnd.CreateSolidBrush( m_background );



// CBlackListBox message handlers

HBRUSH CBlackListBox::CtlColor(CDC* pDC, UINT nCtlColor)
      // TODO: Change any attributes of the DC here
     pDC->SetTextColor( m_text );    // text
     pDC->SetBkColor( m_background );    // text bkgnd
     return m_brBkgnd;                // ctl bkgnd
      // return NULL;


and I add a ListBox in my main dialog box and attach a CBlacklistbox ctl varible to it.. and I didn't change any settings of the the listbox.
On first inspection it looks ok .. I'll give it a try when I havea few mintes.

rw8Author Commented:
wait ronslow, I just created another listbox and it seems ok now. I'll do some more tests now
the plot thickens....
rw8Author Commented:
Thanx RONSLOW. It's working now. And thanx  V Bapat too. :)
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Get your problem seen by more experts

Be seen. Boost your question’s priority for more expert views and faster solutions

  • 5
  • 5
Tackle projects and never again get stuck behind a technical roadblock.
Join Now