[PR #2654] [MERGED] add a spec for initial position and launch mode #25014

Open
opened 2026-01-31 09:06:43 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/2654
Author: @KaiyuWang16
Created: 9/4/2019
Status: Merged
Merged: 1/7/2020
Merged by: @DHowett-MSFT

Base: masterHead: dev/kawa/1043-set-initial-position-for-the-terminal


📝 Commits (10+)

  • 9e85571 Spec version 1 for set terminal initial position
  • dbbba4e Fix the spec title
  • 5a919c7 Add a new function to read initial position properties
  • 5e8dc81 Merge branch 'master' into dev/kawa/1043-set-initial-position-for-the-terminal
  • 5e947ac Test version - 9-6
  • 8fbe307 spec for code review updates
  • fa5f5c7 Add edge cases - window hangs off the screen
  • 3eac231 Merge branch 'dev/kawa/1043-set-initial-position-for-the-terminal' of https://github.com/microsoft/terminal into dev/kawa/1043-set-initial-position-for-the-terminal
  • 33d37ca remove window partialy hangs off edge case
  • d4494a5 Remove unnecessary code changes and update spec for edge cases

📊 Changes

1 file changed (+100 additions, -0 deletions)

View changed files

doc/specs/#1043 - Set the initial position of the Terminal/spec.md (+100 -0)

📄 Description

This PR is for the GitHub task: https://github.com/microsoft/terminal/issues/1043

For now, we are going to implement two features:

  1. Users could set the initial position of the terminal window using the json settings file.
  2. Users could choose to maximize the window once the App is launched.

The current PR is only for the spec review. There is no code change yet.

References

PR Checklist

  • Closes #xxx
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Requires documentation to be 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

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/2654 **Author:** [@KaiyuWang16](https://github.com/KaiyuWang16) **Created:** 9/4/2019 **Status:** ✅ Merged **Merged:** 1/7/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/kawa/1043-set-initial-position-for-the-terminal` --- ### 📝 Commits (10+) - [`9e85571`](https://github.com/microsoft/terminal/commit/9e85571ac4d4c53963fc051c035ffb9230e09f68) Spec version 1 for set terminal initial position - [`dbbba4e`](https://github.com/microsoft/terminal/commit/dbbba4e9c817564a62370a81ca1422d66bcbd2f7) Fix the spec title - [`5a919c7`](https://github.com/microsoft/terminal/commit/5a919c7af46baef97c48b75fd66bf966c0b83ef0) Add a new function to read initial position properties - [`5e8dc81`](https://github.com/microsoft/terminal/commit/5e8dc8106a1321119bbf20fbcff143b604ffdd5c) Merge branch 'master' into dev/kawa/1043-set-initial-position-for-the-terminal - [`5e947ac`](https://github.com/microsoft/terminal/commit/5e947ac9bba357146a3e958f1c669f4bc0da9b40) Test version - 9-6 - [`8fbe307`](https://github.com/microsoft/terminal/commit/8fbe307db22e5a44e5fa7ee589b05d1df6aa934d) spec for code review updates - [`fa5f5c7`](https://github.com/microsoft/terminal/commit/fa5f5c7ab25271ef13c5f3ed5a922738374c85ef) Add edge cases - window hangs off the screen - [`3eac231`](https://github.com/microsoft/terminal/commit/3eac231c58593d0c85f91a32b5fd9dee4d0f4b64) Merge branch 'dev/kawa/1043-set-initial-position-for-the-terminal' of https://github.com/microsoft/terminal into dev/kawa/1043-set-initial-position-for-the-terminal - [`33d37ca`](https://github.com/microsoft/terminal/commit/33d37ca12742c5655645247f775fa50e6a172336) remove window partialy hangs off edge case - [`d4494a5`](https://github.com/microsoft/terminal/commit/d4494a55bafb3ed61e1ce2c3532da8abfb4de25c) Remove unnecessary code changes and update spec for edge cases ### 📊 Changes **1 file changed** (+100 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `doc/specs/#1043 - Set the initial position of the Terminal/spec.md` (+100 -0) </details> ### 📄 Description <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> This PR is for the GitHub task: https://github.com/microsoft/terminal/issues/1043 For now, we are going to implement two features: 1. Users could set the initial position of the terminal window using the json settings file. 2. Users could choose to maximize the window once the App is launched. The current PR is only for the spec review. There is no code change yet. <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [ ] Closes #xxx * [ ] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [ ] Requires documentation to be 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 <!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:06:43 +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#25014