Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 330804 - Change the default selection of Save Conflict dialog
Summary: Change the default selection of Save Conflict dialog
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3 M4   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-22 09:48 EST by Samuel Wu CLA
Modified: 2010-12-14 07:23 EST (History)
1 user (show)

See Also:


Attachments
patch to make save as the default (3.79 KB, patch)
2010-11-30 12:19 EST, David McKnight CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Samuel Wu CLA 2010-11-22 09:48:58 EST
Build Identifier: RSE 3.2.1

The current defaut selection of the Save Conflict dialog is Replace contents of editor with remote file which causes the pending change made in editor lost. The default selection should be Save the contents of the editor to a different remote file which allows the user to keep the change made.

Reproducible: Always

Steps to Reproduce:
1. Open a file in a editor through RSE system view
2. Open the same file through another editor outside Eclipse and make a change to it.
3. Go back to eclipse editor and make another change
4. When saving the file, you will get a Save Conflict dialog 
5. The radio button Replace contents of editor with remote file is selected by default.
6. Some of the users hit the Enter button too soon and lose the changes made in the editor.
Since content conflict is a situation which needs more attention, the default selection of the dialog should be Save the contents of the editor to a different remote file and the OK button should be disabled. In this way the user can save the local change,compare it with the remote one and merge the change.
Comment 1 David McKnight CLA 2010-11-30 12:19:49 EST
Created attachment 184151 [details]
patch to make save as the default
Comment 2 David McKnight CLA 2010-11-30 12:22:56 EST
I've committed the change to cvs.