Please change Btn to Button and re-align the ':'. In general please don't use these kinds of simplifications unless it's too obvious like Rectangle->rect.
Later please use more familiar terms like "Option Button". The layout "grid of options" is feature of "Option Button" not the component itself. RESOLVED
Absolutely, I'm doing it incrementally and slowly to not to make the code reviews harder and confusing git. I did for so many and this file will get its turn soon.
Absolutely, I'm doing it incrementally and slowly to not to make the code reviews harder and confusing git. I did for so many and this file will get its turn soon.
Absolutely, I'm doing it incrementally and slowly to not to make the code reviews harder and confusing git. I did for so many and this file will get its turn soon.
Description added. I am not familiar with Sphinx and that's not the documentation we should use for UI Software. The comments are in the Markup language and Sphinx, like all the other documentation tools, must have a way to understand it.
Rather than use the if statement to assign variable energized to TRUE, i recommend to assign the if condition directly to the variable energized. So the whole if statement can be replaced by one line - BOOL const energized = (STATE_OPEN == state). Both has the same result, but the other way has less code.
I've updated it to use an attached scrollbar. Just FYI, for some reason the attached scrollbar doesn't hide automatically when the content height is less than the height of the screen. Will post my update soon
Why another class has been created for saving files. What is the advantage of this class other than inherited from Q_Object and can have signal/slot. It could be a simple single function in another class inherited from Q_Object.
I set the result variable to TRUE by default so if any of these cases are not true, the variable is returned as TRUE. So there is implicitly an else statement.
