apport-gtk: clicking on the details tree jumps the scroll position to top

Bug #1387328 reported by Marius Gedminas on 2014-10-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Undecided
Unassigned

Bug Description

1. Wait for a crash or run apport-bug /var/crash/something.crash
2. Click Show Details
3. Scroll down the tree view until you see something interesting like Stacktrace
4. Click 'Stacktrace' to expand it

What happens:
- tree view gets focus, 1st (i.e. topmost) item in it gets selected and scrolled into view

What I expect:
- tree view gets focus, the 'Stacktrace' item gets selected and view is not scrolled away from what I was interested in.

ProblemType: Bug
DistroRelease: Ubuntu 14.10
Package: apport-gtk 2.14.7-0ubuntu8
ProcVersionSignature: Ubuntu 3.16.0-23.31-generic 3.16.4
Uname: Linux 3.16.0-23-generic x86_64
ApportVersion: 2.14.7-0ubuntu8
Architecture: amd64
CurrentDesktop: GNOME
Date: Wed Oct 29 21:10:33 2014
EcryptfsInUse: Yes
InstallationDate: Installed on 2012-07-25 (826 days ago)
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release amd64 (20120425)
PackageArchitecture: all
SourcePackage: apport
UpgradeStatus: Upgraded to utopic on 2014-10-24 (5 days ago)

Related branches

Marius Gedminas (mgedmin) wrote :
Marius Gedminas (mgedmin) wrote :

This error goes away if I comment out the code that putting the treeview inside a Gtk.Overlay().

Bug in GTK+?

Marius Gedminas (mgedmin) wrote :

What breaks this is inserting the overlay between the treeview and it's scrolledwindow. Putting the overlay outside makes it work.

I tried to fix this in the code by adding one more level of .getparent() in add_spinner_over_treeview, but simply removing the scrolledwindow and adding it back moves the details to the bottom of the vbox, putting the checkboxes above it. Not nice. So instead of that I moved the Gtk.Overlay() into the .ui file.

(I attempted to also move the spinner to the .ui file but failed miserably: the tree view stopped receiving mouse events as soon as I put a <child type="overlay"> with the spinner. And since moving everything to the .ui made the patch bigger, I gave up and went with the smaller change.)

Martin Pitt (pitti) wrote :

Thank you! Merged into trunk r2872.

Changed in apport (Ubuntu):
status: New → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apport - 2.15-0ubuntu3

---------------
apport (2.15-0ubuntu3) vivid; urgency=medium

  * Merge further test robustification and translation updates from trunk.
 -- Martin Pitt <email address hidden> Tue, 16 Dec 2014 11:01:20 +0100

Changed in apport (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers