Possibly only Cosmetic change

[epage]

Code: Select all

void wxFrameManager::OnFloatingPaneMoving(wxWindow* wnd)
{
...
    // no hint for toolbar floating windows
    if (pane.IsToolbar() && m_action == actionDragFloatingPane)
    {
        if (m_action == actionDragFloatingPane)
        {
            ...
        }
        return;
    }
...
}


is equivelant to

Code: Select all

void wxFrameManager::OnFloatingPaneMoving(wxWindow* wnd)
{
...
    // no hint for toolbar floating windows
    if (pane.IsToolbar() && m_action == actionDragFloatingPane)
    {
        ...
        return;
    }
...
}


The extra if-statement was redundant, but was something else meant?

[epage]

Code: Select all

    if (pane.IsToolbar() && m_action == actionDragFloatingPane)
    {
        if (m_action == actionDragFloatingPane)
        {


Code: Select all

    if (pane.IsToolbar())
    {


For me, if I pulled out a toolbar and moved it over a docking area again, it will dock without a mouse release.
If I released the mouse and left it floating, and then docked the toolbar, it would act like a docking window.

This makes it always act like the first case