Forum

Hi

There is a bug in ExtButton.cpp (using profuis version 2.87, but also present in 2.90)
Both the CExtCheckBox and the CExtRadioButton don’t draw their states correctly.
When a checkbox or radio has a state that is not BST_UNCHECKED, then it is drawn as if it were being pressed.
Not only will this draw the checkbox and radio incorrectly, it also breaks the hover behaviour when they’re checked.

This bug occurs in the methods:

CExtCheckBox::_RenderImpl
CExtRadioButton::_RenderImpl

In this line:

bool bPushed = (m_bPushed || GetCheck()) ? true : false;

The checked state of a radio or checkbox has nothing to do with the button being pressed.

It should therefore be

bool pPushed = m_bPushed;

or just use m_bPushed instead of pPushed.

Once you change this, the drawing and hover behavior are correct.

Thank you for reporting this issue. The bool bPushed = (m_bPushed || GetCheck()) ? true : false; code is really should be the bool pPushed = m_bPushed; code.