Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364207 - RelativePathBrowseActionHandler should set initial selection if the path based property contains a valid path
Summary: RelativePathBrowseActionHandler should set initial selection if the path base...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sapphire (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Konstantin Komissarchik CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-18 17:50 EST by Shenxue Zhou CLA
Modified: 2021-11-19 09:21 EST (History)
1 user (show)

See Also:

Attachments
Set the initial selection for the dlg (2.64 KB, patch)
2011-11-18 18:00 EST, Shenxue Zhou 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 Shenxue Zhou CLA 2011-11-18 17:50:49 EST 
When a path based property has a @ProjectRelativePath or @WorkspaceRelativePath annotation, Sapphire UI renders a "browse" btn on its right. When the "browse" btn is clicked, RelativePathBrowseActionHandler is invoked and a relative path selection dialog is launched. However, the path dialog doesn't select the path as its initial selection. When user clicks the browse btn repeatedly, he/she has to start over.
Comment 1 Shenxue Zhou CLA 2011-11-18 18:00:36 EST 
Created attachment 207257 [details]
Set the initial selection for the dlg
Comment 2 Shenxue Zhou CLA 2011-11-18 18:02:50 EST 
Konstantin, could you take a quick look at the patch? That class is very complicated and I want to make sure I'm not missing things. Thanks!
Comment 3 Konstantin Komissarchik CLA 2011-11-21 18:47:11 EST 
No. The entire fix needs quite a bit more than what's shown in the patch.

Issue fixed. Please verify.
Comment 4 Shenxue Zhou CLA 2011-11-21 19:05:11 EST 
I've verified the fix. Closing.