Bug 116123 - launcher should allow browsing to the default configuration area
Summary: launcher should allow browsing to the default configuration area
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.4 M5   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2005-11-12 21:00 EST by Erkki Lindpere CLA
Modified: 2008-02-05 13:55 EST (History)
3 users (show)

See Also:
curtis.windatt.public: review? (caniszczyk)

Attachments
Fix (1.51 KB, patch)
2007-12-17 12:30 EST, Curtis Windatt CLA 		no flags Details | Diff
mylyn/context/zip (1.01 KB, application/octet-stream)
2008-01-02 12:21 EST, Chris Aniszczyk CLA 		no flags Details
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 Erkki Lindpere CLA 2005-11-12 21:00:08 EST 
Occasionally I want to browse the files in the default configuration area
created by a launcher. For example to copy the config.ini to a "real"
configuration area that I set up to launch a product manually from the command line.

However, doing it is quite annoying because the path to the area is only
displayed when the input field in the launcher is disabled so it can't even be
copied. Ideally I would like to see a button there that would open the folder in
the current platform's file browser, but being able to copy the path (preferably
absolute) would be enough.
Comment 1 Erkki Lindpere CLA 2005-11-12 21:02:00 EST 
Forgot to mention: this applies both to Equinox and Eclipse launchers.
Comment 2 Curtis Windatt CLA 2007-12-17 12:30:38 EST 
Created attachment 85397 [details]
Fix

Simple fix that sets the text to always be enabled, but only editable if a custom location is being chosen.  This allows the user to select the text and copy it.
Comment 3 Curtis Windatt CLA 2007-12-17 12:34:20 EST 
Brian has a ton of my other bugs to review.  So maybe Chris wants to look at it instead? :)
Comment 4 Curtis Windatt CLA 2007-12-17 12:35:39 EST 
Strange, I don't see Chris on the e-mail list, even if I add him as a reviewer.  cc'ing him just in case.
Comment 5 Chris Aniszczyk CLA 2008-01-02 12:21:49 EST 
simple and elegant Curtis, thank you.
Comment 6 Chris Aniszczyk CLA 2008-01-02 12:21:52 EST 
Created attachment 85998 [details]
mylyn/context/zip
Comment 7 Brian Bauman CLA 2008-02-05 13:55:24 EST 
Verified on I20080204-0800.  I made the same modfication on ConfigurationTemplateBlock to have both sections behave the same.  Thanks for the elegant patch Curtis!