Bug 2518 - [Navigator] should be able to rename by clicking twice (1GFFEKT)
Summary: [Navigator] should be able to rename by clicking twice (1GFFEKT)
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All Windows 98
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: investigate, usability
: 2250 9605 156530 (view as bug list)
Depends on:
Blocks:
 
Reported: 2001-10-10 22:37 EDT by Greg Adams CLA
Modified: 2017-01-23 04:22 EST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Adams CLA 2001-10-10 22:37:59 EDT
Should work similar to file explorer. Should be able to rename a file by clicking
on its name while it is selected.

NOTES:
Comment 1 DJ Houghton CLA 2001-10-29 18:57:14 EST
PRODUCT VERSION:
125 jre			

Comment 2 Kevin Haaland CLA 2002-02-13 14:31:52 EST
*** Bug 9605 has been marked as a duplicate of this bug. ***
Comment 3 Lynne Kues CLA 2002-03-12 11:38:07 EST
You do not want the rename operation to occur when a double click action occurs, 
so you cannot simply hook the selection event.  I imagine this is the reason 
that there is a slight delay in IE when you attempt to rename an item  by double 
selecting it.
Comment 4 Lynne Kues CLA 2002-03-13 16:40:30 EST
Changed the code to allow a mouse click outside of the in-line editor but on 
the same line as the edited item to end the edit.
Comment 5 Lynne Kues CLA 2002-03-14 15:16:54 EST
Backed out above fix since it is not appropriate for Linux.  

Fixed a bug with ending the inline rename operation --> problems with edit 
widget being destroyed properly and an inconsistent tree state when you click 
on the icon of the item being renamed (which causes a selection event for that 
item).  Used an async in saveChangesAndDispose to fix the problem.
Comment 6 Lynne Kues CLA 2002-03-23 13:04:39 EST
Reassigning to RG since he has started work on this.  

Randy, based on my investigation, this is a tricky problem to get correct.  I 
was looking at using a timer to trigger the rename operation in order to deal 
with the double click vs. single selection of the same item.  

There are problems with the current released implementation.  The 
implementation should work like the explorer does in windows (i.e. only invoke 
the inline rename when a click on the label of an already selected item 
occurs). 

1) Go to resources perspective.  Select a project in the navigator.  Expand the 
project by clicking on the plus icon.  The inline rename operation is invoked 
when it should not be.
2) Go to resources perspective.  Double click on a resource to invoke its 
editor.  Click on the navigator view.  The inline rename operation is invoked 
when it should not be.
3) Invoke the inline rename operation.  Click anywhere outside of the edit 
widget.  This should end the rename operation (by virtue of the edit widget 
losing focus).

There is also a general problem with ending the rename operation.  If you click 
to the right of the item being renamed on the same line, the operation is not 
ended.  It should be.  This has to do with the fact that the edit widget is 
spanning the entire width of the navigator.  

Also note that there are Linux/non-Win issues to consider when doing this.  
Loss of focus works differently on Linux and focus can be customized (e.g., set 
to follow the mouse), so you need to make sure that any Windows solution does 
not break other platform rules.  For example, in Windows when you click on the 
NavigatorView, the edit widget loses focus.  In Linux this is not the case.  I 
tried a fix for the above problem by setting focus to the NavigatorView on 
mouse down (so the edit would end), but this solution is incorrect for Linux.
Comment 7 Lynne Kues CLA 2002-03-23 13:07:11 EST
Released change to RenameResourceAction.  Didn't quite get the fix right for 
the bug noted above under my 2002-03-14 15:16 entry.
Comment 8 Randy Giffen CLA 2002-05-27 21:32:39 EDT
Defer additional work
Comment 9 Randy Giffen CLA 2002-08-08 11:49:28 EDT
Reopen for investigation
Comment 10 Knut Radloff CLA 2002-09-05 17:02:32 EDT
*** Bug 2250 has been marked as a duplicate of this bug. ***
Comment 11 Knut Radloff CLA 2003-09-02 17:01:21 EDT
Reassigning to Nick since he is taking ownership of Navigator
Comment 12 Dani Megert CLA 2008-05-08 03:40:53 EDT
*** Bug 156530 has been marked as a duplicate of this bug. ***
Comment 13 Dani Megert CLA 2009-02-04 05:15:10 EST
Franics, this bug is targeted at the Navigator view and is a general issue for all views i.e. if we change it, other views need to adopt this to get a consistent LAF.
Comment 14 Francis Upton IV CLA 2009-02-04 05:45:55 EST
(In reply to comment #13)
> Franics, this bug is targeted at the Navigator view and is a general issue for
> all views i.e. if we change it, other views need to adopt this to get a
> consistent LAF.
> 
I agree, I might move this to the CommonNavigator.  I just took over ownership of the Navigator, which means I just say that we won't do anything with it, as it's essentially replayed by the CNF.
Comment 15 Dani Megert CLA 2009-02-04 05:49:04 EST
Well, it is not just replaced by CNF. If we change this all other views that allow renaming need to be adjusted as well, hence it is a global Platform UI thing.
Comment 16 Francis Upton IV CLA 2009-02-04 05:52:40 EST
(In reply to comment #15)
> Well, it is not just replaced by CNF. If we change this all other views that
> allow renaming need to be adjusted as well, hence it is a global Platform UI
> thing.
> 
Oh yes, I completely agree.  I was not intending to contradict your statement earlier.
Comment 17 Mickael Istria CLA 2016-11-10 10:36:12 EST
The natural expectation for double-click is Open. There are shortcuts and easy context-menu for Rename. IF we change double-click all of a sudden to rename, it would be a major usability loss as people want to open files way more often than they want to remain them. Double-click as to remain attached to most important operation.
Comment 18 Dani Megert CLA 2017-01-23 04:22:17 EST
(In reply to Mickael Istria from comment #17)
> The natural expectation for double-click is Open. There are shortcuts and
> easy context-menu for Rename. IF we change double-click all of a sudden to
> rename, it would be a major usability loss as people want to open files way
> more often than they want to remain them. Double-click as to remain attached
> to most important operation.

+1.