Rename select_document() function to new_window()

Rename select_document() to new_window() to better encapsulate the fact
that this function is opening a new Dangerzone window.
This commit is contained in:
deeplow 2022-10-06 09:46:51 +01:00
parent e487b7f0a9
commit e8b56627c9
No known key found for this signature in database
GPG key ID: 577982871529A52A

View file

@ -86,7 +86,7 @@ def gui_main(filename: Optional[str]) -> bool:
del windows[window_id] del windows[window_id]
# Open a document in a window # Open a document in a window
def select_document(filename: Optional[str] = None) -> bool: def new_window(filename: Optional[str] = None) -> bool:
document = Document(filename) document = Document(filename)
window_id = uuid.uuid4().hex window_id = uuid.uuid4().hex
window = MainWindow(global_common, gui_common, window_id) window = MainWindow(global_common, gui_common, window_id)
@ -111,20 +111,20 @@ def gui_main(filename: Optional[str]) -> bool:
# Open a new window if not filename is passed # Open a new window if not filename is passed
if filename is None: if filename is None:
select_document() new_window()
else: else:
# If filename is passed as an argument, open it # If filename is passed as an argument, open it
if not select_document(filename): if not new_window(filename):
return True return True
# Open a new window, if all windows are closed # Open a new window, if all windows are closed
def application_activated() -> None: def application_activated() -> None:
if len(windows) == 0: if len(windows) == 0:
select_document() new_window()
# If we get a file open event, open it # If we get a file open event, open it
app_wrapper.document_selected.connect(select_document) app_wrapper.document_selected.connect(new_window)
app_wrapper.new_window.connect(select_document) app_wrapper.new_window.connect(new_window)
# If the application is activated and all windows are closed, open a new one # If the application is activated and all windows are closed, open a new one
app_wrapper.application_activated.connect(application_activated) app_wrapper.application_activated.connect(application_activated)