-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Allow setting of temperature through button & update docs #1855
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry - sent you a message on Teams expressing how bad I got this wrong. Made way more work for you than it should have been!
I just tried the signal connection, and for me adding this in the ConfigurationTab
constructor worked just fine, and lets you simplify your PR quite a bit:
connect(ui_.TemperatureToolButton, SIGNAL(clicked(bool)), dissolveWindow_, SLOT(on_ConfigurationAdjustTemperatureAction_triggered(bool)));
@trisyoungs Please double-check the changes to the Sillica example, I am not entirely sure if the node is appropriate for this example. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need a change to the silica example. Did you see my suggestion for connecting the signal as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Removes depreciated
TemperatureProcedureNode
and instead sets temperature through a toolbutton in the configuration tab.