This is fresh code and ocx

 

 

 

Yes, the 'click' problem was fixed with your first post, it was only the icon alignment that still needed fixing.

 

I made the changes you have shown for the icon alignment, but it still didn't work correctly.  After a bit of debugging I found the problem to be that the mask values in the code shown in your post (0x0F00000 and 0x00F0000) are only 7 digits long, they need an extra 0 on the end so they become 0x0F000000 and 0x00F00000.  With those changes in place everything now works as expected.

 

I also extend it to cover combination of alignment flags

 

I just moved to version 13.1 myself and this bug has re-appeared as I had feared, so it's good to get an offical fix for it at last.

 

You said "I consider the way to click on item rectangle (not only on glyph) is simplification for clients", but as I pointed out in my original post this just doesn't work if you have text AND a checkbox in the same cell, clicking on the text to edit it causes the checkbox to change state which is clearly wrong.

 

So thanks for the fix, I've applied it and all is well again, although it looks like the other bug in this method that I reported in my original post is still present, i.e. if you have BOTH a horizontal and vertical alignment set it won't work correctly becuse no masking is done on the nIconAlign value (to extract the horizontal and vertical styles) before each switch statement. 

 

 

I changed it to glyph rectangle now.

 

You can apply this update yourself - add last line in function

 

 

Thanks.

 

 

 

 

In the previous version you had to click in the checkbox to change it (as expected), now clicking anywhere in the cell causes it to change!  This is especially problematic when you also have text in the cell, since double clicking the cell to edit the text also causes the checkbox to change state! 

 

I don't know if this is a bug, but I would like to get back to the previous behavior, can you tell me how to do this?

 

Thanks.

 

PS. I just had a look at the code and the problem appears to be in CXTPReportRecordItem::DrawCheckBox() method which no longer adjusts the m_rcGlyph member based on the icon position, and instead uses a copy of this rect!