ISISComputingGroup / IBEX

Top level repository for IBEX stories
5 stars 2 forks source link

GUI: Eclipse-4/CSS-4 Prototype [TIMEBOX: 10d] #2376

Closed kjwoodsISIS closed 7 years ago

kjwoodsISIS commented 7 years ago

As a developer, I want to create a prototype IBEX GUI using Eclipse-4/CSS-4, so that I understand what it takes to build an Eclipse-4/CSS-4 application.

Acceptance Criteria

  1. A prototype of the IBEX GUI built using Eclipse-4/CSS-4.
    1. Prototype should attempt to replicate the main features of the current IBEX GUI in Eclipse-4/CSS-4 environment.
  2. Written documentation on how to build a fully-fledged IBEX GUI using Eclipse-4/CSS-4.

Notes

  1. Consult with Diamond to learn about their experience of the process.
  2. Prototype is intended to help IBEX developers understand the most effective way to build Eclipse-4/CSS-4 applications.
  3. Documentation should:
    1. Set out architectural principles that IBEX GUI developers need to understand
    2. Explain how to "go with the E4/CSS4 flow" (i.e. work with E4/CSS4, rather than against it).
    3. Recommend strategies for migrating the current IBEX GUI to E4/CSS4.
      1. strategies might include using the E3/E4 compatibility layer
  4. Prototype may be used to help scientists understand what a new IBEX GUI might look like and to solicit feedback.
AdrianPotter commented 7 years ago

https://github.com/ISISComputingGroup/ibex_developers_manual/wiki/Eclipse-4-migration-notes

kjwoodsISIS commented 7 years ago

I have reviewed the above notes. All looks good to me.

AdrianPotter commented 7 years ago

For code review:

Make sure that you can:

Have a look at the main places the code has changed and review:

Note that this is a prototype and so you aren't reviewing the aesthetics or trying to incite weird behaviour by manipulating the view parts. The exception is that I expect the dashboard and beam status view parts to behave sensibly when they are resized.

mattclarke commented 7 years ago

The first time you open this in eclipse you need to set the target platform.