Code review comment for lp:~lukas-kde/unity8/dynamicLauncherIcons

Revision history for this message
Michael Zanetti (mzanetti) wrote :

resetting the model will mess up scrolling positions, it would destroy opened quicklists and what not.

Which I understand that some sort of hack will be needed, I'd suggest something easier for the view. Perhaps like this?

QString tmp = m_icon;
m_icon.clear();
emit dataChanged({RoleIcon});
m_icon = tmp;
emit dataChanged({RoleIcon});

review: Needs Information

« Back to merge proposal