jovibor / listex Goto Github PK

Hi,

adding a column with width 0 causes an assertion failure in the method:
void CListExHdr::OnDrawItem(CDC* pDC, int iItem, CRect rect, BOOL bIsPressed, BOOL bIsHighlighted)
when the CMemDC memDC(*pDC, rect); is created.
Inserting a test helps :
if(rc.IsRectEmpty())
return;

Also in the CListEx class OnPaint() method when the MemDC is created it might crash.

Thanks
Ralf

1.vs2019, I uncomment the code snippet, and run the sample. the result is "Debug Assertion Failed!"
//For classical list.
/*
...
m_pList->SetItemText(0, 1, L"Test item - row:0/column:1....");
...
*/

2. use other way:
   int nIndex = m_pList->GetItemCount();
   LV_ITEM lvitemAdd = { 0 };
   lvitemAdd.mask = LVIF_TEXT;
   lvitemAdd.iItem = nIndex;
   lvitemAdd.iSubItem = 0;
   lvitemAdd.pszText = _T("Test item - row:0");;
   if (m_pList->InsertItem(&lvitemAdd) != -1)
   {
   LV_ITEM lvitem = { 0 };
   lvitem.mask = LVIF_TEXT;
   lvitem.iItem = nIndex;
   lvitem.iSubItem = 1;
   lvitem.pszText = _T("Test item - row:1");
   m_pList->SetItem(&lvitem);
   }
   the cell is null.

ListEx allows multiple rows to be selected by holding ctrl (or shift) and left clicking.

The Windows list control supports the same behaviour with ctrl (or shift) + up/down arrow keys. Suggest ListEx could be improved if it also supported keyboard input for multi-selection.

The cell font size can be changed using Ctrl-Mouse Wheel. However, this doesn't work for cells that contain hyperlinks.