In T15613#279133, @ngraham wrote:

because the subwindow for selecting a property to change completely closes after setting every single one

Thankfully that's like a one-line change. :) Feel free to propose it in the form of a merge request or a bug report.

In T12472#259169, @nicolasfella wrote:

Remaining users of KIO::AccessManager or KIO::Integration::AccessManager are:

• basket (https://invent.kde.org/utilities/basket/-/merge_requests/38)

In T12724#255354, @aronkvh wrote:

I'd suggest keeping Milky Way or Rainy Morning

About this issue, there are currently three ways to launch a KCM, maybe more if we include Plasma::AppletConfiguration.

I was planning to use it on https://invent.kde.org/frameworks/kcmutils/-/merge_requests/27.

Independent of more thoughtful discussions, I'd like to propose to change how (mathematically) the launch count gets applied.

Any chance to retake this and move it to invent?

Crystalline: +1
Grand Canyon: -2
Vera: -2
Beach: +1
Flow Dark: +1
Altai: +1
Milky Way: +2
Rainy Morning: +2
Slant: -1
Kay: -2
Iridescent Shell: +1

Yay! https://invent.kde.org/frameworks/kio/-/merge_requests/80

If I recall/guess correctly this comes originally from konqueror, so it is probably used there too.
And that might be also why this kcm is still in kio repository, which surprised me a lot.

Now that krunner calls System Settings directly, I think it'd make sense to move webshortcuts from shortcuts to search group.
Despite the name, I find that it doesn't fit very well with the keyboard shortcuts, being more a "search providers" thing than the usual meaning a "shortcut" has in desktop SW.

Nice! I had some redish patch depending on this :D

LGTM!

No rush, I just wanted to use Requested Changes for the first and last time here at Phab :D

Ping @ognarb? I think it's just the fileUrl thing that's missing.

• Make it const

• Address comments
• Use match() to get the index

• Add indexOf() method
• Delegate suggestions to a different method

ping on this?