Adds validation state feedback to nodes

[tatatupi]

The idea is to have a NodeValidationState member object in the NodeDelegateModel class, that defaults to a valid and empty state message. The API allows for updates to be made accordingly, and the UI will respond automatically.

UI will show a node red (error) or yellow (warning). A tooltip is added with the state message. And a small icon for information is displayed as a badge, to help the user access the tooltip. I added a screenshot of the calculator example to help discussions. in the example, the tooltip for the Warning state is displayed.

Key Changes

• Introduces NodeValidationState struct class.
• Getter and setter in NodeDelegateModel for validation.
• DataFlowGraphModel handles NodeRole::ValidationState updates.
• Painting logic reflects invalid states in DefaultNodePainter.
• Tooltip shows validation message in NodeGraphicsObject.
• Calculator example demonstrates usage.

Solves #226
Solves #75

P.S.: some changes happened automatically when saving files, due to autoformatting using nodeeditor clang format style.

[tatatupi]

I also changed the linux-gcc qt version to 6, so that QtGui/QUndoCommand include could work

[tatatupi]

I will change the coloring scheme to color only the node border.

[lonnie-souder-ii]

I also changed the linux-gcc qt version to 6, so that QtGui/QUndoCommand include could work

You shouldn't need to change the Qt version for this. You can support both by changing the include to just #include <QUndoCommand> like I've done in this PR https://github.com/paceholder/nodeeditor/pull/476/files

Cool feature by the way!

[lonnie-souder-ii]

Everything appears to be building and working for me.

Just that one note above about the QtGui/X.h includes. Doesn't look like you've added any of those includes here, so this should build with or without the version change. If anything I'd recommend adding both major Qt versions back to the build matrix, but not strictly necessary.

[tatatupi]

Thanks for the inputs, @Llcoolsouder. I have made the changes you requested.
I also changed the UI, as I mentioned in a conversation I had with @paceholder. It now looks like this:

[paceholder]

This would eat CPU resources (maybe very insignificantly) for converting the icon in each rendering pass.
Maybe we could have used QGraphicsSvgItem that has caching and it also could render the icon smoothly when zoomed-in. What do you think?

[paceholder]

Maybe this one is not needed in the header file, as we instantiate the icon one more time in the rendering function in *.cpp. Or vice versa

[paceholder]

Probably we shouldn't check if we can convert it everywhere.
If we call model.nodeData<NodeValidationState>(...) it will return the converted value from QVariant or the defaul-constructed value for this type. Which is "valid + empty message" for NodeValidationState.