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 83552 - mouse clicking in the list view shouldn't initiate rename
mouse clicking in the list view shouldn't initiate rename
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Views: List View
0.x.x [obsolete]
Other other
: High major
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 82575 88650 92019 99988 100562 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-05-30 15:26 UTC by Dave Bordoley [Not Reading Bug Mail]
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (529 bytes, patch)
2002-07-03 18:21 UTC, Andrew Sobala
none Details | Review
here is a more up to date patch that does the same thing, just against nautilus-2.0.4 (529 bytes, patch)
2002-08-15 05:58 UTC, Alex Duggan
none Details | Review
patch that fixes this bug but maintains hotkey and rename-from-context-menu functionality (1.87 KB, patch)
2003-01-11 13:21 UTC, marten ter borgh
none Details | Review
Updated patch against current cvs. (2.30 KB, patch)
2003-01-17 19:38 UTC, marten ter borgh
none Details | Review

Description Dave Bordoley [Not Reading Bug Mail] 2002-05-30 15:26:00 UTC
double right clicking in the list view shouldn't initiate rename.

reasons:
1. It is too easy to accidentally initiate a rename when simply trying to
open or launch an item. (This happens here and there for me still)

2. Renaming is a somewhat dangerous operation. We shouldn't make it too
easy, especially with the lack of an undo feature.

3. Creates a behavioral difference between single and double click modes.
Obviously in single click mode this feature will never work.

4. Is inconsistent with the behavior of the icon view.

We have a rename option in the context menu. Making the user explicitly
choose to rename is a much better ui, and prevents them from accidently
renaming files.
Comment 1 Dave Bordoley [Not Reading Bug Mail] 2002-06-06 15:39:44 UTC
I mean left clicking, but this should apply to all clicks...
Comment 2 Dave Bordoley [Not Reading Bug Mail] 2002-06-11 12:43:22 UTC
*** Bug 82575 has been marked as a duplicate of this bug. ***
Comment 3 Andrew Sobala 2002-07-03 18:21:54 UTC
Created attachment 9618 [details] [review]
Proposed patch
Comment 4 Dave Bordoley [Not Reading Bug Mail] 2002-07-19 14:03:10 UTC
andrew did you email this patch to the nautilus list???
Comment 5 Dave Bordoley [Not Reading Bug Mail] 2002-07-22 10:09:55 UTC
*** Bug 88650 has been marked as a duplicate of this bug. ***
Comment 6 Dave Bordoley [Not Reading Bug Mail] 2002-07-22 10:11:14 UTC
Usability, anyone disagree with this???
Comment 7 Calum Benson 2002-07-24 18:39:17 UTC
Fine by me, especially if the F2 shortcut now works properly for
desktop icons as well.
Comment 8 Dave Bordoley [Not Reading Bug Mail] 2002-07-27 17:32:34 UTC
*** Bug 89214 has been marked as a duplicate of this bug. ***
Comment 9 Alex Duggan 2002-08-15 05:58:14 UTC
Created attachment 10497 [details] [review]
here is a more up to date patch that does the same thing, just against nautilus-2.0.4
Comment 10 Alex Duggan 2002-08-15 06:11:15 UTC
after further review of the patch, this also seems to disable the F2
function that makes a filename editable.  After applying the patch and
building, double clicking on a file in list view does not make the
filename editable, nor does pressing F2.
Comment 11 Alex Duggan 2002-09-02 17:44:36 UTC
The above patch is NOT the correct fix.  Applying this patch disables
the "F2" and Right Click->Rename functionality.  The appropriate fix
is to remove code that causes a double click to initiate and editable
cell.
Comment 12 Mark Finlay 2002-09-14 19:18:20 UTC
*** Bug 92019 has been marked as a duplicate of this bug. ***
Comment 13 Mattias Dahlberg 2002-10-04 18:50:02 UTC
Whether or not you should be able to rename the file by single 
clicking is one thing, but the current behaviour when renaming is 
activated when you double-click on a filename in the list view is a 
real show-stopper.
Comment 14 Dave Bordoley [Not Reading Bug Mail] 2002-11-30 17:35:22 UTC
*** Bug 99988 has been marked as a duplicate of this bug. ***
Comment 15 Alex Duggan 2002-12-07 01:31:37 UTC
*** Bug 100562 has been marked as a duplicate of this bug. ***
Comment 16 marten ter borgh 2003-01-11 13:21:16 UTC
Created attachment 13492 [details] [review]
patch that fixes this bug but maintains hotkey and rename-from-context-menu functionality
Comment 17 Mark Finlay 2003-01-11 21:52:54 UTC
This problem is totally crippling the tree view in double click mode:
* double clicking an already selected item renames instead of running it
* double clicking an unselected item after bug 91176 is fixed causes the
file to be opened and then renamed.

As well as all the reasons above as to why it shouldn't be there in
the first place. This really needs to go in for 2.2. Setting 
TARGET2.2.0 and GNOMEVER2.1 keywords.
Comment 18 marten ter borgh 2003-01-11 23:29:30 UTC
I tested the patch for 91176 together with my patch for this bug.
With the patches, the problems Mark describes in the previous comment
are both fixed.
Comment 19 Mark Finlay 2003-01-12 12:10:20 UTC
I can confirm that these two patches applied together *unbreak*
the list view in double click mode.

I'm upping the priority of this bug because it has a patch
and because rename is causing so much breakage and usibility
woes.
Comment 20 Mattias Dahlberg 2003-01-15 16:57:02 UTC
After reading this and the discussions in nautilis-list, could we 
say that consensus is that the current implementation has problems 
and better be disabled for GNOME 2.2? This is not to say that 
renaming by single-clicking should be gone forever, but it's easier 
to disable it for now and look at other ways of implementating it 
after 2.2. For example, *double-clicking* should never initiate a 
rename, which happens with the current code if the item is selected.

So, who applies the patch? Alex?
Comment 21 marten ter borgh 2003-01-17 19:38:54 UTC
Created attachment 13654 [details] [review]
Updated patch against current cvs.
Comment 22 marten ter borgh 2003-01-17 19:43:33 UTC
I updated my patch to apply to the current CVS. I also made a small
change to make the code look better (thanks, Dave).
Comment 23 Dave Camp 2003-01-18 21:18:25 UTC
I applied this patch, thanks.
Comment 24 Mattias Dahlberg 2003-02-06 16:09:54 UTC
Could changing this have caused bug 104334?