[PR #12435] [MERGED] Fix browse buttons getting cut off when the window is too narrow #29035

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/12435
Author: @PankajBhojwani
Created: 2/8/2022
Status: ✅ Merged
Merged: 2/8/2022
Merged by: @DHowett

Base: main ← Head: dev/pabhoj/text_box_fix


📝 Commits (2)

📊 Changes

3 files changed (+25 additions, -19 deletions)

View changed files

📝 src/cascadia/TerminalSettingsEditor/Appearances.xaml (+10 -8)
📝 src/cascadia/TerminalSettingsEditor/CommonResources.xaml (+1 -1)
📝 src/cascadia/TerminalSettingsEditor/Profiles_Base.xaml (+14 -10)

📄 Description

Summary of the Pull Request

With the recent change to allow text boxes to be bigger, the Browse button that some of them have was getting cut off when the window was too narrow. This change puts the Browse button below the text box instead of next to it to prevent this issue.

PR Checklist

Validation Steps Performed

commandlinenarrow commandlinewide startingDir

🔄 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/12435 **Author:** [@PankajBhojwani](https://github.com/PankajBhojwani) **Created:** 2/8/2022 **Status:** ✅ Merged **Merged:** 2/8/2022 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `main` ← **Head:** `dev/pabhoj/text_box_fix` --- ### 📝 Commits (2) - [`173b221`](https://github.com/microsoft/terminal/commit/173b2215286678043e0bd72a80f9fc4485e86afc) commandline box fix - [`deac1c0`](https://github.com/microsoft/terminal/commit/deac1c063af33d3a951b3fd911d59ef3eea0dbf4) fix other text boxes ### 📊 Changes **3 files changed** (+25 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalSettingsEditor/Appearances.xaml` (+10 -8) 📝 `src/cascadia/TerminalSettingsEditor/CommonResources.xaml` (+1 -1) 📝 `src/cascadia/TerminalSettingsEditor/Profiles_Base.xaml` (+14 -10) </details> ### 📄 Description ## Summary of the Pull Request With the recent change to allow text boxes to be bigger, the `Browse` button that some of them have was getting cut off when the window was too narrow. This change puts the `Browse` button below the text box instead of next to it to prevent this issue. ## PR Checklist * [x] Closes #12335 * [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 work here ## Validation Steps Performed <img width="348" alt="commandlinenarrow" src="https://user-images.githubusercontent.com/26824113/153058035-ecbea7f7-849c-4cae-8b0c-0c9372b6ece5.png"> <img width="903" alt="commandlinewide" src="https://user-images.githubusercontent.com/26824113/153058056-7224140d-0708-4629-8b07-345107eb6f98.png"> <img width="552" alt="startingDir" src="https://user-images.githubusercontent.com/26824113/153058076-35281475-2d5d-40ab-be8c-5db14e493212.png"> --- <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:32:22 +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#29035