amantia requested changes to this revision.
amantia added a comment.
This revision now requires changes to proceed.
Unfortunately the current approach fails when the folderview points to
desktop:/ . In that case the targetUrl in the copyJobStarted is different
("desktop:/foo") from the url we get from the dirlister model
("file://home/user/Desktop/./foo") and thus it fails to find the saved
positions from the hash.
Another note is that in the "map" lambda, there is a need to call
m_screenMapper->addMapping(targetUrl.toString(), m_screen,
ScreenMapper::DelayedSignal);
so it goes to the right screen (as soon as https://phabricator.kde.org/D8493
gets in). This one could be done in a followup commit though.
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D8598
To: mwolff, hein, amantia
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg,
abetts, sebas, apol, mart
