[PR #13467] Improve Launch MVVM #29565

Closed
opened 2026-01-31 09:35:39 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/13467

State: closed
Merged: Yes


Summary of the Pull Request

The xaml file no longer directly accesses the settings model object, and the settings model object is no longer exposed on the view model

References

#13377

PR Checklist

  • Closes #xxx
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated.
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed

Still works

**Original Pull Request:** https://github.com/microsoft/terminal/pull/13467 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request The xaml file no longer directly accesses the settings model object, and the settings model object is no longer exposed on the view model ## References #13377 ## PR Checklist * [ ] Closes #xxx * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx * [ ] Schema updated. * [x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx ## Validation Steps Performed Still works
claunia added the pull-request label 2026-01-31 09:35:39 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29565