Bug 389385 - Import wizard should show what the active target platform is
Summary: Import wizard should show what the active target platform is
Status: RESOLVED WORKSFORME
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.8   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: PDE-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted
Depends on:
Blocks:
 
Reported: 2012-09-12 05:21 EDT by Dani Megert CLA
Modified: 2018-12-13 09:07 EST (History)
4 users (show)

See Also:

Attachments
How import wizard would look like with applied patchset1 (43.01 KB, image/png)
2017-01-04 12:10 EST, Martin Karpisek 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 Dani Megert CLA 2012-09-12 05:21:05 EDT 
3.8 / 4.2.

When importing I selected 'The active target platform' but had not clue what it is.

It would be nice to show the name in the label:

(x) The active target platform: '<name goes here>'
e.g.
(x) The active target platform: 'Running Platform'
Comment 1 Martin Karpisek CLA 2017-01-04 12:10:02 EST 
Created attachment 266120 [details]
How import wizard would look like with applied patchset1

following patch propose slightly modified fix from comment 1 (same way, but uses separate label control for target platform name)
(reason why: having long label of radio button was clashing with grid layout for longer names & resizing dialog)  

Following proposed patch uses separate label for name of target platform. 
in addition proposes to simplify link to target configuraton preference page (reduce amount of text)

see screenshot on how it would look like

patch is following
Comment 2 Eclipse Genie CLA 2017-01-04 12:17:21 EST 
New Gerrit change created: https://git.eclipse.org/r/88028
Comment 3 Vikas Chandra CLA 2017-01-05 01:08:07 EST 
The active platform should be within single quotes.

I am not too sure about the text change.

Dani, will you have the time to have a look at this fix?
Comment 4 Dani Megert CLA 2017-01-05 11:34:07 EST 
(In reply to Vikas Chandra from comment #3)
> The active platform should be within single quotes.
> 
> I am not too sure about the text change.
> 
> Dani, will you have the time to have a look at this fix?

Can you attach a screenshot on how it would look like?


This bug is quite old. By now, the Directory field actually shows the active target platform path. So, maybe we could simply change the label to:

The active target platform (see directory below)
Comment 5 Dani Megert CLA 2017-01-05 11:35:55 EST 
(In reply to Dani Megert from comment #4)
> Can you attach a screenshot on how it would look like?

Sorry, just saw it above. Definitely a no go.
Comment 6 Vikas Chandra CLA 2017-03-01 08:09:03 EST 
This is an old bug.  Moving out of 4.7


Martin, if you update the patch, you can re-target to 4.7
Comment 7 Dani Megert CLA 2018-12-13 09:07:13 EST 
The current solution (see comment 4) works for me.