-
Notifications
You must be signed in to change notification settings - Fork 4
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
expanded on open recent functionality #554
base: master
Are you sure you want to change the base?
Conversation
Quality Gate passedIssues Measures |
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.
Add descriptions for the tests
for recentProject in _deque(reversed(RecentProjectsHandler.recentProjects)): | ||
_qtw.QListWidgetItem(str(recentProject), self.listWidget) | ||
|
||
def clickButtonHandler(self, clickedItem): |
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.
try dictionary
from trnsysGUI.common import cancelled as _ccl | ||
from trnsysGUI.diagram import Editor as _de | ||
from trnsysGUI.storageTank.widget import StorageTank | ||
from trnsysGUI.constants import UNSAVED_PROGRESS_LOST |
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.
do constants.UNSAVED_PROGRESS_LOST
@@ -245,25 +252,18 @@ def shutdown(self) -> None: | |||
self.editor.shutdown() | |||
|
|||
def newDia(self): |
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.
make more descriptive
if recentProject != currentProject and recentProject.exists(): | ||
recentProjectAction = _qtw.QAction(str(recentProject), self) | ||
self.recentProjectsMenu.addAction(recentProjectAction) | ||
print("Help me: ", RecentProjectsHandler.recentProjects) |
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.
remove
e.accept() | ||
elif ret == _qtw.QMessageBox.Save: | ||
return | ||
if ret == _qtw.QMessageBox.Save: | ||
self.editor.save() |
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.
Is it reasonable to change this to saveProject?
@@ -599,16 +591,14 @@ def closeEvent(self, e): | |||
ret = qmb.exec() |
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.
Change to messagebox please.
|
||
|
||
class RecentProjectsHandler: | ||
recentProjects = list[_pl.Path] |
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.
typing of Deque
No description provided.