Summary: | Import wizard should show what the active target platform is | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] PDE | Reporter: | Dani Megert <daniel_megert> | ||||
Component: | UI | Assignee: | PDE-UI-Inbox <pde-ui-inbox> | ||||
Status: | RESOLVED WORKSFORME | QA Contact: | |||||
Severity: | enhancement | ||||||
Priority: | P3 | CC: | julian.honnen, Lars.Vogel, martin.karpisek, Vikas.Chandra | ||||
Version: | 3.8 | Keywords: | bugday, helpwanted | ||||
Target Milestone: | --- | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
See Also: | https://git.eclipse.org/r/88028 | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Dani Megert
2012-09-12 05:21:05 EDT
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 New Gerrit change created: https://git.eclipse.org/r/88028 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? (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) (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. This is an old bug. Moving out of 4.7 Martin, if you update the patch, you can re-target to 4.7 |