https://bugs.kde.org/show_bug.cgi?id=365105

--- Comment #6 from David Edmundson <k...@davidedmundson.co.uk> ---
it's getting inside the first if() in here:


void KStatusNotifierItemDBus::Activate(int x, int y)
{
    if (m_statusNotifierItem->d->associatedWidget ==
m_statusNotifierItem->d->menu) {
        ContextMenu(x, y);
    } else {
        m_statusNotifierItem->activate(QPoint(x, y));
    }
}


The reason being d->assoociatedWidget is null
and d->menu is null

hence it matches, but it's not really what that code is trying to do.

Doing the obvious fix on that fixes the bug. (plus again in
KStatusNotifierItem::activate where the code is repeated for no reason)

However, d->menu shouldn't  be null - you're clearly setting one.

Will investigate that - or maybe just get rid of the plasma SNI integration
given that's in Qt now..

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to