After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 154709 - don't use the cursor for preview
don't use the cursor for preview
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkFileChooser
unspecified
Other Linux
: High normal
: Small fix
Assigned To: Federico Mena Quintero
Federico Mena Quintero
filechooser-harder-fix
: 540760 738735 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-10-06 18:07 UTC by Matthias Clasen
Modified: 2014-10-19 21:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Matthias Clasen 2004-10-06 18:07:07 UTC 
The file chooser shouldn't use the treeview cursor for updating the preview,
especially not in a selection changed handler. There are a couple of bugs caused
by the treeview updating the cursor position and the selection in different orders.
Comment 1 Michael Natterer 2005-05-18 23:22:38 UTC 
If it used the selection instead of the cursor, previews won't work
with multiple selections.
Comment 2 Federico Mena Quintero 2005-05-18 23:23:47 UTC 
See bug #132255 - this is why things got changed to use the cursor rather than
just the selection.  It does refer to multiple selection.
Comment 3 Federico Mena Quintero 2005-05-18 23:32:41 UTC 
What about this?

In single selection mode, just use the selected row.

In multiple selection mode,

  - If there is one and only one row selected, use that.
  - If there is more than one row selected and the cursor is on a selected row,
use that.
  - If the cursor is not on a selected row, don't display a preview.
Comment 4 Matthias Clasen 2005-10-28 18:35:14 UTC 
See the related bug 143825 and bug 319525
Comment 5 Michael Natterer 2008-06-30 08:53:10 UTC 
*** Bug 540760 has been marked as a duplicate of this bug. ***
Comment 6 Matthias Clasen 2014-10-19 18:35:24 UTC 
*** Bug 738735 has been marked as a duplicate of this bug. ***