subsequent searches expand first row in search results
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| software-center (Ubuntu) |
Medium
|
Unassigned |
Bug Description
When searching sometimes the first row is expaned but not selected, this loos rather odd. To reproduce:
- open software-center
- search for "foo"
- select one of the search results
- search for "bar"
The first row in the search is expanded but not selected. It should not expand.
Related branches
- Kiwinote: Approve on 2012-03-30
-
Diff: 19 lines (+8/-1)1 file modifiedsoftwarecenter/ui/gtk3/widgets/apptreeview.py (+8/-1)
Michael Vogt (mvo) wrote : | #1 |
Changed in software-center (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Michael Vogt (mvo) wrote : | #2 |
It appears the issue is that clear_model() needs to disconnect the view to avoid a whole bunch of "cursor_changed" signals.
Launchpad Janitor (janitor) wrote : | #3 |
This bug was fixed in the package software-center - 5.1.14
---------------
software-center (5.1.14) precise; urgency=low
[ Michael Vogt ]
* lp:~mvo/software-center/lp962580:
- add locking to the expunge helper process to fix bugs that are triggered
if multiple expunge cache processes are run (LP: #962580)
* lp:~mvo/software-center/cache-refresh-glitch:
- ensure that we get a full refresh if a pkg was not available before
show_app is called
* lp:~mvo/software-center/lp940482:
- fix crash if the debfile does not return proper utf8 for the
description (LP: #940482)
* lp:~mvo/software-center/lp966514:
- properly handle network disconnect conditions with the Ubuntu
SSO dialog (LP: #966514)
* lp:~mvo/software-center/lp966879:
- fix for crashes in the installed view treeview (LP: #966879,
LP: #950899)
* lp:~mvo/software-center/lp846204:
- fix ValueError crashes in get_iter due to invalid tree paths
(LP: #846204)
* lp:~mvo/software-center/lp964433:
- disconnect the model from the view before calling set_from_matches
(LP: #964433)
* lp:~mvo/software-center/treeview-keep-state-on-db-cache-change:
- restore the state of the installed view treeview when the
the db or cache changes, such as on an app install or remove
* lp:~mvo/software-center/946393:
- fix installing multiple apps when in a custom list view (LP: #946393)
* lp:~mvo/software-center/lp969050:
- disconnect the view when the model is cleared to avoid a furry of
cursor_changed signals as the rows get removed (LP: #969050)
[ Gary Lasker ]
* lp:~gary-lasker/software-center/installed-pane-refresh:
- avoid rebuilding the treeview in the installedpane if its not
required (LP: #828887)
* lp:~gary-lasker/software-center/fix-crash-lp967036:
- Small branch to fix a crash due to a UnicodeDecodeError when accessing
the short description for H/W requirements (LP: #967036)
* lp:~gary-lasker/software-center/fix-crash-lp935930:
- fix a crash due to a UnicodeDecodeError (LP: #935930)
-- Michael Vogt <email address hidden> Fri, 30 Mar 2012 18:00:50 +0200
Changed in software-center (Ubuntu): | |
status: | Confirmed → Fix Released |
Disabling _update_ selected_ row fixes it, so its something to do with that:
=== modified file 'softwarecenter /ui/gtk3/ widgets/ apptreeview. py' ui/gtk3/ widgets/ apptreeview. py 2012-03-30 07:26:46 +0000 ui/gtk3/ widgets/ apptreeview. py 2012-03-30 10:16:58 +0000
self. _update_ selected_ row(view, tr, path)
--- softwarecenter/
+++ softwarecenter/
@@ -285,6 +296,7 @@
def _update_ selected_ row(self, view, tr, path=None):
+ return
# keep track of the currently selected row renderer and associated
# doc for use when updating the widgets and for use with the Unity
# integration feature