Skip to content

feat(developer): New Project opens in new process 🦕#10155

Merged
mcdurdin merged 1 commit intomasterfrom
feat/developer/10138-new-project-new-process
Dec 12, 2023
Merged

feat(developer): New Project opens in new process 🦕#10155
mcdurdin merged 1 commit intomasterfrom
feat/developer/10138-new-project-new-process

Conversation

@mcdurdin
Copy link
Copy Markdown
Member

@mcdurdin mcdurdin commented Dec 6, 2023

Fixes #10138.

If the current instance of Keyman Developer already has a project open, then the New Project dialog will now open the new project in a new instance of Keyman Developer.

Note: this also moves a couple of functions out of dmActionsMain and into UfrmMain, as their functionality belongs more closely there. The TmodActionsMain.OpenProject function has been renamed to TfrmMain.OpenProjectInCurrentProcess to clarify its usage and context.

User Testing

  • TEST_NEW_PROJECT: Open Keyman Developer, and open a project. Create a new project, and verify that the new project opens in a new instance of Keyman Developer.

  • TEST_NEW_PROJECT_NO_PROJECT_OPEN: Open Keyman Developer, and make sure no project is open (Project|Close Project). Create a new project, and verify that the project opens in the current instance of Keyman Developer.

Fixes #10138.

If the current instance of Keyman Developer already has a project open,
then the New Project dialog will now open the new project in a new
instance of Keyman Developer.

Note: this also moves a couple of functions out of dmActionsMain and
into UfrmMain, as their functionality belongs more closely there. The
`TmodActionsMain.OpenProject` function has been renamed to
`TfrmMain.OpenProjectInCurrentProcess` to clarify its usage and context.
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner December 6, 2023 05:13
@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Dec 6, 2023
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Dec 6, 2023

User Test Results

Test specification and instructions

  • TEST_NEW_PROJECT (PASSED): Tested with the attached PR build (Keyman 17.0.225-alpha-test-10155) on Windows 11 OS (VM) and here is my observation: 1. Opened a project in Keyman Developer. 2. Created a new project using the Project / New Project option. 3. Verified that the new project opens in a separate window.
  • TEST_NEW_PROJECT_NO_PROJECT_OPEN (PASSED): 1. Closed an existing project in Keyman Developer. 2. Created a new project using Project / New Project option. 3. Verified that the new project opens in the current instance of Keyman Developer.

Test Artifacts

Copy link
Copy Markdown
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcdurdin mcdurdin changed the title feat(developer): New Project opens in new process feat(developer): New Project opens in new process 🦕 Dec 6, 2023
@bharanidharanj
Copy link
Copy Markdown

Test Results

  • TEST_NEW_PROJECT (PASSED): Tested with the attached PR build (Keyman 17.0.225-alpha-test-10155) on Windows 11 OS (VM) and here is my observation: 1. Opened a project in Keyman Developer. 2. Created a new project using the Project / New Project option. 3. Verified that the new project opens in a separate window.

  • TEST_NEW_PROJECT_NO_PROJECT_OPEN (PASSED): 1. Closed an existing project in Keyman Developer. 2. Created a new project using Project / New Project option. 3. Verified that the new project opens in the current instance of Keyman Developer.

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Dec 6, 2023
@mcdurdin mcdurdin modified the milestones: A17S27, A17S28 Dec 8, 2023
Base automatically changed from feat/developer/10149-window-menu to master December 12, 2023 21:45
@mcdurdin mcdurdin merged commit 1b45c6b into master Dec 12, 2023
@mcdurdin mcdurdin deleted the feat/developer/10138-new-project-new-process branch December 12, 2023 21:45
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.229-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(developer): New Project should present dialog and then open a new process

4 participants