Bug 9615 - yocto-bsp: The design for the yocto-bsp wizard needs some work
Summary: yocto-bsp: The design for the yocto-bsp wizard needs some work
Status: RESOLVED WONTFIX
Alias: None
Product: Eclipse Plugin
Classification: Yocto Project Subprojects
Component: eclipse-plugin (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 2.2
Assignee: Tim Orling
QA Contact: Libertad
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-05-10 19:03 UTC by Humberto Ibarra
Modified: 2016-09-27 17:39 UTC (History)
4 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments
Main window (47.58 KB, image/png)
2016-05-10 19:04 UTC, Humberto Ibarra
no flags Details
Properties window (37.01 KB, image/png)
2016-05-10 19:04 UTC, Humberto Ibarra
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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.