From the C++ coding standard: "4. Functions and variables should have descriptive and meaningful names or well-commented declarations; variables with a small scope can have short names."
In HD, AlarmDefs.h is always included via common.h which includes AlarmMgmt.h which includes AlarmDefs.h. DG should be same way so you shouldn't need to include it here.
Preferred architecture is to just set flag here and then let your state machine see the flag and change the state and command the fpga when it next executes.
Please document why this file is created. And Please move it to its appropriate folder "./sources/gui/qml/components". And also this component seems unnecessary since it's so small and looks like being used only for one purpose, so please move it in the main component it has been used for.
If this class is the Treatment Parameters Request, please refactor the class and file names to TreatmentParametersRequest to be more clear since this is the first class which has its own separate request class.
Please add the same messaging documentation for your new message container models should be like below (only for your new message, the others are already added):
/*!
* \brief The MAdjustDurationReq class
* \details The treatment duration change request model
*
* | MSG | CAN ID | Box | Type | Ack | Src | Dst | Description |
* |:----:|:------:|:---:|:------:|:---:|:---:|:---:|:-----------: |
* |0x1600| 0x100 | 9 | Req | Y | UI | HD | Treatment Duration Change Request |
*
* | Payload ||
* | ||
* | #1:(U32) | \ref duration |
*
*/
Why does it even need styling? Couldn't we use the keyboard default styling? Please provide two of the default and styled screenshot versions of the keyboard for comparison.
