-
Notifications
You must be signed in to change notification settings - Fork 35
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
Remove the Python text box at the right-hand side #146
Conversation
Change all applications to set the Python command line instead.
@@ -9,7 +9,6 @@ set(MODMESH_VIEW_HEADERS | |||
${CMAKE_CURRENT_SOURCE_DIR}/RAxisMark.hpp | |||
${CMAKE_CURRENT_SOURCE_DIR}/RMainWindow.hpp | |||
${CMAKE_CURRENT_SOURCE_DIR}/RPythonConsoleDockWidget.hpp | |||
${CMAKE_CURRENT_SOURCE_DIR}/RPythonText.hpp |
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.
Bye bye, RPythonText
.
m_pycon = new RPythonConsoleDockWidget(QString("Console"), this); | ||
m_pycon->setAllowedAreas(Qt::BottomDockWidgetArea | Qt::TopDockWidgetArea); | ||
m_pycon->setAllowedAreas(Qt::AllDockWidgetAreas); |
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.
Allow narrow command line dock on left and right.
"command", | ||
[](wrapped_type const & self) | ||
{ | ||
return self.command().toStdString(); |
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.
Later when we have sufficiently many such wrappers, we should add a caster for QString
.
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.
maybe open an issue, otherwise it's easy to forget lol
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.
Sounds like a good idea. After filing one could you tag it here?
# Need to hold the win object to keep PySide alive. | ||
win, svr = mm.app.bad_euler1d.run(animate=True, interval=10) | ||
""".lstrip() | ||
cmd = "win, svr = mm.app.bad_euler1d.run(animate=True, interval=10)" |
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.
pycon
takes a one liner. It does not yet have a check in the engine so that we can fine-tune the implementation.
@tigercosmos I am merging and would like you have your comments. |
Change all applications to set the Python command line instead.
Ref #104