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

Bug #1387328 reported by Marius Gedminas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apport (Ubuntu)
Fix Released
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

Revision history for this message
Marius Gedminas (mgedmin) wrote :
Revision history for this message
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+?

Revision history for this message
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.)

Revision history for this message
Martin Pitt (pitti) wrote :

Thank you! Merged into trunk r2872.

Changed in apport (Ubuntu):
status: New → Fix Committed
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.