leinir abandoned this revision.
leinir added a comment.
Incidentally, because KNS itself will already postpone non-cached searches by
one second, the new proposed solution above (to only to this for KNS sources)
is already implemented by way of using https://phabricator.kde.org/D6191
apol requested changes to this revision.
apol added a comment.
This revision now requires changes to proceed.
Bump?
REPOSITORY
R134 Discover Software Store
REVISION DETAIL
https://phabricator.kde.org/D5767
To: leinir, apol
Cc: jensreuterberg, markg, plasma-devel, ZrenBot, spstarr,
apol added a comment.
In https://phabricator.kde.org/D5767#108317, @jensreuterberg wrote:
> Sry for seeming obtuse here but isn't the results from previous queries
saved between sessions and displayed for the search?
>
> I mean I see that when doing a search within "Plasma Addons"
jensreuterberg added a comment.
Sry for seeming obtuse here but isn't the results from previous queries saved
between sessions and displayed for the search?
REPOSITORY
R134 Discover Software Store
REVISION DETAIL
https://phabricator.kde.org/D5767
To: leinir, apol
Cc: jensreuterberg,
leinir added a comment.
In https://phabricator.kde.org/D5767#108234, @apol wrote:
> It could make sense to move this to the KNSBackend, any appstream-based
backends don't have such a problem.
Good point indeed - while it isn't really kns specific, we don't have any
other
apol added a comment.
It could make sense to move this to the KNSBackend, any appstream-based
backends don't have such a problem.
REPOSITORY
R134 Discover Software Store
REVISION DETAIL
https://phabricator.kde.org/D5767
To: leinir, apol
Cc: markg, plasma-devel, ZrenBot, spstarr,
leinir added a comment.
In https://phabricator.kde.org/D5767#108088, @markg wrote:
> I don't think adding a (rather massive) delay is the real fix here. It only
masks the actual issue.
>
> What really happens (just opened the discover store for the first time
ever) is that entries
markg added a comment.
I don't think adding a (rather massive) delay is the real fix here. It only
masks the actual issue.
What really happens (just opened the discover store for the first time ever)
is that entries can flow in at any point, that might be an issue.
Every batch can
leinir created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
We currently postpone searches 200ms, but as this causes a great deal of
online traffic, and no small amount of visual clutter (a whole lot of