Bug 50914 - Apply Patch dialog doesn't initialize file browse dialog
Summary: Apply Patch dialog doesn't initialize file browse dialog
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Compare (show other bugs)
Version: 3.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.0 M9   Edit
Assignee: Platform-Compare-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-01-29 16:40 EST by Jared Burns CLA
Modified: 2004-04-23 10:31 EDT (History)
0 users

See Also:

Attachments
50914.patch (1.58 KB, patch)
2004-04-21 17:08 EDT, Jared Burns CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jared Burns CLA 2004-01-29 16:40:19 EST 
Build 20040129

1. The first time you bring up the Apply Patch dialog, the patches file location field initializes to 
empty.
2. The first time you press the Browse button, it opens a (native) file selection dialog initialized to 
the Eclipse install directory.
3. After applying a patch, subsequent invocations of the Apply Patch dialog will initialize the file 
location field to the location of the last patch.
4. Pressing the Browse button, however, still initializes to the Eclipse install directory.

In step 2, the file selection dialog should be initialized to the user's home directory instead of the 
Eclipse install directory. User's generally don't (and shouldn't) store patch files in that location.
In step 4, the file selection dialog should be initialized to the directory of the patch that's shown in 
the text field.
Comment 1 Jared Burns CLA 2004-04-21 17:08:23 EDT 
Created attachment 9806 [details]
50914.patch

Attached is a patch that fixes this problem. I use patches very often and this
was just irritating me silly. :-)
Comment 2 Andre Weinand CLA 2004-04-23 10:31:08 EDT 
Jared, thanks a lot for the patch!
Will be in today's N-build.