typing in search field of date/time settings blocks UI

[Bill Filler]

Steps to reproduce:
- reflash your phone
- go to date/time settings
- start typing "Boston" in search field

Expected results:
- Should see the text "Boston" immediately entered into the search field, with results updated as each letter is typed

Actual results:
- I only see "B" entered into the text field and everything blocks until the first set of search results comes back, then "Boston" is filled in the text field

Seems the search must be happening in the same thread as UI which is causing typing of the search string to block

[Iain Lane (laney)]

On Fri, Sep 04, 2015 at 01:14:11PM -0000, Bill Filler wrote:
> Public bug reported:
>
> Steps to reproduce:
> - reflash your phone
> - go to date/time settings
> - start typing "Boston" in search field
>
> Expected results:
> - Should see the text "Boston" immediately entered into the search field, with results updated as each letter is typed
>
> Actual results:
> - I only see "B" entered into the text field and everything blocks until the first set of search results comes back, then "Boston" is filled in the text field
>
> Seems the search must be happening in the same thread as UI which is
> causing typing of the search string to block

No, we use QtConcurrent to do that.

It's probably something else, like incorrect blocking or maybe a
different part which is done synchronously.

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

[Jonas G. Drange (jonas-drange)]

It seems we can improve the following and kill this bug in the process:
 * move qSort out of GUI thread into the worker thread,
 * instantiate timedatepanel plugin only once,
 * and move away from the deprecated pattern of subclassing QThread.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-system-settings - 0.3+15.10.20150918-0ubuntu1

---------------
ubuntu-system-settings (0.3+15.10.20150918-0ubuntu1) wily; urgency=medium

  [ jonas-drange ]
  * [hotspot] hide/show based on modem availability from Connectivity
    (which also requires a bumped dep). (LP: #1487157)
  * [time-date] Migrate threaded code to worker-object pattern, move
    sorting to a worker thread from the GUI thread, and only instantiate
    UbuntuTimeDatePanel plugin once. (LP: #1492260)
  * [wifi/wpa2ep] allow passwords of length 1 for wpa enterprise
    authentication schemes. (LP: #1490417)

 -- <email address hidden> (Jonas G. Drange) Fri, 18 Sep 2015 14:20:27 +0000