Bug 9615

Summary: yocto-bsp: The design for the yocto-bsp wizard needs some work
Product: [Yocto Project Subprojects] Eclipse Plugin Reporter: Humberto Ibarra <humberto.ibarra.lopez>
Component: eclipse-pluginAssignee: Tim Orling <tim.orling>
Status: RESOLVED WONTFIX QA Contact: Libertad <libertad.gonzalez.de.la.cruz>
Severity: enhancement    
Priority: Medium CC: brian.avery, ioanax.grigoropol, yp.ep.watcher, yp.watcher
Version: 2.2   
Target Milestone: 2.2   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)
Attachments:
Description Flags
Main window
none
Properties window none

Description Humberto Ibarra 2016-05-10 19:03:44 UTC 
The design for the wizard to create a new Yocto BSP could use some work. If it is changed to a style more in line with the rest of the eclipse wizards, it would be much better.

Currently we have 2 windows in the yocto-bsp wizard, a main one and a properties one. Some of the characteristics of the current design are:

1 An empty title bar
2 A lower header that shows "yocto-bsp Main page" for the first window and "yocto-bsp Properties page" for the second one.
3 A description asking to enter the needed fields, pointing to required ones with "*" for the first window and an empty description for the second window.
4 Error and description messages always ending in "!"
5 Required fields marked with "*"
6 Qemu arch field explicitly points that it is only valid when selecting "qemu" as arch, even when the field is disabled in any other selection.

Some comments on how to improve this:

1 Making more in line with eclipse wizards, put "New Yocto BSP" for both windows in the title bar
2 The words "Main" and "page" in the lower headers seem redundant. I think "Yocto BSP" and "Kernel and BSP Properties" would be better.
3 Descriptions might look better as "Create a new Yocto BSP." and "Select correct settings."
4 The "!" seems out of place, as if shouting to the user every time. I think it should be removed.
5 I think no other eclipse wizard use "*" to point out to required fields. They just block the "Next" and "Finish" buttons until those fields are filled, and put messages about it. Since we are already doing this, it seems to me that the "*" are not needed. Please also consider that the properties page has no "*" markers, so this is also an issue of inconsistency.
6 The "Qemu arch" is already disabled when an arch different than "qemu" is selected, not sure why is there a need to point to it again. Simple seems better.


Have no comments for the rest of the properties page, since that one seems simple enough.
The messages I put here are just a suggestion to give an idea of the problem, but maybe a different wording can work better.
Comment 1 Humberto Ibarra 2016-05-10 19:04:22 UTC 
Created attachment 3172 [details]
Main window
Comment 2 Humberto Ibarra 2016-05-10 19:04:42 UTC 
Created attachment 3173 [details]
Properties window
Comment 3 brian avery 2016-09-27 17:39:40 UTC 
The existing Eclipse plugin is supported for build and debug functionality, only. 
Current efforts are going into the new CROPS Eclipse plugin.

That being said, the interface we plan to have between Eclipse and devtool needs a lot of attention and some of these things you mention here will be kept in mind for that.