Switch to KNSQuick for getting new wallpapers
ClosedPublic

Authored by leinir on Jan 14 2020, 2:37 PM.

Details

Summary

This switches the image plugin to using KNewStuffQuick instead of
launching the widget dialog through that ol' widget dialog hack.
It further uses the big preview mode by default, which just seems
to make a lot of sense for wallpapers.

Test Plan

Without: Uses the old dialog
With: Uses the new dialog, as seen below

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
leinir created this revision.Jan 14 2020, 2:37 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 14 2020, 2:37 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
leinir requested review of this revision.Jan 14 2020, 2:37 PM
mart accepted this revision.Jan 14 2020, 2:37 PM
This revision is now accepted and ready to land.Jan 14 2020, 2:37 PM
leinir edited the test plan for this revision. (Show Details)Jan 14 2020, 2:40 PM
leinir added reviewers: Plasma, ngraham.
This revision was automatically updated to reflect the committed changes.
downloadNewWhat: i18n("Wallpapers")

But now the translation does not work.
This is reported here https://bugs.kde.org/show_bug.cgi?id=415541
Now I am fixing this error with three patches.




Before:

After:

What about the "Get New Plugins" button in the same dialog? It still uses the old style.
Probably unrelated, but for me, both of these buttons fail to actually load anything, stuck connecting forever. I would blame all of my internet connections that I tried it with, but it does not happen in System Settings, so maybe there's some bug here.

And also, looking at @victorr's screenshot, I see that the "Close" button has no label on it. I got same problem here as well.

What about the "Get New Plugins" button in the same dialog? It still uses the old style.
Probably unrelated, but for me, both of these buttons fail to actually load anything, stuck connecting forever. I would blame all of my internet connections that I tried it with, but it does not happen in System Settings, so maybe there's some bug here.

The reason that wasn't changed in this step is simple - because it isn't simple ;) You'll notice that the change here was very much a trivial swap. Changing the other KNS integration points out for the new one will be considerably more effort, hence it isn't happening just yet.

And also, looking at @victorr's screenshot, I see that the "Close" button has no label on it. I got same problem here as well.

That really is a very strange error. Not sure why such a thing would fail. The code in question can be seen here: https://phabricator.kde.org/source/knewstuff/browse/master/src/qtquick/qml/Dialog.qml$98 - However, the whole thing is also being discussed elsewhere, making this the wrong place. Would be good to get more eyes on the situation :) https://bugs.kde.org/show_bug.cgi?id=413434

What about the "Get New Plugins" button in the same dialog? It still uses the old style.

The reason that wasn't changed in this step is simple - because it isn't simple ;) You'll notice that the change here was very much a trivial swap. Changing the other KNS integration points out for the new one will be considerably more effort, hence it isn't happening just yet.

But now we have three different user interfaces for getting KNS content: 1. Discover 2. old dialog 3. new dialog. This is really kind of non-ideal. I was hoping we could swap out all the old dialogs for the new ones throughout System Settings and Plasma for 5.18. In the long term, I'd like to have one interface for all GHNS stuff, whether it be the new dialog or Discover, or something else. But three is now a regression in our quest for Goal: Consistency. :/

And also, looking at @victorr's screenshot, I see that the "Close" button has no label on it. I got same problem here as well.

That really is a very strange error. Not sure why such a thing would fail. The code in question can be seen here: https://phabricator.kde.org/source/knewstuff/browse/master/src/qtquick/qml/Dialog.qml$98 - However, the whole thing is also being discussed elsewhere, making this the wrong place. Would be good to get more eyes on the situation :) https://bugs.kde.org/show_bug.cgi?id=413434

Just delete it :)