Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wpf: Add AutoFill to control whether the connection/buffer resizes #7853

Merged
9 commits merged into from
Oct 9, 2020

Conversation

javierdlg
Copy link
Member

@javierdlg javierdlg commented Oct 7, 2020

Adds the ability to manually handle the terminal renderer resizing
events by allowing different render size and WPF control size. This is
done by adding an AutoFill property to the control that prevents the
renderer from automatically resizing and tells the WPF control to fill
in the extra space with the terminal background as shown below:

This PR adds the following:

  • Helper method in the DX engine to convert character viewports into
    pixel viewports
  • AutoFill property that prevents automatic resizing of the renderer
  • Tweaks and fixes that automatically fill in the empty space if
    AutoFill is set to false
  • Fixes resizing methods and streamlines their codepath

Validation Steps Performed

Manual validation with the Visual Studio Integrated Terminal tool
window.

@javierdlg javierdlg requested a review from miniksa October 7, 2020 23:05
@javierdlg javierdlg changed the title Dev/jadelaga/auto fill control Add ability to manually control renderer resizing and automatically fill empty space Oct 7, 2020
@javierdlg javierdlg requested a review from DHowett October 9, 2020 19:50
Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

This is great. Thanks!

@DHowett DHowett changed the title Add ability to manually control renderer resizing and automatically fill empty space WPF: Add AutoFill, which controls whether the connection/buffer resizes Oct 9, 2020
@DHowett DHowett changed the title WPF: Add AutoFill, which controls whether the connection/buffer resizes wpf: Add AutoFill to control whether the connection/buffer resizes Oct 9, 2020
@DHowett DHowett added the AutoMerge Marked for automatic merge by the bot when requirements are met label Oct 9, 2020
@ghost
Copy link

ghost commented Oct 9, 2020

Hello @DHowett!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 9e86e29 into master Oct 9, 2020
@ghost ghost deleted the dev/jadelaga/AutoFillControl branch October 9, 2020 22:26
skyline75489 pushed a commit to skyline75489/terminal that referenced this pull request Oct 10, 2020
…icrosoft#7853)

Adds the ability to manually handle the terminal renderer resizing
events by allowing different render size and WPF control size. This is
done by adding an `AutoFill` property to the control that prevents the
renderer from automatically resizing and tells the WPF control to fill
in the extra space with the terminal background as shown below:

This PR adds the following:
- Helper method in the DX engine to convert character viewports into
  pixel viewports
- `AutoFill` property that prevents automatic resizing of the renderer
- Tweaks and fixes that automatically fill in the empty space if
  `AutoFill` is set to false
- Fixes resizing methods and streamlines their codepath

## Validation Steps Performed
Manual validation with the Visual Studio Integrated Terminal tool
window.
DHowett pushed a commit that referenced this pull request Oct 13, 2020
…7853)

Adds the ability to manually handle the terminal renderer resizing
events by allowing different render size and WPF control size. This is
done by adding an `AutoFill` property to the control that prevents the
renderer from automatically resizing and tells the WPF control to fill
in the extra space with the terminal background as shown below:

This PR adds the following:
- Helper method in the DX engine to convert character viewports into
  pixel viewports
- `AutoFill` property that prevents automatic resizing of the renderer
- Tweaks and fixes that automatically fill in the empty space if
  `AutoFill` is set to false
- Fixes resizing methods and streamlines their codepath

## Validation Steps Performed
Manual validation with the Visual Studio Integrated Terminal tool
window.

(cherry picked from commit 9e86e29)
DHowett pushed a commit that referenced this pull request Oct 13, 2020
…7853)

Adds the ability to manually handle the terminal renderer resizing
events by allowing different render size and WPF control size. This is
done by adding an `AutoFill` property to the control that prevents the
renderer from automatically resizing and tells the WPF control to fill
in the extra space with the terminal background as shown below:

This PR adds the following:
- Helper method in the DX engine to convert character viewports into
  pixel viewports
- `AutoFill` property that prevents automatic resizing of the renderer
- Tweaks and fixes that automatically fill in the empty space if
  `AutoFill` is set to false
- Fixes resizing methods and streamlines their codepath

## Validation Steps Performed
Manual validation with the Visual Studio Integrated Terminal tool
window.

(cherry picked from commit 9e86e29)
@ghost
Copy link

ghost commented Nov 11, 2020

🎉Windows Terminal v1.4.3141.0 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link

ghost commented Nov 11, 2020

🎉Windows Terminal Preview v1.5.3142.0 has been released which incorporates this pull request.:tada:

Handy links:

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge Marked for automatic merge by the bot when requirements are met
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants