Discussion:
[krunner] [Bug 389611] New: Milou cancels/resets the search if there are no first result after 500 ms
(too old to reply)
Friedrich W. H. Kossebau
2018-01-29 18:02:47 UTC
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=389611

Bug ID: 389611
Summary: Milou cancels/resets the search if there are no first
result after 500 ms
Product: krunner
Version: 5.11.95
Platform: Other
OS: Linux
Status: UNCONFIRMED
Severity: major
Priority: NOR
Component: general
Assignee: ***@privat.broulik.de
Reporter: ***@kde.org
Target Milestone: ---

When using a krunner plugin which takes some time to deliver results (e.g.
because querying remote services) and no other krunner plugins deliver results
as well quickly, the internal delayed model reset timeout snaps in and does a
complete clearing of the model, but also the matching manager, incl. the match
context.

So one the krunner plugins got their data prepared, they find the match context
they worked for now being invalid.

See class Milou::SourcesModel, its m_resetTimer and
SourcesModel::slotResetTimeout.

No immediate own idea how to fix that, so braindumping things for now here.

Can be triggered e.g. with the (ported version of the) mediawiki krunner plugin
from kdeplasma-addons.
--
You are receiving this mail because:
You are watching all bug changes.
David Edmundson
2018-03-13 15:12:33 UTC
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=389611

David Edmundson <***@davidedmundson.co.uk> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@davidedmundson.co.uk

--- Comment #1 from David Edmundson <***@davidedmundson.co.uk> ---
Relevant commit message:

SourcesModel: Delay clearing the model on a new query

We delay querying the model by about 500msec or until we get results when
the query string is changed. This way, we avoid the extra resizing of
the window between when the results show up and the query is sent


It doesn't sound like it should be clearing the matching manager and match
context, it should be just a visual thing.
--
You are receiving this mail because:
You are watching all bug changes.
Simon Andric
2018-04-05 10:58:17 UTC
Permalink
Raw Message
https://bugs.kde.org/show_bug.cgi?id=389611

Simon Andric <***@gmail.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |***@gmail.com
--
You are receiving this mail because:
You are watching all bug changes.
Loading...