initialRows still isn't working completely right #8843

Closed
opened 2026-01-31 01:39:23 +00:00 by claunia · 5 comments
Owner

Originally created by @keathmilligan on GitHub (Jun 5, 2020).

Environment

Windows build number: 10.0.19041.0
Windows Terminal version (if applicable): 1.0.1401.0

Steps to reproduce

Set InitialRows to 40.
Open Windows Terminal.
In Powershell, run:

for ($i = 1; $i -le 40; $i++) { Write-Host; Write-Host -NoNewLine $i }; [console]::ReadKey()

Expected behavior

You should see all 40 rows.

Actual behavior

Instead, you only see rows 2-40. The area of the window that should contain the last row is not used:

image

Now grab the bottom edge of the window, shrink it by one and then return it to its original size. Run the script above again and see that all 40 rows are now displayed with the last row being utilized.

image

Originally created by @keathmilligan on GitHub (Jun 5, 2020). # Environment ```none Windows build number: 10.0.19041.0 Windows Terminal version (if applicable): 1.0.1401.0 ``` # Steps to reproduce Set InitialRows to 40. Open Windows Terminal. In Powershell, run: ```powershell for ($i = 1; $i -le 40; $i++) { Write-Host; Write-Host -NoNewLine $i }; [console]::ReadKey() ``` # Expected behavior You should see all 40 rows. # Actual behavior Instead, you only see rows 2-40. The area of the window that should contain the last row is not used: ![image](https://user-images.githubusercontent.com/2936304/83914040-132cd800-a736-11ea-918d-f2bda99111ad.png) Now grab the bottom edge of the window, shrink it by one and then return it to its original size. Run the script above again and see that all 40 rows are now displayed with the last row being utilized. ![image](https://user-images.githubusercontent.com/2936304/83914091-2b9cf280-a736-11ea-8891-6e47320b9d95.png)
Author
Owner

@DHowett commented on GitHub (Jun 5, 2020):

Huh, I can't reproduce this.

  • What's your DPI display scaling?
  • Can you run mode con?
  • Can you select the blank line under 40?
@DHowett commented on GitHub (Jun 5, 2020): Huh, I can't reproduce this. * What's your ~DPI~ display scaling? * Can you run `mode con`? * _Can you select the blank line under 40?_
Author
Owner

@keathmilligan commented on GitHub (Jun 5, 2020):

DPI scaling is 150% on a 4k display.

When I first open the window, before manually resizing, mode con shows 39 lines even though InitialRows is set to 40. And no, the blank line cannot be selected.

@keathmilligan commented on GitHub (Jun 5, 2020): DPI scaling is 150% on a 4k display. When I first open the window, before manually resizing, mode con shows 39 lines even though InitialRows is set to 40. And no, the blank line cannot be selected.
Author
Owner

@DHowett commented on GitHub (Jun 6, 2020):

Thanks! I thought we had a duplicate of this, but I think we thought we'd fixed it.

@DHowett commented on GitHub (Jun 6, 2020): Thanks! I thought we had a duplicate of this, but I think we thought we'd fixed it.
Author
Owner

@zadjii-msft commented on GitHub (Feb 9, 2021):

Hey I think we fixed this in 1.5 / 1.6 - there was #8527 (which has almost exactly the same title) which should be fixed

🎉This issue was addressed in #8584, which has now been successfully released as Windows Terminal Preview v1.6.10272.0.🎉

I'm more shocked that we didn't dupe that other issue to this one in the first place

/dup #8527

@zadjii-msft commented on GitHub (Feb 9, 2021): Hey I think we fixed this in 1.5 / 1.6 - there was #8527 (which has almost exactly the same title) which should be fixed > 🎉This issue was addressed in #8584, which has now been successfully released as Windows Terminal Preview v1.6.10272.0.🎉 I'm more shocked that we didn't dupe that other issue to this one in the first place /dup #8527
Author
Owner

@ghost commented on GitHub (Feb 9, 2021):

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost commented on GitHub (Feb 9, 2021): Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#8843