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

--- Comment #4 from Vlad Zagorodniy <vladz...@gmail.com> ---
Argh, this piece of code assumes that only kcm adds virtual desktops (during
synchronization)

        // Remove dummy data.
        const QString dummyId = m_desktops.at(data.position);

but that's not the case. It's also worth to point out that we can't just do

    if (uint(m_desktops.count()) < data.position) {
        // Replace the dummy virtual desktop with the actual one.
        emit dataChanged(idx, idx, { Id });
    } else {
        beginInsertRows();
        // Append the new virtual desktop.
        endInsertRows();
    }

because this will overwrite virtual desktops created by the user.

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

Reply via email to